Skip to content

fix: return ProtocolViolation instead of syntax error in admin DB#900

Merged
levkk merged 2 commits intomainfrom
levkk-return-protocol-violation
Apr 14, 2026
Merged

fix: return ProtocolViolation instead of syntax error in admin DB#900
levkk merged 2 commits intomainfrom
levkk-return-protocol-violation

Conversation

@levkk
Copy link
Copy Markdown
Collaborator

@levkk levkk commented Apr 14, 2026

Fix #539

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

❌ Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pgdog/src/net/messages/error_response.rs 0.00% 11 Missing ⚠️
pgdog/src/admin/server.rs 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@levkk levkk merged commit 04bac3d into main Apr 14, 2026
9 of 10 checks passed
@levkk levkk deleted the levkk-return-protocol-violation branch April 14, 2026 22:33
@blacksmith-sh
Copy link
Copy Markdown
Contributor

blacksmith-sh bot commented Apr 14, 2026

Found 1 test failure on Blacksmith runners:

Failure

Test View Logs
test_float_binary_roundtrip View Logs

Fix in Cursor

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.

Admin database issues with psycopg2 and pgcli: syntax error in admin command

1 participant