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

Logout SLO is not working #332

Closed
umardraz opened this issue Dec 24, 2018 · 3 comments
Closed

Logout SLO is not working #332

umardraz opened this issue Dec 24, 2018 · 3 comments

Comments

@umardraz
Copy link

umardraz commented Dec 24, 2018

Hello I am running PHPSAMLSSO, I have one wordpress website and one nodejs site.

Now the issue is whenever I am logged on both site using SSO and try to logout from wordpress site then it send/redirect me to my nodejs site.

But if I try to logout from nodejs site then both site successfully logout from SSO.

It means there might be issue with my code or passport-saml modules has lack of SLO (logout) functionality here is my logout code

app.get('/logout', function(req, res) {
  samlStrategy.logout(req, function(err, request) {
    if (!err) {
      res.redirect(request);
    }
  });
});

app.get('/logout/callback', function(req, res) {
  req.logout();
  res.redirect('/');
});

Please help me how I can solve this issue

@umardraz umardraz changed the title Logout SLO is working Logout SLO is not working Dec 24, 2018
@stavros-wb
Copy link
Contributor

@umardraz make sure you:
• Use the latest passport-saml release (1.0.0)
• Configure SimpleSAMLphp with SingleLogoutService (for the node project) to point to the /logout endpoint, so it uses the samlStrategy.logout as well

@christiaanwesterbeek
Copy link

And also, I believe that:

app.get('/logout/callback'...

actually should be:

app.post('/logout/callback'...

See: #221 (comment)

@cjbarth
Copy link
Collaborator

cjbarth commented Sep 20, 2021

Closed due to inactivity.

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

No branches or pull requests

4 participants