Skip to content

[v13] improve postgres bind params audit logging#34434

Merged
GavinFrazar merged 1 commit intobranch/v13from
gavinfrazar/v13-update-postgres-bind-logging
Nov 10, 2023
Merged

[v13] improve postgres bind params audit logging#34434
GavinFrazar merged 1 commit intobranch/v13from
gavinfrazar/v13-update-postgres-bind-logging

Conversation

@GavinFrazar
Copy link
Copy Markdown
Contributor

@GavinFrazar GavinFrazar commented Nov 9, 2023

Backport #34321 to branch/v13.

Manual backport to resolve a merge conflict in the audit test file only, which was just some ClickHouse stuff that didn't exist in v13.

changelog: Added binary formatted parameters as base64 encoded strings to PostgreSQL Statement Bind audit log events

Backport #34321 to branch/v13.

* log with same format for all params when one format code is given
* log binary format params as base64 encoded string
@github-actions github-actions Bot added backport database-access Database access related issues and PRs size/sm labels Nov 9, 2023
@GavinFrazar GavinFrazar added this pull request to the merge queue Nov 10, 2023
Merged via the queue into branch/v13 with commit ef51129 Nov 10, 2023
@GavinFrazar GavinFrazar deleted the gavinfrazar/v13-update-postgres-bind-logging branch November 10, 2023 19:43
This was referenced Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport database-access Database access related issues and PRs size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants