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

Error retrieving quarkus.http.auth.form.cookie-name #1270

Closed
mcollovati opened this issue Feb 7, 2025 · 1 comment · Fixed by #1276
Closed

Error retrieving quarkus.http.auth.form.cookie-name #1270

mcollovati opened this issue Feb 7, 2025 · 1 comment · Fixed by #1276
Assignees
@mcollovati
Labels
bug Something isn't working released Issue has been released security

Comments

@mcollovati
Copy link
Owner

After Quarkus refactoring in quarkusio/quarkus#45769 the quarkus.http.auth.form.cookie-name setting is available only at runtime.
HillaSecurityRecorder.setupFormAuthenticationMechanism should be changed to avoid getting the value at build time, postponing the lookup in the supplier instance or in the constructor of HillaFormAuthenticationMechanism.

  Caused by: java.util.NoSuchElementException: SRCFG00014: The config property quarkus.http.auth.form.cookie-name is required but it could not be found in any config source
  	at io.smallrye.config.SmallRyeConfig.convertValue(SmallRyeConfig.java:507)
  	at io.smallrye.config.SmallRyeConfig.getValue(SmallRyeConfig.java:452)
  	at io.smallrye.config.SmallRyeConfig.getValue(SmallRyeConfig.java:431)
  	at com.github.mcollovati.quarkus.hilla.security.HillaSecurityRecorder.setupFormAuthenticationMechanism(HillaSecurityRecorder.java:34)
  	at io.quarkus.runner.recorded.QuarkusHillaSecurityProcessor$registerHillaFormAuthenticationMechanism1975585193.deploy_0(Unknown Source)
  	at io.quarkus.runner.recorded.QuarkusHillaSecurityProcessor$registerHillaFormAuthenticationMechanism1975585193.deploy(Unknown Source)
  	... 8 more
@mcollovati mcollovati added the bug Something isn't working label Feb 7, 2025
@mcollovati mcollovati moved this from 🆕 New to 🔖 Ready in quarkus-hilla backlog Feb 7, 2025
mcollovati added a commit that referenced this issue Feb 7, 2025
@mcollovati
fix: retrieve cookie name at runtime
066266d 
Quarkus 3.19 refactors HTTP configuration in a way that cookie name
is available only at runtime.
This change move the configuration retrival inside the Supplier
returned by setupFormAuthenticationMechanism so that the configuration
can be retrieved.

Fixes #1270
@mcollovati mcollovati self-assigned this Feb 7, 2025
@mcollovati mcollovati moved this from 🔖 Ready to 🏗 In progress in quarkus-hilla backlog Feb 7, 2025
@mcollovati mcollovati mentioned this issue Feb 7, 2025
Merged
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in quarkus-hilla backlog Feb 8, 2025
@github-actions GitHub Actions
Copy link

This issue has been resolved in 24.7.0-alpha2 (Release
Notes)

@github-actions github-actions bot added the released Issue has been released label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcollovati mcollovati

Labels
bug Something isn't working released Issue has been released security
Projects
quarkus-hilla backlog
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: retrieve cookie name at runtime mcollovati/quarkus-hilla
1 participant
@mcollovati