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

Index custom JAX-RS providers #33262

Closed
matejvasek opened this issue May 10, 2023 · 5 comments · Fixed by #33267
Closed

Index custom JAX-RS providers #33262

matejvasek opened this issue May 10, 2023 · 5 comments · Fixed by #33267
Assignees
@geoand
Labels
area/rest kind/enhancement New feature or request
Milestone
3.1.0.CR1

Comments

@matejvasek
Copy link
Contributor

matejvasek commented May 10, 2023 •
edited
Loading

Description

When a developer uses custom JAX-RS message reader/writer they are not included in build index.
This may pose difficulty for creating native-image with custom providers -- Quarkus build is not registering endpoints for reflection.

Developer can workaround this by quarkus.index-dependency in application.properties, however it would be nice if this worked automatically.

Implementation ideas

No response
@matejvasek matejvasek added the kind/enhancement New feature or request label May 10, 2023
@matejvasek
Copy link
Contributor Author

I encountered this when trying to make RESTEasy reactive work with custom message writer/reader from CloudEvent SDK.

@matejvasek
Copy link
Contributor Author

/cc @geoand

@matejvasek
Copy link
Contributor Author

related issue: cloudevents/sdk-java#546

geoand added a commit to geoand/quarkus that referenced this issue May 10, 2023
@geoand
Automatically lookup Providers based on the presence of standard Jaka…
0aabc21 
…rta file

Closes: quarkusio#33262
@geoand geoand mentioned this issue May 10, 2023
Merged
@geoand
Copy link
Contributor

geoand commented May 10, 2023

#33267 fixes the issue

@quarkus-bot
Copy link

quarkus-bot bot commented May 10, 2023

/cc @FroMage (resteasy-reactive), @Sgitario (resteasy-reactive), @stuartwdouglas (resteasy-reactive)

@geoand geoand self-assigned this May 10, 2023
geoand added a commit to geoand/quarkus that referenced this issue May 11, 2023
@geoand
Automatically lookup Providers based on the presence of standard Jaka…
5f9c396 
…rta file

Closes: quarkusio#33262
geoand added a commit to geoand/quarkus that referenced this issue May 11, 2023
@geoand
Automatically lookup Providers based on the presence of standard Jaka…
c6ce5bb 
…rta file

Closes: quarkusio#33262
@quarkus-bot quarkus-bot bot added this to the 3.1 - main milestone May 11, 2023
michelle-purcell pushed a commit to michelle-purcell/quarkus that referenced this issue May 11, 2023
@geoand
Merge pull request quarkusio#33267 from geoand/quarkusio#33262
15575ef 
Automatically lookup Providers based on the presence of standard Jakarta file
michelle-purcell pushed a commit to michelle-purcell/quarkus that referenced this issue May 11, 2023
@geoand @michelle-purcell
Automatically lookup Providers based on the presence of standard Jaka…
8d83731 
…rta file

Closes: quarkusio#33262
manofthepeace pushed a commit to manofthepeace/quarkus that referenced this issue May 16, 2023
@geoand @manofthepeace
Automatically lookup Providers based on the presence of standard Jaka…
3ee6bb4 
…rta file

Closes: quarkusio#33262
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoand geoand

Labels
area/rest kind/enhancement New feature or request
Projects
None yet
Milestone
3.1.0.CR1
Development

Successfully merging a pull request may close this issue.

Automatically lookup Providers based on the presence of standard Jakarta file geoand/quarkus
2 participants
@matejvasek @geoand