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

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.

@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants