SLING-13138 Add configurable fail-fast parameter limit handling

[aswinad]

SLING-13138
Improve request parameter limit handling in ParameterMap.

This change updates the parameter limit check from size() == maxParameters
to size() >= maxParameters for more defensive boundary handling.

It also introduces an optional configuration to control behavior when the
configured parameter limit is exceeded:

• default: log a warning and ignore additional parameters
• fail-fast mode: throw IllegalStateException immediately

The default behavior remains unchanged for backward compatibility.

Unit tests are included for both modes.

[joerghoh]

LGTM

although I have never seen that warning being written (highly doubt that as the default size is 10k), so I question a bit the value of it. But fixing TODOs is definitely good.

[aswinad]

Thanks! The main goal here was addressing the TODO and making the behavior configurable.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
80.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[aswinad]

Hi @joerghoh — just following up on this PR.

Please let me know if any changes are needed from my side. If not, I’d appreciate your help merging it since I don’t have merge permissions.

Thanks!
Aswin

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
80.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud