Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upgrade saml-passport version to major 4.0.0 (4.0.4) #6506

Closed
aHenryJard opened this issue Mar 28, 2024 · 0 comments · Fixed by #7387
Closed

Upgrade saml-passport version to major 4.0.0 (4.0.4) #6506

aHenryJard opened this issue Mar 28, 2024 · 0 comments · Fixed by #7387
Assignees
@aHenryJard @nino-filigran
Labels
authentication Linked to authentication, log in, log out, SSO, etc. breaking-change Indicates that the PR introduce a potential breaking change. feature use for describing a new feature to develop solved use to identify issue that has been solved (must be linked to the solving PR)
Milestone
Release 6.2.0

Comments

@aHenryJard
Copy link
Member

aHenryJard commented Mar 28, 2024 •
edited
Loading

Use case

Upgrade of SAML passeport version requires implementation changes. This item is use to follow the topic.

Current Workaround

Proposed Solution

Additional Information

If the feature request is approved, would you be willing to submit a PR?

Yes
@aHenryJard aHenryJard added feature use for describing a new feature to develop needs triage use to identify issue needing triage from Filigran Product team labels Mar 28, 2024
@aHenryJard aHenryJard self-assigned this Mar 28, 2024
@aHenryJard aHenryJard linked a pull request Apr 3, 2024 that will close this issue
Replace dependency passport-saml with @node-saml/passport-saml 4.0.4 - autoclosed #6545
Closed
1 task
@nino-filigran nino-filigran removed the needs triage use to identify issue needing triage from Filigran Product team label Apr 3, 2024
@SamuelHassine SamuelHassine added filigran team use to identify PR from the Filigran team and removed filigran team use to identify PR from the Filigran team labels Apr 20, 2024
@aHenryJard aHenryJard mentioned this issue Jun 17, 2024
Merged
5 tasks
@aHenryJard aHenryJard changed the title Upgrade saml-passport version to major 4.0.0, maybe major 5.0.0 Upgrade saml-passport version to major 4.0.0 (4.0.4) Jun 17, 2024
@aHenryJard aHenryJard linked a pull request Jun 17, 2024 that will close this issue
Upgrade saml-passport to 4.0.4 (#6506) #7387
Merged
5 tasks
@aHenryJard aHenryJard added the breaking-change Indicates that the PR introduce a potential breaking change. label Jun 21, 2024
@Jipegien Jipegien added this to the Release 6.2.0 milestone Jun 24, 2024
aHenryJard added a commit that referenced this issue Jun 25, 2024
@aHenryJard
Upgrade saml-passport to 4.0.4 (#6506)
122d864
@aHenryJard aHenryJard added the solved use to identify issue that has been solved (must be linked to the solving PR) label Jun 25, 2024
dotdev8 pushed a commit to fbicyber/opencti__opencti that referenced this issue Jun 26, 2024
@aHenryJard @dotdev8
Upgrade saml-passport to 4.0.4 (OpenCTI-Platform#6506)
587acfe
@Jipegien Jipegien added the authentication Linked to authentication, log in, log out, SSO, etc. label Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aHenryJard aHenryJard

@nino-filigran nino-filigran

Labels
authentication Linked to authentication, log in, log out, SSO, etc. breaking-change Indicates that the PR introduce a potential breaking change. feature use for describing a new feature to develop solved use to identify issue that has been solved (must be linked to the solving PR)
Projects
None yet
Milestone
Release 6.2.0
4 participants
@SamuelHassine @aHenryJard @Jipegien @nino-filigran