Correctly setup default unlimited maxFormKeys as default

olamy commented 3 months ago

Signed-off-by: Olivier Lamy olamy@apache.org

Testing done

Submitter checklist

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

olamy commented 3 months ago

btw those unlimited values could be dangerous (ddos attack) but I have no real idea about what sort of possible values could work for Jenkins (1000 is already a lot)

olamy commented 3 months ago

from https://github.com/jenkinsci/winstone/pull/393#issuecomment-2190217013

looks to be fixed as well with the next PR #395

jenkinsci / winstone

Correctly setup default unlimited maxFormKeys as default #394

Testing done

Submitter checklist