Skip to content

Document forward auth setup for public profiles - #315

Merged
Git-Romer merged 1 commit into
mainfrom
docs/authentik-public-routes
Jul 27, 2026
Merged

Document forward auth setup for public profiles#315
Git-Romer merged 1 commit into
mainfrom
docs/authentik-public-routes

Conversation

@Git-Romer

Copy link
Copy Markdown
Owner

Summary

  • Document the full reverse-proxy allowlist needed by public trainer profiles and binders.
  • Add Authentik examples for single-application and domain-level forward auth.
  • Include card images, trainer sprites, compiled frontend bundles, fallback artwork, and browser assets.
  • Warn against bypassing authentication for all /api routes.
  • Explain that native OIDC remains a separate authentication integration.

Why

Public profile routes are anonymous inside PokéCollector, but an external forward-auth proxy sees requests first. Without explicit exceptions, /u remains inaccessible or loads without its API data and assets.

This follows up on the deployment concern raised by @engelben in #308.

Validation

  • git diff --check
  • node scripts/check-version.mjs
  • Positive and negative checks for the documented path and full-URL regexes, including query strings
  • Fresh reviewer pass with no blocking findings

Copilot AI review requested due to automatic review settings July 27, 2026 21:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Git-Romer Git-Romer mentioned this pull request Jul 27, 2026
12 tasks
@Git-Romer
Git-Romer merged commit 5be7d73 into main Jul 27, 2026
6 checks passed
@Git-Romer
Git-Romer deleted the docs/authentik-public-routes branch July 27, 2026 21:11
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.

2 participants