search

FasterXML / jackson-jakarta-rs-providers

Multi-module project that contains Jackson-based "new" Jakarta-RS (nee "JAX-RS" -- ones under `jakarta.ws.rs`) providers for JSON, XML, YAML, Smile, CBOR formats
Apache License 2.0
14 stars 11 forks source link

Try to validate goodness of Jakarta-RS providers for 2.13 #1

Closed cowtowncoder closed 3 years ago

cowtowncoder commented 3 years ago

This is sort of a meta-task to gather information on what works and (more importantly) what does not, wrt whole JAX-x / Jakarta-x migration.

Users SHOULD be able to use Jakarta-rs (jakarta.ws.rs) Providers from this repo, over JAX-RS variants from https://github.com/FasterXML/jackson-jaxrs-providers -- and this includes use of new Jakarta variants of XML Bind annotations, which come from here:

https://github.com/FasterXML/jackson-modules-base/tree/master/jakarta-xmlbind

But right now I suspect even "may be able to" is an overstatement. So I am looking for help in validating this approach.

cowtowncoder commented 3 years ago

@GedMarc as per above, do you think you could poke providers, see what kind of issues I have missed? :)

GedMarc commented 3 years ago

On it sir!

cowtowncoder commented 3 years ago

Excellent!

cowtowncoder commented 3 years ago

Hi @GedMarc -- have had any time to see how things work (or do not)?

GedMarc commented 3 years ago

hey, yeah I have / just haven't gotten round to testing on the cxf 3.4.4 release (two days ago) just yet,

it looks good so far, let me do one final run through, back in 10

cowtowncoder commented 3 years ago

Np; I have been swamped with things so no hurry. But wanted to see if there are specific findings, things I forgot etc.

Also looks like only one JAX/Jakarta-RS provider (Jersey) has fully migrated (has a new version for Jakarta stuff)? Although others are looking for various approaches. But that's fine, I think we mostly need one for testing.

GedMarc commented 3 years ago

So far tomat and jersey yeah,

oh are you leaving the module-info inside the versions/11 folder? Think I've found something just going to make a quick fork and make sure

cowtowncoder commented 3 years ago

I have no strong opinion on versions/11 or alternatives, open to suggestions. Esp. since these are "new" modules.

cowtowncoder commented 3 years ago

Merged suggested changes so I hope we are closer to good state here?

cowtowncoder commented 3 years ago

Since 2.13.0 is now released we might get some actual Beta testing; closing this issue.