Skip to content

Enable/Disable Items with authentification headers#1454

Merged
KodeStar merged 14 commits into
linuxserver:2.xfrom
knom:2.x
Jul 10, 2025
Merged

Enable/Disable Items with authentification headers#1454
KodeStar merged 14 commits into
linuxserver:2.xfrom
knom:2.x

Conversation

@knom
Copy link
Copy Markdown
Contributor

@knom knom commented May 5, 2025

Hey there,
I'm trying to reinvigorate #1304 by doing the the changes requested @KodeStar and @albinmedoc.

Use case:

  • the Heimdall server is located behind a login page provides by a SSO service like authelia or Keycloak
  • You want to define common services only once and share them between users enabled by the roles they're assigned to

New feature when enabled:
*The Heimdall settings (order of items, application list, tag list, settings) are only visible for users with role admin.

  • For every application and tag the new field Authentification role is added. This filed contains the role to witch the application or tag is displayed.
  • Applications and tags are only displayed when the logged in user is member of the configured role.

Configuration:

  • The feature can be enabled/disabled in the env file with AUTH_ROLES_ENABLE.
  • It is possible to configure the header containg the roles by AUTH_ROLES_HEADER and AUTH_ROLES_HTTP_HEADER.
  • The role to enable the Heimdall settings can be set wit AUTH_ROLES_ADMIN.

@KodeStar:
Please let me know if anything else is required - as I really would like to ship this (for my own sake - to use with my AUTHENTIK instance).

Thanks, knom.

@LinuxServer-CI
Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@knom
Copy link
Copy Markdown
Contributor Author

knom commented Jun 9, 2025

@KodeStar @albertsj1 @tkolzer?

Anybody willing to do the pr review/lgtp approval?

@LinuxServer-CI
Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link
Copy Markdown
Contributor

@tkolzer tkolzer left a comment

Choose a reason for hiding this comment

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

Everything looks good to me.

@KodeStar KodeStar merged commit 749442e into linuxserver:2.x Jul 10, 2025
5 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs to Done in Issue & PR Tracker Jul 10, 2025
@LinuxServer-CI
Copy link
Copy Markdown

This pull request is locked due to inactivity

@linuxserver linuxserver locked as resolved and limited conversation to collaborators Aug 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants