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

[feature] Add support for decoding session state info in Ok packets #227

Closed
mirromutth opened this issue Feb 9, 2024 · 0 comments · Fixed by #230
Closed

[feature] Add support for decoding session state info in Ok packets #227

mirromutth opened this issue Feb 9, 2024 · 0 comments · Fixed by #230
Assignees
Labels
enhancement New feature or request

Comments

@mirromutth
Copy link
Contributor

mirromutth commented Feb 9, 2024

Is your feature request related to a problem? Please describe.

The OkMessage.parse currently does not attempt to decode session state informations, because it is not useful which at the time.

Currently, session state information of type SESSION_TRACK_SYSTEM_VARIABLES contains some potentially useful messages. e.g. auto_commit, transaction_isolation, or redirect_url (MariaDB only)

Describe the solution you'd like

  1. Make Capability.SESSION_TRACK available and disable it by default. Server will not respond session state info if it is disabled
  2. Add session state information decoding in OkMessage, other types except SESSION_TRACK_SYSTEM_VARIABLES remain ignored
  3. Add method Map<String, String> getSystemVariables() to OkMessage for getting value from the map of SESSION_TRACK_SYSTEM_VARIABLES

Additional context

See also MySQL docs and MariaDB docs, MariaDB redirect_url ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant