Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow to use JSON-P without RESTEasy #1480

Closed
gunnarmorling opened this issue Mar 14, 2019 · 3 comments
Closed

Allow to use JSON-P without RESTEasy #1480

gunnarmorling opened this issue Mar 14, 2019 · 3 comments
Assignees
@gsmet
Milestone
0.12.0

Comments

@gunnarmorling
Copy link
Contributor

We had issues to use JSON-P due to missing reflection configs. As I learned from @gsmet, the required config is contributed through the RESTEasy module, while our service doesn't use that (using JSON-P directly to deserialize JSON payloads received from Kafka).
@gsmet
Copy link
Member

gsmet commented Mar 14, 2019

@gunnarmorling could you add some details about the API calls you make so that we can add the appropriate test cases?

@gsmet gsmet self-assigned this Mar 14, 2019
@gsmet gsmet mentioned this issue Mar 14, 2019
Closed
@gunnarmorling
Copy link
Contributor Author

@gsmet, you can see the original service here: https://github.com/debezium/debezium-examples/blob/master/outbox/shipment-service/src/main/java/io/debezium/examples/outbox/shipment/facade/OrderEventHandler.java#L42-L59 It's this usage and within the called ShipmentService.

gsmet added a commit to gsmet/quarkus that referenced this issue Mar 15, 2019
@gsmet
Introduce JSON-B and JSON-P extensions
19092ac 
I haven't added a FeatureBuildItem as I don't think they are worth an
item in the feature list at startup.

Fixes quarkusio#1480 and quarkusio#1482 and a few other issues in passing.
@gsmet gsmet mentioned this issue Mar 15, 2019
Merged
gsmet added a commit to gsmet/quarkus that referenced this issue Mar 16, 2019
@gsmet
Introduce JSON-B and JSON-P extensions
27dd081 
I haven't added a FeatureBuildItem as I don't think they are worth an
item in the feature list at startup.

Fixes quarkusio#1480 and quarkusio#1482 and a few other issues in passing.
gsmet added a commit to gsmet/quarkus that referenced this issue Mar 19, 2019
@gsmet
Introduce JSON-B and JSON-P extensions
ad96240 
I haven't added a FeatureBuildItem as I don't think they are worth an
item in the feature list at startup.

Fixes quarkusio#1480 and quarkusio#1482 and a few other issues in passing.
@gsmet gsmet added this to the 0.12.0 milestone Mar 19, 2019
@gsmet
Copy link
Member

gsmet commented Mar 19, 2019

@gunnarmorling you will have quarkus-jsonp/quarkus-jsonb extensions in tomorrow's release.

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

@gsmet gsmet

Labels
None yet
Projects
None yet
Milestone
0.12.0
Development

No branches or pull requests
2 participants
@gunnarmorling @gsmet