Skip to content

fix: Internal error on warning injection - #3177

Open
godexsoft wants to merge 2 commits into
XRPLF:developfrom
godexsoft:bugfix/warning-injection-plain-response
Open

fix: Internal error on warning injection#3177
godexsoft wants to merge 2 commits into
XRPLF:developfrom
godexsoft:bugfix/warning-injection-plain-response

Conversation

@godexsoft

Copy link
Copy Markdown
Collaborator

This PR fixes a bug which produced internal error output instead of the correct plain-text error output in cases where it overlapped with maxing out the DOSGuard session allowance.

@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/web/ng/RPCServerHandler.hpp 50.00% 1 Missing and 1 partial ⚠️
src/web/impl/HttpBase.hpp 50.00% 0 Missing and 1 partial ⚠️
src/web/impl/WsBase.hpp 50.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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