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.

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

Add nginx-ldapauth-proxy allowing to set authorization header #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

iblackman
Copy link
Contributor

@iblackman iblackman commented Jan 20, 2023

PR Description

Extracted from https://github.com/helm/charts/tree/master/stable/nginx-ldapauth-proxy and added an option to pass basic auth.

Checklist

  • Have you reviewed and updated the chart default values if necessary?
  • Have you reviewed and updated the chart documentation if necessary?
  • Does your branch follow the naming convention of {chartNameWithDashes}-v{versionString}-{optionalPatchVersion}?
  • Have you bumped the version in the chart's Chart.yaml?

Tagged Releases

Please remember to make a tagged release after merging your PR that:

  • Has a tag name that matches your PR branch name (see above)
  • Has a description that summarizes the changes made

This makes it possible to use previous versions of the charts maintained here as new releases are published. Please see the release history of this repository for examples.

@iblackman iblackman requested a review from a team as a code owner January 20, 2023 15:42
@iblackman iblackman enabled auto-merge (squash) March 31, 2023 14:14
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.

1 participant