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

Jaxrs: user annotated Providers are not registered. #490

Closed
FroMage opened this issue Jan 13, 2019 · 2 comments
Closed

Jaxrs: user annotated Providers are not registered. #490

FroMage opened this issue Jan 13, 2019 · 2 comments
Milestone
0.6.0

Comments

@FroMage
Copy link
Member

FroMage commented Jan 13, 2019

We register default providers and those declared in /META-INF/services but not those in the application index that are just annotated @Provider.
@gsmet
Copy link
Member

gsmet commented Jan 13, 2019

Are we supposed to discover them automatically? The user is not required to add a service file with them?

stuartwdouglas referenced this issue in stuartwdouglas/quarkus-old Jan 14, 2019
@stuartwdouglas
Fixes #490, custom @Provider classes are not registered
5aca5b7
stuartwdouglas referenced this issue in stuartwdouglas/quarkus-old Jan 14, 2019
@stuartwdouglas
Fixes #490, custom @Provider classes are not registered
5d6e0f4
@FroMage
Copy link
Member Author

FroMage commented Jan 14, 2019

Are we supposed to discover them automatically? The user is not required to add a service file with them?

We detect resources, we must detect the rest ;)

stuartwdouglas referenced this issue in stuartwdouglas/quarkus-old Jan 14, 2019
@stuartwdouglas
Fixes #490, custom @Provider classes are not registered
e9bf79c
stuartwdouglas added a commit that referenced this issue Jan 15, 2019
@stuartwdouglas
Merge pull request #491 from stuartwdouglas/490
b2f3986 
Fixes #490, custom @Provider classes are not registered
@cescoffier cescoffier added this to the 0.6.0 milestone Jan 16, 2019
maxandersen added a commit to maxandersen/quarkus that referenced this issue Nov 5, 2022
@maxandersen
feat: run jar from a https url
890787f 
Can now do `jbang https://bintray.com/cardillo/maven/download_file?file_path=joinery%2Fjoinery-dataframe%2F1.9%2Fjoinery-dataframe-1.9-jar-with-dependencies.jar`
without it resolving bad classpath.

Fixes quarkusio#490
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
3 participants
@FroMage @cescoffier @gsmet