Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add additonal remote auth settings #62

Closed
cstisa opened this issue Nov 11, 2021 · 1 comment · Fixed by #64
Closed

Add additonal remote auth settings #62

cstisa opened this issue Nov 11, 2021 · 1 comment · Fixed by #64
Assignees
@bootc
Labels
enhancement New feature or request
Milestone
4.0

Comments

@cstisa
Copy link
Contributor

cstisa commented Nov 11, 2021

Currently the remoteAuth setting contains only the following values :

remoteAuth:
  enabled: false
  backend: netbox.authentication.RemoteUserBackend
  header: HTTP_REMOTE_USER
  autoCreateUser: true
  defaultGroups: []
  defaultPermissions: {}

It would be nice to add the rest of settings for remote auth supported by NetBox, like REMOTE_AUTH_GROUP_HEADER or REMOTE_AUTH_GROUP_SYNC_ENABLED for example.
@cstisa cstisa mentioned this issue Nov 15, 2021
Merged
@bootc bootc self-assigned this Nov 27, 2021
@bootc bootc added enhancement New feature or request pending Issue is in a branch waiting for a release labels Nov 27, 2021
@bootc bootc added this to the 4.0 milestone Nov 27, 2021
@bootc
Copy link
Member

bootc commented Nov 29, 2021

I've just released version 4.0.1 of this chart which resolves this issue.

@bootc bootc closed this as completed Nov 29, 2021
@bootc bootc removed the pending Issue is in a branch waiting for a release label Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bootc bootc

Labels
enhancement New feature or request
Projects
None yet
Milestone
4.0
Development

Successfully merging a pull request may close this issue.

Add additonal remote auth settings
2 participants
@bootc @cstisa