Skip to content

website/integrations: fix komodo provider url#14912

Merged
dewi-tik merged 2 commits intogoauthentik:mainfrom
dylangottlieb:website/integrations--fix-komodo-oidc-provider-uri
Jun 5, 2025
Merged

website/integrations: fix komodo provider url#14912
dewi-tik merged 2 commits intogoauthentik:mainfrom
dylangottlieb:website/integrations--fix-komodo-oidc-provider-uri

Conversation

@dylangottlieb
Copy link
Contributor

@dylangottlieb dylangottlieb commented Jun 5, 2025 •
edited by dewi-tik
Loading

Details

This PR changes the Komodo integration instructions page on the website. The KOMODO_OIDC_PROVIDER value is incorrect. Following the current instructions, Komodo will crash with the following error:

thread 'tokio-runtime-worker' panicked at bin/core/src/auth/oidc/client.rs:54:6:
called `Result::unwrap()` on an `Err` value: Failed to initialize OIDC client.

Caused by:
    0: Failed to get OIDC /.well-known/openid-configuration
    1: Server returned invalid response: HTTP status code 404 Not Found at https://authentik.company/application/o/authorize/.well-known/openid-configuration
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Error: task 9 panicked with message "called `Result::unwrap()` on an `Err` value: Failed to initialize OIDC client.\n\nCaused by:\n    0: Failed to get OIDC /.well-known/openid-configuration\n    1: Server returned invalid response: HTTP status code 404 Not Found at https://authentik.company/application/o/authorize/.well-known/openid-configuration"

The correct KOMODO_OIDC_PROVIDER value is https://authentik.company/application/o/application-slug/

This PR updates the website with the new value.

Closes #14913

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 website)

@dylangottlieb dylangottlieb requested a review from a team as a code owner June 5, 2025 02:25
@netlify
Copy link

netlify bot commented Jun 5, 2025 •
edited
Loading

✅ Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 66ed357
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/684101d826b09c00083a2aaf

@netlify
Copy link

netlify bot commented Jun 5, 2025 •
edited
Loading

✅ Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 66ed357
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/684101d8f3ce6a000869d45b
😎 Deploy Preview https://deploy-preview-14912--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.

@dylangottlieb dylangottlieb mentioned this pull request Jun 5, 2025
Closed
dominic-r
dominic-r reviewed Jun 5, 2025
View reviewed changes
dominic-r
dominic-r requested changes Jun 5, 2025
View reviewed changes
Copy link
Member

@dominic-r dominic-r left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks for your PR! We actually prefer this format for the slug as it is a variable ^^

@dylangottlieb @dominic-r
Update website/integrations/services/komodo/index.mdx
66ed357 
Co-authored-by: Dominic R <dominic@sdko.org>
Signed-off-by: Dylan Gottlieb <dylangottlieb@users.noreply.github.com>
dewi-tik
dewi-tik approved these changes Jun 5, 2025
View reviewed changes
Copy link
Contributor

@dewi-tik dewi-tik left a comment

Choose a reason for hiding this comment

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

You're completely right, thanks for catching this. I must have pasted the incorrect value while writing the doc.

@dewi-tik dewi-tik changed the title fix KOMODO_OIDC_PROVIDER uri docs website/integrations: fix komodo provider url Jun 5, 2025
@dewi-tik dewi-tik enabled auto-merge (squash) June 5, 2025 09:03
@codecov
Copy link

codecov bot commented Jun 5, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.76%. Comparing base (52ca70d) to head (66ed357).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main   #14912   +/-   ##
=======================================
  Coverage   92.76%   92.76%           
=======================================
  Files         817      817           
  Lines       42197    42197           
=======================================
  Hits        39146    39146           
  Misses       3051     3051
Flag Coverage Δ
e2e 47.60% <ø> (ø)
integration 24.38% <ø> (ø)
unit 90.65% <ø> (ø)

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.

@dewi-tik dewi-tik merged commit e29961b into goauthentik:main Jun 5, 2025
97 checks passed
kensternberg-authentik added a commit that referenced this pull request Jun 6, 2025
@kensternberg-authentik
Merge branch 'main' into dev
0457b59 
* main: (54 commits)
  web/user: fix user settings flow not loading (#14911)
  website/docs: fix outdated and incorrect example kubernetes deployment (#14928)
  docusaurus-config: Update deps, colors. (#14796)
  admin: only run update checks in the default tenant (#14874)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#14923)
  core: bump astral-sh/uv from 0.7.10 to 0.7.11 (#14918)
  providers/proxy: set_oauth_defaults in reconcile instead of task (#14875)
  *: use ManagedAppConfig everywhere (#14839)
  tenants: fix tenant aware celery scheduler (#14921)
  core, web: update translations (#14910)
  core: bump goauthentik.io/api/v3 from 3.2025041.4 to 3.2025060.1 (#14919)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in zh_CN (#14915)
  translate: Updates for file web/xliff/en.xlf in zh_CN (#14916)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in zh-Hans (#14914)
  website/integrations: improve komodo config verification (#14849)
  website/integrations: fix komodo provider url (#14912)
  website/docs: fix note at end of rac credentials prompt (#14909)
  website/docs: add credentials prompt for rac doc (#14840)
  core: bump redis from 6.0.0 to v6.2.0 (#14895)
  core: bump protobuf from 6.30.2 to v6.31.1 (#14894)
  ...
kensternberg-authentik added a commit that referenced this pull request Jun 6, 2025
@kensternberg-authentik
Merge branch 'main' into web/standards/fix-boolean-attribute-abuse
7d983e5 
* main: (202 commits)
  web/user: fix user settings flow not loading (#14911)
  website/docs: fix outdated and incorrect example kubernetes deployment (#14928)
  docusaurus-config: Update deps, colors. (#14796)
  admin: only run update checks in the default tenant (#14874)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in fr (#14923)
  core: bump astral-sh/uv from 0.7.10 to 0.7.11 (#14918)
  providers/proxy: set_oauth_defaults in reconcile instead of task (#14875)
  *: use ManagedAppConfig everywhere (#14839)
  tenants: fix tenant aware celery scheduler (#14921)
  core, web: update translations (#14910)
  core: bump goauthentik.io/api/v3 from 3.2025041.4 to 3.2025060.1 (#14919)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in zh_CN (#14915)
  translate: Updates for file web/xliff/en.xlf in zh_CN (#14916)
  translate: Updates for file locale/en/LC_MESSAGES/django.po in zh-Hans (#14914)
  website/integrations: improve komodo config verification (#14849)
  website/integrations: fix komodo provider url (#14912)
  website/docs: fix note at end of rac credentials prompt (#14909)
  website/docs: add credentials prompt for rac doc (#14840)
  core: bump redis from 6.0.0 to v6.2.0 (#14895)
  core: bump protobuf from 6.30.2 to v6.31.1 (#14894)
  ...
cavefire pushed a commit to cavefire/authentik that referenced this pull request Jun 24, 2025
@dylangottlieb @dominic-r @cavefire
website/integrations: fix komodo provider url (goauthentik#14912)
68b07f3 
* fix KOMODO_OIDC_PROVIDER uri docs

* Update website/integrations/services/komodo/index.mdx

Co-authored-by: Dominic R <dominic@sdko.org>
Signed-off-by: Dylan Gottlieb <dylangottlieb@users.noreply.github.com>

---------

Signed-off-by: Dylan Gottlieb <dylangottlieb@users.noreply.github.com>
Co-authored-by: Dominic R <dominic@sdko.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dominic-r dominic-r dominic-r approved these changes

@dewi-tik dewi-tik dewi-tik approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Incorrect value for Komodo KOMODO_OIDC_PROVIDER environment variable

3 participants

@dylangottlieb @dominic-r @dewi-tik