Skip to content

fix(q): reject non-response frames - #5

Open
belowzeroff wants to merge 1 commit into
RayforceDB:masterfrom
belowzeroff:fix/q-response-message-type
Open

fix(q): reject non-response frames#5
belowzeroff wants to merge 1 commit into
RayforceDB:masterfrom
belowzeroff:fix/q-response-message-type

Conversation

@belowzeroff

Copy link
Copy Markdown
Contributor

Summary:

  • require q_exchange to receive Q response frames (msgtype 2)
  • reject async/sync frame headers returned on the client response path
  • add a socketpair regression self-test that exercises the bad msgtype path

Tests:

  • make lint
  • bash -n test/run.sh
  • git diff --check
  • make test Q_BINARY=/nonexistent

Note: real-q interop remains skipped locally because the available q install cannot connect to its license daemon.

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