Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prelogin and login7 messages are missing some fields, leading to connection failures against certain endpoints #226

Closed
raymondb1 opened this issue Nov 5, 2024 · 0 comments · Fixed by #228

Comments

@raymondb1
Copy link
Member

The connection id / activity id / sequence number should be populated with valid values and semantics similar to what the other drivers (OLEDB, ODBC / SqlClient(s)).

Those are very important for traceability and debugging purposes (they allow correlation between what a client might see and execution server side).

Also the client id and PID of the login 7 message should be populated - the spec defines them as:

  • ClientID: The unique client ID (created by using the NIC address). ClientID is the MAC address of the physical network layer. It is used to identify the client that is connecting to the server.
  • ClientPID: The process ID of the client application.
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 a pull request may close this issue.

1 participant