Updated to latest versions of Jetty, Jersey, Jackson and Guice #2

Merged
merged 4 commits into from May 18, 2012

Conversation

Projects
None yet
2 participants
Contributor

cjsmith commented May 18, 2012

Also added ablility to generate executable jar with mvn package and simple README.

+ <dependency>
+ <groupId>org.codehaus.jackson</groupId>
+ <artifactId>jackson-mapper-asl</artifactId>
+ <version>1.9.6</version>
@sunnygleason

sunnygleason May 18, 2012

Owner

missed a property substitution - I'll merge the pull request & fix it

+ <dependency>
+ <groupId>org.codehaus.jackson</groupId>
+ <artifactId>jackson-jaxrs</artifactId>
+ <version>1.9.6</version>
@sunnygleason

sunnygleason May 18, 2012

Owner

ditto as above, will fix

+ <jersey.version>1.12</jersey.version>
+ <jackson.version>1.9.6</jackson.version>
+ <guice.version>3.0</guice.version>
+ </properties>
@sunnygleason

sunnygleason May 18, 2012

Owner

this is really nice!

root.addEventListener(new SampleConfig());
- root.addFilter(GuiceFilter.class, "/*", 0);
-
+ root.addFilter(GuiceFilter.class, "/*", EnumSet.of(DispatcherType.REQUEST));
@sunnygleason

sunnygleason May 18, 2012

Owner

Much nicer than zero :)

@@ -31,11 +33,12 @@ protected void configureServlets() {
/* bind jackson converters for JAXB/JSON serialization */
bind(MessageBodyReader.class).to(JacksonJsonProvider.class);
bind(MessageBodyWriter.class).to(JacksonJsonProvider.class);
-
+ Map<String, String> initParams = new HashMap<String, String>();
+ initParams.put("com.sun.jersey.config.feature.Trace",
@sunnygleason

sunnygleason May 18, 2012

Owner

Presuming this is to remove the dependency on immutable map?

@cjsmith

cjsmith May 18, 2012

Contributor

Yup. You could add google collections if you want to get it back.

@sunnygleason

sunnygleason May 18, 2012

Owner

Right on, this looks excellent, less dependencies == more goodness :)

Owner

sunnygleason commented May 18, 2012

This is an awesome contribution. Thank you so much. I'm merging now and making the couple small tweaks I mentioned. Thanks again!

sunnygleason added a commit that referenced this pull request May 18, 2012

Merge pull request #2 from cjsmith/master
Updated to latest versions of Jetty, Jersey, Jackson and Guice

@sunnygleason sunnygleason merged commit 49b8ac0 into sunnygleason:master May 18, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment