Skip to content

Conversation

@kuba-orlik
Copy link

@kuba-orlik kuba-orlik commented Oct 20, 2025

Authentik uses trailing slashes in URLs

Summary by CodeRabbit

  • Documentation
    • Updated OAuth environment variable configuration URLs in the installation guide to include trailing slashes for authentication endpoint, token endpoint, and user information endpoint URLs.

Authentik uses trailing slashes in URLs
@coderabbitai
Copy link

coderabbitai bot commented Oct 20, 2025

Walkthrough

Three 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

Cohort / File(s) Summary
OAuth URL Configuration Updates
pages/installation/docker-compose.mdx
POSTIZ_OAUTH_AUTH_URL, POSTIZ_OAUTH_TOKEN_URL, and POSTIZ_OAUTH_USERINFO_URL values updated to include trailing slashes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three URLs now stand with trailing grace,
Each slash a small but perfect place,
In Docker's dance, they'll find their way,
OAuth flows brighter every day! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Add trailing slashes to Authentik URLs in docker-compose docs" directly and accurately describes the main change in the changeset. The modifications update three OAuth-related environment variables (POSTIZ_OAUTH_AUTH_URL, POSTIZ_OAUTH_TOKEN_URL, and POSTIZ_OAUTH_USERINFO_URL) in the docker-compose documentation to append trailing slashes, which aligns perfectly with the title. The title is concise, specific, and clearly conveys the purpose of the change without unnecessary noise or vague language, making it easy for a teammate to understand the changeset when scanning history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a 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.com while the base URL (line 145) and the other two OAuth endpoints (lines 146-147) use auth.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.com for 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/"
📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 55797b5 and d2b00b6.

📒 Files selected for processing (1)
  • pages/installation/docker-compose.mdx (1 hunks)

@vercel
Copy link

vercel bot commented Oct 20, 2025

@kuba-orlik is attempting to deploy a commit to the Listinai Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant