Skip to content

website/docs: update url to docker-compose.yml#16901

Merged
tanberry merged 1 commit into
goauthentik:mainfrom
nalves599:fix-docs-docker-compose-url
Sep 23, 2025
Merged

website/docs: update url to docker-compose.yml#16901
tanberry merged 1 commit into
goauthentik:mainfrom
nalves599:fix-docs-docker-compose-url

Conversation

@nalves599

Copy link
Copy Markdown
Contributor

Details

Update URL of docker-compose.yml from "https://goauthentik.io/docker-compose.yml" to "https://docs.goauthentik.io/docker-compose.yml"


Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make docs)

@nalves599
nalves599 requested a review from a team as a code owner September 21, 2025 14:58
@netlify

netlify Bot commented Sep 21, 2025

Copy link
Copy Markdown

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 9af2568
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/68d0127b87715e000824d6b4
😎 Deploy Preview https://deploy-preview-16901--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Sep 21, 2025

Copy link
Copy Markdown

Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit 9af2568
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/68d0127b56134c000881119a
😎 Deploy Preview https://deploy-preview-16901--authentik-integrations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Sep 21, 2025

Copy link
Copy Markdown

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 9af2568
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/68d0127bb36b2d0008dac5d1

@dominic-r dominic-r left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I vote against this personally. Both URLs are valid and it's merely a redirect. It works fine as is

@nalves599

nalves599 commented Sep 21, 2025

Copy link
Copy Markdown
Contributor Author

I did this change because when you do curl -O https://goauthentik.io/docker-compose.yml it doesn't follow the redirect automatically.

@dominic-r Would you prefer to add the flag -L to curl in order to follow the redirect?

@codecov

codecov Bot commented Sep 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.77%. Comparing base (e823687) to head (9af2568).
⚠️ Report is 54 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #16901      +/-   ##
==========================================
- Coverage   92.83%   92.77%   -0.07%     
==========================================
  Files         838      838              
  Lines       45396    45396              
==========================================
- Hits        42143    42114      -29     
- Misses       3253     3282      +29     
Flag Coverage Δ
e2e 46.59% <ø> (-0.01%) ⬇️
integration 23.44% <ø> (-0.06%) ⬇️
unit 90.92% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tanberry tanberry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks so much for catching and fixing this @nalves599 !

@tanberry

Copy link
Copy Markdown
Contributor

Thanks so much for catching and fixing this @nalves599 !

Ooops sorry @dominic-r I missed your comment. I do think it is fine to have the full proper URL, even if the redirect was working.

@tanberry
tanberry merged commit 1dda417 into goauthentik:main Sep 23, 2025
106 checks passed
@dominic-r

Copy link
Copy Markdown
Member

Yea, we came to that conclusion this morning. Was waiting for you to handle the merge.

@dominic-r dominic-r added backport/version-2025.8 Add this label to PRs to backport changes to version-2025.8 backport/version-2025.6 labels Sep 24, 2025
@authentik-automation

Copy link
Copy Markdown
Contributor

🍒 Cherry-pick to version-2025.6 created: #16985

@authentik-automation

Copy link
Copy Markdown
Contributor

🍒 Cherry-pick to version-2025.8 created: #16986

dominic-r pushed a commit that referenced this pull request Sep 24, 2025
…16986)

website/docs: update url to docker-compose.yml (#16901)

Update URL of docker-compose.yml from "https://goauthentik.io/docker-compose.yml" to "https://docs.goauthentik.io/docker-compose.yml"

Co-authored-by: Nuno Alves <nuno.alves02@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/version-2025.8 Add this label to PRs to backport changes to version-2025.8

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants