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.

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

Prepend http.root-path to permission paths without '/' #25181

Merged

Conversation

sberyozkin
Copy link
Member

@sberyozkin sberyozkin commented Apr 26, 2022

This PR relates to #25154, it does what I did for 2.8.1 then reverted for 2.8.2, in a better non-breaking way as recommended by Erin.

I will follow up with another PR where I'll go through various OIDC paths carefully and see if the same logic can be applied.

@sberyozkin
Copy link
Member Author

It should be safe for 2.9.0.CR1

Copy link
Member

@ebullient ebullient left a comment

Choose a reason for hiding this comment

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

Writing is hard. See what you think.

docs/src/main/asciidoc/http-reference.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/http-reference.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/security-authorization.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/security-authorization.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/security-authorization.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/security-authorization.adoc Outdated Show resolved Hide resolved
@sberyozkin
Copy link
Member Author

Hi @ebullient These editorial updates are very welcome, thanks

@sberyozkin sberyozkin force-pushed the security_policy_path_with_http_root branch from ddfb47c to c7b1280 Compare April 26, 2022 18:34
@ebullient ebullient added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Apr 26, 2022
@sberyozkin
Copy link
Member Author

Thanks, quarkus-integration-test-jpa-oracle has failed so far, it does not use these permission tests

@quarkus-bot
Copy link

quarkus-bot bot commented Apr 26, 2022

Failing Jobs - Building c7b1280

Status Name Step Failures Logs Raw logs
Native Tests - Data2 Build Failures Logs Raw logs

Failures

⚙️ Native Tests - Data2 #

- Failing: integration-tests/jpa-oracle 

📦 integration-tests/jpa-oracle

Failed to execute goal io.fabric8:docker-maven-plugin:0.39.1:start (docker-start) on project quarkus-integration-test-jpa-oracle: I/O Error

@sberyozkin
Copy link
Member Author

I think it is good to go

@sberyozkin sberyozkin merged commit 45154d6 into quarkusio:main Apr 27, 2022
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Apr 27, 2022
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Apr 27, 2022
@sberyozkin sberyozkin deleted the security_policy_path_with_http_root branch April 27, 2022 07:05
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