Skip to content

OpenAPI Auto security works with enabled-in-dev config from security #47933

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
Merged
merged 1 commit into from
May 20, 2025
Merged

OpenAPI Auto security works with enabled-in-dev config from security #47933

merged 1 commit into from
May 20, 2025

Conversation

phillip-kruger
Copy link
Member

@phillip-kruger phillip-kruger commented May 19, 2025 •
edited by geoand
Loading

@phillip-kruger phillip-kruger requested a review from gsmet May 19, 2025 02:13
@phillip-kruger
Copy link
Member Author

@melloware - if you can, please test with your case.

@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented May 19, 2025 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 59bf81c.

✅ 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.

You can consult the Develocity build scans.

melloware
melloware approved these changes May 19, 2025
View reviewed changes
Copy link
Contributor

@melloware melloware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent this is working and what I expected!

@gsmet gsmet merged commit 0e94356 into quarkusio:main May 20, 2025
30 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.24 - main milestone May 20, 2025
@gsmet gsmet modified the milestones: 3.24 - main, 3.23.0 May 20, 2025
@aloubyansky
Copy link
Member

Unfortunately, this change broke Maven DevModeIT tests, which weren't included in the CI for this PR. You can see them failing in #47963, for example.

@aloubyansky aloubyansky mentioned this pull request May 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gsmet gsmet gsmet approved these changes

+1 more reviewer

@melloware melloware melloware approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area/openapi area/smallrye

Projects

None yet

Milestone

3.23.0

Development

Successfully merging this pull request may close these issues.

Security: quarkus.security.auth.enabled-in-dev-mode=false breaks SwaggerUI

4 participants

@phillip-kruger @aloubyansky @gsmet @melloware