Skip to content

Conversation

michalvavrik
Copy link
Member

  • allow to configure path-specific authorization in code instead of in application.properties
  • this PR covers portion of Provide a fluent API to set up Quarkus Security #16728
  • more will follow in separate smaller PRs for other configuration (like form auth config, inclusive auth and so on), the specifics needs to be discussed first

This comment has been minimized.

Copy link

github-actions bot commented Jun 19, 2025

🙈 The PR is closed and the preview is expired.

This comment has been minimized.

@michalvavrik michalvavrik force-pushed the feature/fluent-vertx-http-security branch from 68ccfbe to 33b8eba Compare June 19, 2025 21:56

This comment has been minimized.

This comment has been minimized.

@sberyozkin
Copy link
Member

I suggest to set to Draft as I'm trying to find time to comment on a few things at a time, it looks very good, but it might require a few minor consecutive updates to finalize

@michalvavrik
Copy link
Member Author

I suggest to set to Draft as I'm trying to find time to comment on a few things at a time, it looks very good, but it might require a few minor consecutive updates to finalize

Ok.

@michalvavrik michalvavrik marked this pull request as draft June 20, 2025 21:30
@michalvavrik michalvavrik force-pushed the feature/fluent-vertx-http-security branch from 33b8eba to 4fe6054 Compare June 22, 2025 14:53
@sberyozkin sberyozkin marked this pull request as ready for review June 23, 2025 22:00
@sberyozkin
Copy link
Member

@cescoffier FYI, looks good and is part of the effort by @michalvavrik to support fluent API for various parts of Quarkus security

@sberyozkin
Copy link
Member

sberyozkin commented Jun 23, 2025

@michalvavrik I can merge sometime tomorrow, just would like to give @cescoffier some heads up, as some interesting follow ups for TLS registry might be imagined, etc

This comment has been minimized.

@michalvavrik
Copy link
Member Author

I am quite sure that Failing: extensions/smallrye-openapi/deployment only on Windows is not related, but I'll rebase on current main just for a case.

@michalvavrik michalvavrik force-pushed the feature/fluent-vertx-http-security branch from 4fe6054 to 89b0163 Compare June 24, 2025 08:27
Copy link

quarkus-bot bot commented Jun 24, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 89b0163.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@michalvavrik
Copy link
Member Author

I saw the same failures here #48560, hence they are unrelated.

@sberyozkin sberyozkin merged commit e5d3607 into quarkusio:main Jun 24, 2025
56 of 58 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.25 - main milestone Jun 24, 2025
@michalvavrik michalvavrik deleted the feature/fluent-vertx-http-security branch June 24, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants