Skip to content

Lab Report - HTTP 400 fix - #3557

Draft
mpeels wants to merge 3 commits into
mainfrom
mp/NBS-18488/fix-400-on-lab-create
Draft

Lab Report - HTTP 400 fix #3557
mpeels wants to merge 3 commits into
mainfrom
mp/NBS-18488/fix-400-on-lab-create

Conversation

@mpeels

@mpeels mpeels commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Description

During my investigation into the http 400 status errors observed when manually creating a Lab Report, I implemented a quick fix and wanted to push this up as a starting point since it appears to resolve the issue locally. Its possible we might also just be able to use a URLEncoder to encode the URL, but I am concerned that this will lead to a "double encoding" as some characters are already encoded by NBS 6.

Tickets

NBS-Central-18504

Checklist before requesting a review

  • PR focuses on a single story
  • Code has been fully tested to meet acceptance criteria
  • PR is reasonably small and reviewable (Generally less than 10 files and 500 changed lines)
  • All new functions/classes/components reasonably small
  • Functions/classes/components focused on one responsibility
  • Code easy to understand and modify (clarity over concise/clever)
  • PRs containing TypeScript follow the Do's and Don'ts
  • PR does not contain hardcoded values (Uses constants)
  • All code is covered by unit or feature tests

@mpeels mpeels changed the title Mp/nbs 18488/fix 400 on lab create Lab Report - HTTP 400 fix Aug 7, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

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.

1 participant