Skip to content

Ps 11392 8.4 backport pbkdf2 - #6126

Open
jankowsk wants to merge 2 commits into
percona:8.4from
jankowsk:PS-11392-8.4-backport-PBKDF2
Open

Ps 11392 8.4 backport pbkdf2#6126
jankowsk wants to merge 2 commits into
percona:8.4from
jankowsk:PS-11392-8.4-backport-PBKDF2

Conversation

@jankowsk

Copy link
Copy Markdown
Contributor

There are PBKDF2 related commits cherry-picked and merged from 9.7 + minor corrections to make the stuff compile and some MTRs re-recorded.

@jankowsk
jankowsk force-pushed the PS-11392-8.4-backport-PBKDF2 branch 2 times, most recently from 368ed24 to dc1f4cd Compare August 18, 2026 10:29
harinvadodaria and others added 2 commits August 18, 2026 16:53
WL#17160: Support PBKDF2 storage format with Caching_sha2_password

Description:

  - Added PBKDF2 storage format support for plugin
    caching_sha2_password
  - Added option caching_sha2_password_storage_format to control
    password storage format
  - Added option caching_sha2_password_enforce_storage_format to
    control behavior in case password is hashed using format
    other than the one specified by
    caching_sha2_password_storage_format
  - Made caching_sha2_password_digest_rounds dynamic
  - Added unit and functional tests
Bug#38879953 - X Plugin is missing support for PBKDF2 storage format with Caching_sha2_password

Description
===========
WL#17160 introduced a new storage format type (PBKDF2) for caching_sha2_password.
Without implementing support for this type, the X Plugin will reject
authentications targeting caching_sha2_password accounts that have their
password hash stored using PBKDF2 format.

Fix
===
X Plugin must support new type. Thus both hash types my work with X Protocol:
CRYPT5, PBKDF2_SHA512.

Change-Id: I8fc29705eaa1fe33f2d569253d419cf8405ce6ec
@jankowsk
jankowsk force-pushed the PS-11392-8.4-backport-PBKDF2 branch from dc1f4cd to 1113536 Compare August 18, 2026 14:54

@percona-ysorokin percona-ysorokin 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.

LGTM

Sign up for free to 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.

4 participants