[
https://issues.apache.org/jira/browse/SLING-8777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16951695#comment-16951695
]
Karl Pauls commented on SLING-8777:
-----------------------------------
[~ashokpanghal], I think this is not a bug as such. That the saaj impl expects
the CommandMap to be a MailcapCommandMap seems problematic but I would rather
argue it is a problem with saaj rather than the activation bundle. That said, I
think that changing it to be a MailcapCommandMap would still be a nice
improvement in that case - however, we would need to try to make it work
correctly. The patch is a good start but it would need to work the
MailcapCommandMap methods into the rest of the implementation and it seems like
the addMailcap method right now is replacing previous entries (instead of
appending) which I'm not sure is correct.
Could you try to work the patch a little to make the MailcapCommandMap work
better with the rest of the impl?
> Sling activation bundle overrides default CommandMap with wrong type
> --------------------------------------------------------------------
>
> Key: SLING-8777
> URL: https://issues.apache.org/jira/browse/SLING-8777
> Project: Sling
> Issue Type: Bug
> Components: Commons
> Affects Versions: javax.activation 0.1.0
> Reporter: Ashok Kumar
> Priority: Major
> Fix For: javax.activation 0.2.0
>
> Attachments: SLING-8777.patch
>
>
> The root cause of the issue is that the bundle
> org.apache.sling.javax.activation 0.1.0 overrides the Default
> javax.activation.CommandMap extending directly from CommandMap instead of
> MailcapCommandMap:
> [https://github.com/apache/sling-org-apache-sling-javax-activation/blob/master/src/main/java/org/apache/sling/javax/activation/internal/OsgiMailcapCommandMap.java#L50]
> The call fails due to this, here:
> [https://github.com/samskivert/ikvm-openjdk/blob/master/build/linux-amd64/impsrc/com/sun/xml/internal/messaging/saaj/soap/AttachmentPartImpl.java#L112]
> There is also a missing dependency of mimepull. This dependency needs to be
> added to AEM as well:
> [http://repo1.maven.org/maven2/org/jvnet/mimepull/mimepull/1.9.7/mimepull-1.9.7.jar]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
