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.

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

OIDC logout remote not working #7766

Closed
samczsun opened this issue Jul 21, 2024 · 0 comments · Fixed by #8466
Closed

OIDC logout remote not working #7766

samczsun opened this issue Jul 21, 2024 · 0 comments · Fixed by #8466
Assignees
Labels
authentication Linked to authentication, log in, log out, SSO, etc. bug use for describing something not working as expected solved use to identify issue that has been solved (must be linked to the solving PR)
Milestone

Comments

@samczsun
Copy link

Description

I've set PROVIDERS__OPENID__CONFIG__LOGOUT_REMOTE to true, but I'm not being redirected to the logout endpoint. I believe this is due to the logout_remote variable not being copied into the options variable:

const options = { client, passReqToCallback: true, params: { scope: openIdScope } };

openIDStrategy.logout_remote = options.logout_remote;

Environment

  1. OS (where OpenCTI server runs): Ubuntu
  2. OpenCTI version: 6.2.6
  3. OpenCTI client: Frontend
  4. Other environment details:

Reproducible Steps

  1. Set logout_remote to true
  2. Logout

Expected Output

The logout endpoint should redirect me to the end session endpoint

Actual Output

The logout endpoint redirects me to /

Additional information

Screenshots (optional)

@samczsun samczsun added bug use for describing something not working as expected needs triage use to identify issue needing triage from Filigran Product team labels Jul 21, 2024
@nino-filigran nino-filigran removed the needs triage use to identify issue needing triage from Filigran Product team label Jul 29, 2024
@nino-filigran nino-filigran added this to the Bugs backlog milestone Jul 29, 2024
@nino-filigran nino-filigran added the authentication Linked to authentication, log in, log out, SSO, etc. label Aug 12, 2024
@aHenryJard aHenryJard self-assigned this Sep 11, 2024
@aHenryJard aHenryJard added the solved use to identify issue that has been solved (must be linked to the solving PR) label Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authentication Linked to authentication, log in, log out, SSO, etc. bug use for describing something not working as expected solved use to identify issue that has been solved (must be linked to the solving PR)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants