[
https://issues.apache.org/jira/browse/WINK-47?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12729491#action_12729491
]
Mike Rheinheimer commented on WINK-47:
--------------------------------------
Yes, this Jira is specific to Issue 1. The more I think about it, the more I
think that the * character in the MediaType passed to the
ProvidersRegistry.getContextResolver should be treated as a wildcard. I think
forcing it to be a string literal match would probably go against the general
understanding of *. I think that's why Nick pointed out the Accept-header
section of the html spec; just to indicate that it is discussed as a wildcard.
The JAX-RS spec does not explicitly say it should be treated as a wildcard, but
I think this usage has been so pervasive, I can't imagine the JAX-RS spec
authors intended that * be matched as a string literal.
Thoughts?
How do we come to a consensus on this? I don't want to pollute the mailing
list with "vote" emails for a Jira, but if that's the right process, we can go
that way. My informal vote is for patch_02.txt as the fix.
> honor media type wildcards
> --------------------------
>
> Key: WINK-47
> URL: https://issues.apache.org/jira/browse/WINK-47
> Project: Wink
> Issue Type: Bug
> Components: Common
> Affects Versions: 0.1
> Reporter: Mike Rheinheimer
> Fix For: 0.1
>
> Attachments: patch.txt, patch_02.txt
>
>
> This issue probably needs discussion, as it may be a spec interpretation
> issue. Please reference JAX-RS javadoc for
> javax.ws.rs.ext.Providers.getContextResolver here:
> http://jackson.codehaus.org/javadoc/jax-rs/1.0/javax/ws/rs/ext/Providers.html#getContextResolver(java.lang.Class,%20javax.ws.rs.core.MediaType)
> Currently, the implementation does not use the '*' character in a passed-in
> media type as a wildcard. Rather, it matches exactly by just allowing the
> LinkedHashMap.get to it's object compare. Attached is a patch with tests
> that demonstrate the proposed function, and a fix that supports it. All
> existing and new tests in wink-common and wink-server passed for me with this
> change.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
