jenkinsci / winstone

Patched winstone used in Jenkins
https://jenkins.io
Other
88 stars 72 forks source link

[JENKINS-73285] Set maximum parameter count to 10,000 to match the documentation #395

Closed basil closed 3 months ago

basil commented 3 months ago

The documentation states that the Winstone default is 10,000 (unlike the Jetty default of 1,000), but this statement is false, as Winstone only sets the value if the user specifies the option, and otherwise uses the Jetty default. Since Stapler forms are complex and often need thousands of parameters, this PR adjusts Winstone to match its own documentation and use a limit of 10,000.

Testing done

Fixes jetty/jetty.project#11959