Baseflow / Baseflow/cg-dmf-poc
Helm chart uses deprecated nginx as ingress
Nobody has claimed this yet.
- Dominant language
- Kotlin
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
The helm chart included still uses nginx by default. As nginx is deprecated, this default should be changed to something that is still supported, like treafik or other gateway api ingress.
Note that specific header sizes are set for nginx, as well as max request size and duration. This is because of the large requests of POSTs for enkelvoudiginformatieobject. Additionally some people at Gemeente Utrecht run with large cookie headers (8K+), which caused problems with logging in on the admin portal.
If this is modified, it is important to notify infra at GU about the change as it is a significant helm chart change.
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
Inspect the included Helm chart and identify the nginx-specific header sizes, maximum request size, and request duration settings. Compare a supported ingress or Gateway API option, preserve the requirements for large POSTs and cookie headers, and notify infra at Gemeente Utrecht when the default changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- helm, kubernetes, nginx
- Domain
- devops, infrastructure
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100