Skip to content

Conversation

@pwgit-create
Copy link
Contributor

PR

Size Limit

✅ Size Limit Issue

  • Fixed and improved handling of size limit constraints.

  • Ensures better stability and consistency across related operations.


Dependency Updates

Dependency
PostgreSQL Driver 42.7.7 -> 42.7.8
swagger-annotations 2.2.36 -> 2.2.40
maven-compiler-plugin 3.14.0 -> 3.14.1

These updates include minor bug fixes and performance improvements.

⚠️ Deprecated Status Code

The HTTP 422 – Unprocessable Entity status code has been deprecated.

All usages have been replaced with HTTP 400 – Bad Request for consistency and better standard alignment.

Notes

Be sure to verify API consumers handle the updated response codes correctly.

New Minimum size limit is 100mb (info for @lilstiffy)
@pwgit-create pwgit-create linked an issue Nov 1, 2025 that may be closed by this pull request
@pwgit-create
Copy link
Contributor Author

depreceated_status_code

@Operation(summary = "Get Scan Summaries for Scan", description = "Retrieves the scan summaries for a specific scan.")
@ApiResponses(value = {
@ApiResponse(responseCode = "200", description = "Successfully retrieved scan summaries"),
@ApiResponse(responseCode = "400", description = "Bad request"),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small reminder to myself to handle 400's now in my switch statements for in the request methods 💯 📓

* instead of being fully loaded into memory at once.
*/
private final int MAX_SIZE_OF_BYTE_ARRAY = 2147483645;
private final int MEMORY_STRATEGY_LIMIT = 100000000;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🦤 💯
Great comment also 🥇

Copy link
Collaborator

@lilstiffy lilstiffy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great looking PR comrade, and lovely work 🥇
PS. thanks for a really great and exhausting boxing match 😓 🥊

@pwgit-create pwgit-create merged commit dd8dbb8 into develop Nov 2, 2025
1 of 3 checks passed
@pwgit-create pwgit-create deleted the 289-improve-file-size-check-in-fis-be branch November 2, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve File Size Check in FIS BE

3 participants