Skip to content

[v17] Add SID support for PKINIT#54870

Merged
Tener merged 1 commit intobranch/v17from
tener/backport-54719-v17
Jun 13, 2025
Merged

[v17] Add SID support for PKINIT#54870
Tener merged 1 commit intobranch/v17from
tener/backport-54719-v17

Conversation

@Tener
Copy link
Copy Markdown
Contributor

@Tener Tener commented May 16, 2025

Backport #54719 to branch/v17.

Manual backport due to conflicts:

  • make grpc
  • slog vs logrus, fixed in an extra commit

Changelog: Database protocols using Kerberos (SQL Server, Oracle) can now be configured to fetch user SID for Full Enforcement mapping.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 16, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
tener/backport-54719-v17 bede14c 5 ✅SUCCEED tener-backport-54719-v17 2025-06-13 12:24:38

Copy link
Copy Markdown
Contributor

@greedy52 greedy52 left a comment

Choose a reason for hiding this comment

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

Lets wait v18.0 release testing to test this out before merging the backport.

@Tener
Copy link
Copy Markdown
Contributor Author

Tener commented Jun 13, 2025

Merging now that release testing has validated this change on master.

In the meantime there were some other changes on v17 to this area, so I had to do a manual merge to resolve conflicts.

I re-tested the updated code against actual SQL Server with Kerberos.

@Tener Tener enabled auto-merge June 13, 2025 11:49
@Tener Tener force-pushed the tener/backport-54719-v17 branch from be3a072 to bede14c Compare June 13, 2025 12:19
@Tener Tener temporarily deployed to docs-amplify June 13, 2025 12:19 — with GitHub Actions Inactive
@Tener Tener added this pull request to the merge queue Jun 13, 2025
Merged via the queue into branch/v17 with commit b76ed41 Jun 13, 2025
43 checks passed
@Tener Tener deleted the tener/backport-54719-v17 branch June 13, 2025 13:00
@fheinecke fheinecke mentioned this pull request Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants