Comments (5)
I believe this is one of the tasks that should have been solvable by HeaderDelegate
and associated HeaderDelegateProvider
SPI, so that each customer can specify whatever he desires. The SPI has been a bit overlooked in the past, but that's the idea.
Would it help in your case?
from jersey.
Thank you, @jansupol! That sounds like an interesting idea! Let me play with it and see if I can get it to work!
from jersey.
@jansupol, I was able to pinpoint where the whitespace gets stripped in Jersey 2:
- Jersey 1.x:
package com.sun.jersey.core.impl.provider.header;
public class MediaTypeProvider implements HeaderDelegateProvider<MediaType> {
@Override
public String toString(MediaType header) {
StringBuilder b = new StringBuilder();
b.append(header.getType()).append('/').append(header.getSubtype());
for (Map.Entry<String, String> e : header.getParameters().entrySet()) {
b.append("; ").append(e.getKey()).append('='); <<<<== whitespace added after semicolon
WriterUtil.appendQuotedIfNonToken(b, e.getValue());
}
return b.toString();
}
...
}
- Jersey 2:
package org.glassfish.jersey.message.internal;
public class MediaTypeProvider implements HeaderDelegateProvider<MediaType> {
@Override
public String toString(MediaType header) {
throwIllegalArgumentExceptionIfNull(header, MEDIA_TYPE_IS_NULL);
StringBuilder b = new StringBuilder();
b.append(header.getType()).append('/').append(header.getSubtype());
for (Map.Entry<String, String> e : header.getParameters().entrySet()) {
b.append(";").append(e.getKey()).append('='); <<<<== no more whitespace added after semicolon
StringBuilderUtils.appendQuotedIfNonToken(b, e.getValue());
}
return b.toString();
}
...
}
from jersey.
@jansupol, can you tell me how I can replace/override Jersey's MediaTypeProvider with my own?
I see Jersey's RuntimeDelegateImpl
getting initialized as follows:
package org.glassfish.jersey.server.internal;
public RuntimeDelegateImpl() {
super(new MessagingBinders.HeaderDelegateProviders().getHeaderDelegateProviders());
}
where the MessagingBinders.HeaderDelegateProviders
constructor uses Jersey's built-in HeaderDelegateProviders:
public HeaderDelegateProviders() {
Set<HeaderDelegateProvider> providers = new HashSet<>();
providers.add(new CacheControlProvider());
providers.add(new CookieProvider());
providers.add(new DateProvider());
providers.add(new EntityTagProvider());
providers.add(new LinkProvider());
providers.add(new LocaleProvider());
providers.add(new MediaTypeProvider());
providers.add(new NewCookieProvider());
providers.add(new StringHeaderProvider());
providers.add(new UriProvider());
this.providers = providers;
}
I had hoped it would also consider any custom HeaderDelegateProviders declared in META-INF/services
as described here: https://eclipse-ee4j.github.io/jersey.github.io/apidocs/2.34/jersey/org/glassfish/jersey/spi/HeaderDelegateProvider.html
Do you have any recommendation for how to make this work? I noticed the same question was asked here: https://stackoverflow.com/questions/62297701/how-to-inject-custom-implementation-of-org-glassfish-jersey-spi-headerdelegatepr, but it was never answered.
Thank you!
from jersey.
@jansupol, what I ended up doing is extend org.glassfish.jersey.server.internal.RuntimeDelegateImpl
and have my subclass (declared in META-INF/services/javax.ws.rs.ext.RuntimeDelegate
) override createHeaderDelegate
...
from jersey.
Related Issues (20)
- jersey-netty-connector - Cancelling the future returned by rx() client does not terminate the request HOT 4
- Project Loom/JDK 21 compatible HOT 8
- Inquiry about Jersey Project Licensing and Third-Party Content Usage HOT 7
- jersey 3.1 default exception mapper message interferes with setStatusOverSendError=true
- [Feature request] Configuration options for multipart requests (Jersey 3.1.x) HOT 8
- `UriRoutingContext` may not log tracing events
- Each HTTP request unnecessarily starts a new thread, resource leak. HOT 6
- HttpUrlConnector to support domain fronting HOT 2
- Jersey 3.1.6 release HOT 2
- Jersey 3.0.x has no working provider that support Java 17, PATCH, and multipart attachments HOT 3
- JdkConnector does not properly cleanup ERROR state connections HOT 3
- NettyConnector does not respect timeouts HOT 7
- CVE-2023-4043 for jersey-media-json-binding dependency (parsson-1.1.1) HOT 2
- Injection error when using a `DynamicFeature` HOT 3
- Possible NPE in RequestContextFilter when other Filters prioritized
- Carrier Thread pinning on stack CommittingOutputStream.flushBuffer() HOT 9
- Jersey 3.1.7 release HOT 5
- Jersey 3.1.6 not closing request on Jetty earlyEOF HOT 3
- Connection is not stable (wrong detection if SSL context is configured) HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from jersey.