oapi-codegen icon indicating copy to clipboard operation
oapi-codegen copied to clipboard

Fix server variables conflict

Open acm19 opened this issue 7 months ago • 6 comments

Adds a property to opt-in avoiding the default value colliding with the value default for enum server variables. When set to true Enum is added to the variable name so that it doesn't clash with the default variable name. This behaviour is opt-in to avoid backward compatibility, but the default is actually a bug, so it make sense to change that default in the next major release.

Resolves: https://github.com/oapi-codegen/oapi-codegen/issues/2003

acm19 avatar Jul 25 '25 11:07 acm19

Kusari Analysis Results

Analysis for commit: 961dc9c243f145536eab658e51c75719924b7b4d, performed at: 2025-08-12T09:17:46Z

@kusari-inspector rerun - Trigger a re-analysis of this PR

@kusari-inspector feedback [your message] - Send feedback to our AI and team


Recommendation

✅ PROCEED with this Pull Request

Summary

No Flagged Issues Detected

All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Found this helpful? Give it a 👍 or 👎 reaction!

kusari-inspector[bot] avatar Jul 25 '25 11:07 kusari-inspector[bot]

Nice, thank you!

jamietanna avatar Jul 25 '25 14:07 jamietanna

Kusari PR Analysis rerun based on - 4b52cc662883d4d489c3b8f16f62a88ad83d631c performed at: 2025-07-25T18:19:32Z - link to updated analysis

kusari-inspector[bot] avatar Jul 25 '25 18:07 kusari-inspector[bot]

Kusari PR Analysis rerun based on - cd507afb89db08bf43b62974973b261690e974a5 performed at: 2025-07-25T18:20:01Z - link to updated analysis

kusari-inspector[bot] avatar Jul 25 '25 18:07 kusari-inspector[bot]

Hey @jamietanna, bumping this in your inbox so it doesn't get forgotten. I've replied to all your comments so far.

acm19 avatar Aug 12 '25 09:08 acm19

Kusari PR Analysis rerun based on - 961dc9c243f145536eab658e51c75719924b7b4d performed at: 2025-08-12T09:17:25Z - link to updated analysis

kusari-inspector[bot] avatar Aug 12 '25 09:08 kusari-inspector[bot]