Skip to content

website/integrations: adds missing trailing slash in homarr doc#14249

Merged
tanberry merged 1 commit intomainfrom
website/integrations/fix-homarr-doc
Apr 25, 2025
Merged

website/integrations: adds missing trailing slash in homarr doc#14249
tanberry merged 1 commit intomainfrom
website/integrations/fix-homarr-doc

Conversation

@dewi-tik
Copy link
Contributor

Details

Closes #14248

Adds a required trailing slash to the AUTH_OIDC_ISSUER URL


Checklist

If applicable

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

@dewi-tik dewi-tik requested a review from a team as a code owner April 25, 2025 15:44
@netlify
Copy link

netlify bot commented Apr 25, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 755cb97
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/680badfcb06a8f0008dc5a23

@netlify
Copy link

netlify bot commented Apr 25, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 755cb97
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/680badfcc885c100083481be
😎 Deploy Preview https://deploy-preview-14249--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 site configuration.

Copy link
Member

@dominic-r dominic-r left a comment

Choose a reason for hiding this comment

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

After resolution of review suggestion, LGTM thanks Dewi

AUTH_OIDC_CLIENT_ID=<Client ID from authentik>
AUTH_OIDC_CLIENT_SECRET=<Client secret from authentik>
AUTH_OIDC_ISSUER=https://authentik.company/application/o/<slug from authentik>
AUTH_OIDC_ISSUER=https://authentik.company/application/o/<slug from authentik>/ #Trailing slash is required
Copy link
Member

@dominic-r dominic-r Apr 25, 2025

Choose a reason for hiding this comment

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

Suggested change
AUTH_OIDC_ISSUER=https://authentik.company/application/o/<slug from authentik>/ #Trailing slash is required
AUTH_OIDC_ISSUER=https://authentik.company/application/o/<slug from authentik>/

A bit like the apache guacamole doc, we ended up removing the notice. Let's do the same here

Copy link
Member

Choose a reason for hiding this comment

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

p.s. this gives me the idea for another task: standardize the slug angle bracket variables

@codecov
Copy link

codecov bot commented Apr 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.76%. Comparing base (07ec787) to head (755cb97).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #14249      +/-   ##
==========================================
+ Coverage   92.70%   92.76%   +0.05%     
==========================================
  Files         809      809              
  Lines       41533    41533              
==========================================
+ Hits        38504    38527      +23     
+ Misses       3029     3006      -23     
Flag Coverage Δ
e2e 47.85% <ø> (+0.08%) ⬆️
integration 24.39% <ø> (ø)
unit 90.57% <ø> (ø)

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.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Contributor

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-755cb97a10f290d4921a1cc8463a01cc980e25f7
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-755cb97a10f290d4921a1cc8463a01cc980e25f7

Afterwards, run the upgrade commands from the latest release notes.

Copy link
Contributor

@tanberry tanberry left a comment

Choose a reason for hiding this comment

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

Thannks @dewi-tik !

@tanberry tanberry merged commit 57d8375 into main Apr 25, 2025
92 checks passed
@tanberry tanberry deleted the website/integrations/fix-homarr-doc branch April 25, 2025 17:38
kensternberg-authentik added a commit that referenced this pull request Apr 25, 2025
* main: (33 commits)
  website/integrations: adds missing trailing slash in homarr doc (#14249)
  lifecycle: fix test-all in docker (#14244)
  core, web: update translations (#14243)
  web/admin: prevent default logo flashing in admin interface (#13960)
  website/docs: Update release notes for 2025.4 (#14158)
  core, web: update translations (#14241)
  Updates for file web/xliff/en.xlf in zh_TW [Manual Sync] (#14225)
  translate: Updates for file web/xliff/en.xlf in nl [Manual Sync] (#14217)
  translate: Updates for file web/xliff/en.xlf in fi [Manual Sync] (#14219)
  translate: Updates for file web/xliff/en.xlf in de [Manual Sync] (#14220)
  translate: Updates for file web/xliff/en.xlf in fr [Manual Sync] (#14221)
  translate: Updates for file web/xliff/en.xlf in pl [Manual Sync] (#14222)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in es [Manual Sync] (#14223)
  translate: Updates for file web/xliff/en.xlf in zh-Hans [Manual Sync] (#14224)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in de [Manual Sync] (#14226)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fi [Manual Sync] (#14227)
  translate: Updates for file web/xliff/en.xlf in tr [Manual Sync] (#14228)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in pl [Manual Sync] (#14229)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in ko [Manual Sync] (#14230)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in it [Manual Sync] (#14231)
  ...
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.

Homarr integration documentation incorrect

3 participants