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

Application Config should not inject null #199

Closed
cescoffier opened this issue Dec 6, 2018 · 0 comments · Fixed by #205
Closed

Application Config should not inject null #199

cescoffier opened this issue Dec 6, 2018 · 0 comments · Fixed by #205
Labels
kind/bug Something isn't working
Milestone
0.2.0

Comments

@cescoffier
Copy link
Member

The microprofile spec does not allow injecting null, while shamrock does inject null:

@Inject
@ConfigProperty(name="not-in-config")
String boom

If the value is optional, 2 approaches:
@cescoffier cescoffier added the kind/bug Something isn't working label Dec 6, 2018
@mkouba mkouba mentioned this issue Dec 6, 2018
Merged
@cescoffier cescoffier added this to the 0.2.0 milestone Jan 21, 2019
maxandersen added a commit to maxandersen/quarkus that referenced this issue Nov 5, 2022
@maxandersen
Merge pull request quarkusio#199 from quintesse/moar_verbose
7ba4287 
Added a bunch of verbose debug messages to the code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working
Projects
None yet
Milestone
0.2.0
Development

Successfully merging a pull request may close this issue.

Validate config property injection points
2 participants
@cescoffier @mkouba