-
-
Notifications
You must be signed in to change notification settings - Fork 17
Document new protocol versions in wireprotocol #216
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
Comments
I am stuck at parsing protocol 18 row data. this chapter of the documentation doesn't help 😿 |
Yes, that section still needs to be written :/ The problem seems to be that you calculate the wrong length for the null bitset, which results in an off-by-4 error in reading the response (depending on the number of columns you're selecting): Specifically:
should be
Though I would actually recommend to remove that
|
…ecomment-2788453130
I have published a new version (0.17), but that still doesn't describe reading row data. (Due to aggressive caching, it might take up to 24 hours to become visible.) |
I have published a new version (0.18), which includes information on the row BLR and row data. @einar-hjortdal Could you give it a read, specifically https://firebirdsql.org/file/documentation/html/en/firebirddocs/wireprotocol/firebird-wire-protocol.html#wireprotocol-row-data. As you're fresh in implementing it, you might more easily spot errors or things that are missing. |
Note to self:
|
Currently, wireprotocol documents version 10, and some parts of version 11 and 16 (batches).
The text was updated successfully, but these errors were encountered: