Skip to content

Conversation

@helionmusic
Copy link

@helionmusic helionmusic commented Feb 3, 2025

What this PR does / why we need it

Currently in the Linkwarden chart, when attempting to set an issuer url value for a given provider, the template attempts to validate the the issuer value exists in the list of supported auth providers. Instead it should check for provider against that list.

What ends up happening is since the issuer should be a URL, it will never resolve against that list, and thus the *_ISSUER key never gets set.

Checklist

  • Chart Version bumped
  • User name added to AUTHORS file
  • Title of the PR starts with a valid commit scope as detailed in the CONTRIBUTING (e.g.
    fix(charts/linkwarden): ...)

@github-actions
Copy link

github-actions bot commented Mar 5, 2025

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Mar 5, 2025
@bkoerbel
Copy link

This exact issue has blocked me from using this Linkwarden chart with Authentik. Please merge this fix :)

@github-actions github-actions bot removed the stale label Mar 18, 2025
@github-actions
Copy link

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants