-
Notifications
You must be signed in to change notification settings - Fork 56
Add trailing slashes to Authentik URLs in docker-compose docs #200
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
Conversation
There is a typo in redirection from Linkedin if it's "pages" instead of "page"
Update r2.mdx
Update Cloudflare R2 Bucket Documentation
Update linkedin.mdx
Bumps [cross-spawn](https://github.com/moxystudio/node-cross-spawn) to 7.0.6 and updates ancestor dependencies [cross-spawn](https://github.com/moxystudio/node-cross-spawn), [nextra](https://github.com/shuding/nextra) and [nextra-theme-docs](https://github.com/shuding/nextra). These dependencies need to be updated together. Updates `cross-spawn` from 7.0.3 to 7.0.6 - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6) Updates `nextra` from 2.13.4 to 3.2.4 - [Release notes](https://github.com/shuding/nextra/releases) - [Commits](https://github.com/shuding/nextra/compare/[email protected]@3.2.4) Updates `nextra-theme-docs` from 2.13.4 to 3.2.4 - [Release notes](https://github.com/shuding/nextra/releases) - [Commits](https://github.com/shuding/nextra/compare/[email protected]@3.2.4) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: indirect - dependency-name: nextra dependency-type: direct:production - dependency-name: nextra-theme-docs dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
…n/multi-1f810f95bc Bump cross-spawn, nextra and nextra-theme-docs
bugfix: Typo in slack page
bugfix: v2 to v3 migration
Move OAuth2 redirect intructions to generic snippet
Clarify installation method and update recommendations.
Added required settings and various API configurations for Postiz.
Added steps to get project version from Git tag and perform SonarQube analysis.
Added a status page link to the introduction.
Authentik uses trailing slashes in URLs
WalkthroughThree OAuth-related environment variables in the Docker Compose installation documentation are updated to append trailing slashes to their URL values: POSTIZ_OAUTH_AUTH_URL, POSTIZ_OAUTH_TOKEN_URL, and POSTIZ_OAUTH_USERINFO_URL. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
pages/installation/docker-compose.mdx (1)
146-148: Domain inconsistency in POSTIZ_OAUTH_USERINFO_URL (line 148).The trailing slashes are correctly added to all three OAuth URLs per Authentik's requirements. However, line 148 uses
authentik.example.comwhile the base URL (line 145) and the other two OAuth endpoints (lines 146-147) useauth.example.com. This inconsistency could confuse users following the documentation.Please clarify whether this difference is intentional or if line 148 should also use
auth.example.comfor consistency.If unintentional, apply this diff:
- POSTIZ_OAUTH_USERINFO_URL: "https://authentik.example.com/application/o/userinfo/" + POSTIZ_OAUTH_USERINFO_URL: "https://auth.example.com/application/o/userinfo/"
|
@kuba-orlik is attempting to deploy a commit to the Listinai Team on Vercel. A member of the Team first needs to authorize it. |
Authentik uses trailing slashes in URLs
Summary by CodeRabbit