Skip to content

Arcpoc 1569 - BE : Bulk upload validation error message is truncated when response exceeds 4000 characters#669

Merged
mustafaahmeddev merged 12 commits into
masterfrom
ARCPOC-1569
Jul 22, 2026
Merged

Arcpoc 1569 - BE : Bulk upload validation error message is truncated when response exceeds 4000 characters#669
mustafaahmeddev merged 12 commits into
masterfrom
ARCPOC-1569

Conversation

@philhead1

Copy link
Copy Markdown
Contributor

Jira link

ARCPOC-1569

Change description

  • removed truncation of failure message

Testing done

  • uploaded CSV file via postman generating an error message overflowing the old limitation (4000 characters)
  • Unit and integration tests.

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@mustafaahmeddev mustafaahmeddev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, we just need to harden one of the tests and update a comment.

@hmcts-jenkins-cnp
hmcts-jenkins-cnp Bot requested a deployment to preview July 21, 2026 15:30 Abandoned

@mustafaahmeddev mustafaahmeddev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thanks for addressing those issues.

@mustafaahmeddev
mustafaahmeddev merged commit 3df0822 into master Jul 22, 2026
7 checks passed
@mustafaahmeddev
mustafaahmeddev deleted the ARCPOC-1569 branch July 22, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants