Skip to content

[v12] improve postgres bind params audit logging#34436

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

[v12] improve postgres bind params audit logging#34436
GavinFrazar merged 1 commit intobranch/v12from
gavinfrazar/v12-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/v12.

* 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/v12 with commit f2675a4 Nov 10, 2023
@GavinFrazar GavinFrazar deleted the gavinfrazar/v12-update-postgres-bind-logging branch November 10, 2023 19:43
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