[SLING-11656] Prevent AbstractMethodError in SlingHttpServletRequestImpl

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Engine 2.12.0
• Fix Version/s: Engine 2.12.2
• Component/s: Engine
• Assignee: Carsten Ziegeler
• Reporter: Carsten Ziegeler
• Created: 31/Oct/22 10:40
• Updated: 14/Nov/22 05:57
• Resolved: 31/Oct/22 10:55

Description

The recent refactoring of Sling's engine can lead to AbstractMethodErrors in SlingHttpServletRequestImpl due to old/buggy code. The problem is that the updated engine code is counting on a proper servlet 3.0 request implementation, returning a proper ServletContext (via getServletContext()) and that one properly implementing getMimeType().
It turns out that this is not the case in the wild which then leads to the error.

In addition, even if the request implementations are complete, it is a change in implementation - as the mime type service is most likely not used anymore