[SLING-11969] Improve the ConfigurableSerializationModuleProvider

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: Models Jackson Exporter 1.1.6
• Component/s: Sling Models
• Assignee: Joerg Hoh
• Reporter: Joerg Hoh
• Created: 18/Jul/23 12:35
• Updated: 09/Feb/24 11:26
• Resolved: 09/Feb/24 11:26

Description

(Feedback taken over from SLING-11924)

• https://github.com/apache/sling-org-apache-sling-models-jacksonexporter/pull/7/files contains several commented out lines of code and TODO tags, would be good to get rid of all those in the final code base
• the comments in the OSGi config definition should explicitly state that currently only org.apache.sling.api.resource.ResourceResolver is supported. unlike the OSGi config definition itself we can change the comments at any time if we want to support more classes.
• and if a different class name is given for any of those parameters, a warning should be logged that it has no effect.

Issue Links

• is related to: SLING-11924 ModelExporter should not serialize a ResourceResolver
• links to: GitHub Pull Request #10