Skip to content

Fix PGPainless key parameter validation setup - #3254

Merged
sosnovsky merged 4 commits into
masterfrom
issue_3253_fix-PGPainless-key-parameter-validation-setup
Aug 13, 2026
Merged

Fix PGPainless key parameter validation setup#3254
sosnovsky merged 4 commits into
masterfrom
issue_3253_fix-PGPainless-key-parameter-validation-setup

Conversation

@DenBond7

@DenBond7DenBond7 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

his PR fixes the PGPainless initialization to enable key parameter validation on the policy assigned to the new global PGPainless instance.

Previously, the setup modified the policy of the old global instance, which was discarded immediately afterward. As a result, key parameter validation remained disabled.

The PR also:

  • Adds a regression test for the production PGPainless configuration.
  • Re-enables testPublicKey_Issue1358() after the resolution of PGPainless issue #488.
  • Verifies that the issue 1358 public key can be parsed with production key parameter validation enabled.
  • Restores the original global PGPainless instance after each test.

close#3253


Tests(delete all except exactly one):

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7DenBond7 changed the title fix PGPainless key parameter validation setupFix PGPainless key parameter validation setupAug 11, 2026
@DenBond7DenBond7 mentioned this pull request Aug 11, 2026
5 tasks
@DenBond7
DenBond7 marked this pull request as ready for review August 11, 2026 19:15
…roduction policy for corrupted key validation

@sosnovskysosnovsky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sosnovsky
sosnovsky merged commit 94691d7 into masterAug 13, 2026
7 checks passed
@sosnovsky
sosnovsky deleted the issue_3253_fix-PGPainless-key-parameter-validation-setup branch August 13, 2026 11:34
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PGPainless key parameter validation is not enabled after migration to 2.x

2 participants

@DenBond7@sosnovsky