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

Security doc fix: Fix links in Protect a web application topic #37088

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

rolfedh
Copy link
Contributor

@rolfedh rolfedh commented Nov 14, 2023

This PR fixes two broken links that showed up when I was verifying security-oidc-code-flow-authentication-tutorial.adoc ("Protect a web application by using OpenID Connect (OIDC) authorization code flow") for product docs (RHBQ 3.2.8).
The first link depended on a redirect and didn't work in the product docs because PV2 doesn't support redirects.
The second link referenced an obsolete filename, and was broken both upstream and downstream.

@sberyozkin, please approve these changes for main only.
Because these changes might not backport cleanly to 3.2, I have manually cherry-picked these changes to 3.2 and created a separate MR for them: #37089

References QDOCS-534.

Copy link

github-actions bot commented Nov 14, 2023

🙈 The PR is closed and the preview is expired.

@sberyozkin sberyozkin merged commit bb0352b into quarkusio:main Nov 14, 2023
5 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.6 - main milestone Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docstyle issues related for manual docstyle review area/documentation
Projects
Development

Successfully merging this pull request may close these issues.

2 participants