Skip to content

Fix swallowed exception in GetJson - #915

Open
Obito-001 wants to merge 1 commit into
OWASP:devfrom
Obito-001:dev#510
Open

Fix swallowed exception in GetJson#915
Obito-001 wants to merge 1 commit into
OWASP:devfrom
Obito-001:dev#510

Conversation

@Obito-001

Copy link
Copy Markdown

Fixes issue #510 by preventing unrecoverable exceptions in GetJson from being silently ignored.

Changes:

  • Replace the swallowed exception with a RuntimeException.
  • Preserve the original exception as the cause.
  • Add a regression test for BufferedReader failures.

Testing:

  • GetJsonTest: 7 tests passed
  • Spotless formatting check: passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant