Set MULTIPART_FILE_LIMIT_EXCEEDED variable

[hnakamur]

what

• Set the MULTIPART_FILE_LIMIT_EXCEEDED variable to the appropriate value.

why

• The MULTIPART_FILE_LIMIT_EXCEEDED variable was never assigned a value.

references

• None

[airween]

Hi @hnakamur,

could you pick these changes or update your branch?

Thanks!

[Copilot]

Pull request overview

This PR fixes the previously-unset MULTIPART_FILE_LIMIT_EXCEEDED transaction variable by assigning it during multipart request-body completion, and updates the multipart regression tests to assert the variable is populated (including the “no limit exceeded” cases).

Changes:

• Set m_variableMultipartFileLimitExceeded from m_flag_file_limit_exceeded in Multipart::multipart_complete().
• Update multipart strict-validation regression expectations so FL expands to 0 (instead of empty).
• Add a regression assertion that MULTIPART_FILE_LIMIT_EXCEEDED expands to 1 when SecUploadFileLimit is exceeded.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
test/test-cases/regression/request-body-parser-multipart.json	Adjusts expected log output to include FL 0 and adds a case asserting FL 1 when the upload file limit is exceeded.
src/request_body_processor/multipart.cc	Assigns MULTIPART_FILE_LIMIT_EXCEEDED at multipart completion based on the internal file-limit flag.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

[hnakamur]

Hi @airween,

I cherrypicked the merge commit of #3577 with the following command:

git cherry-pick -m 1 f5a6fcd37aea3a7618408b7d1f03b94de9164637