Skip to content

Use @node-saml/passport-saml instead#54

Merged
ukch merged 2 commits intofix/passport-samlfrom
fix/passport-saml-2
Jan 30, 2026
Merged

Use @node-saml/passport-saml instead#54
ukch merged 2 commits intofix/passport-samlfrom
fix/passport-saml-2

Conversation

@Fydon
Copy link

@Fydon Fydon commented Jan 26, 2026

This is based on the work of #51, but rebased on eb83311 as newer commits result in no response from express. When bm-develop is working again, this can be rebase onto fix/passport-saml to remove the altered commits from that branch.

@Fydon Fydon requested a review from a team January 26, 2026 16:56
@Fydon Fydon force-pushed the fix/passport-saml-2 branch from 9c0678d to 948d705 Compare January 26, 2026 16:59
Copy link

@ukch ukch left a comment

Choose a reason for hiding this comment

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

This looks good to me. Let's figure out the exact issue with the build and then rebase this into the other PR.

@ukch ukch changed the base branch from bm-develop to fix/passport-saml January 30, 2026 04:12
@ukch ukch force-pushed the fix/passport-saml-2 branch from 948d705 to 8d74a79 Compare January 30, 2026 04:12
@ukch ukch force-pushed the fix/passport-saml-2 branch from 8d74a79 to 20d2873 Compare January 30, 2026 04:20
@ukch ukch merged commit c2932ca into fix/passport-saml Jan 30, 2026
6 checks passed
@ukch ukch deleted the fix/passport-saml-2 branch January 30, 2026 04:21
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.

2 participants