concourse / concourse/concourse-chart
The values.yaml file enables local auth by default and creates a default user.
Open
Nobody has claimed this yet.
bug
- Dominant language
- Mustache
- Stars
- 160
- Forks
- 189
- PR merge metrics
- No merged PRs in 30d
Description
Describe the bug
The values.yaml file enables local auth by default and creates a default user.:
- https://github.com/concourse/concourse-chart/blob/master/values.yaml#L1093
- https://github.com/concourse/concourse-chart/blob/master/values.yaml#L2855
- https://github.com/concourse/concourse-chart/blob/master/values.yaml#L276
This seems dangerous as a default value. It would be better to include these defaults in the comments and disable local auth by default.
Reproduction steps
...
Expected behavior
No default.
Additional context
No default.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with values.yaml at the three linked locations and inspect how the local-auth and default-user settings are represented. Update the defaults so local auth and the default user are not enabled, while documenting the former defaults in comments; done means the chart no longer creates these defaults when installed without overrides.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- helm
- Domain
- devops, security
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 52/100