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

Sign Out from edX, then sign back in takes you to edX dashboard #391

Open
briangrossman opened this issue Jan 25, 2022 · 6 comments
Open
Labels

Comments

@briangrossman
Copy link

Steps to Reproduce

  • Sign in to MITx Online
  • Subscribe to a course if you haven't already
  • From your dashboard, navigate to the course
  • From within the course click Sign Out
    image
  • Immediately sign back in

Expected Behavior

  • You will be taken to the MITx Online home page after signing back in

Actual Behavior

Related Issues

We had some similar issues in xPRO:

Screenshot or Screencast

Here's a video showing this behavior: https://www.dropbox.com/s/mzmhzlyw5gtbzxn/edXLogoutLogin.mov?dl=0

@rhysyngsun
Copy link
Collaborator

I think the fix is handling the no_redirect param openedx sends when logging out and not redirecting again back to openedx: https://github.com/openedx/edx-platform/blob/ddc55483dad824443f01b59c0da2a32c41eb3002/openedx/core/djangoapps/user_authn/views/logout.py#L100

@umarmughal824 umarmughal824 self-assigned this Feb 17, 2022
@umarmughal824
Copy link
Contributor

umarmughal824 commented Feb 17, 2022

@briangrossman this seems to be working fine on RC. You can have a look at that.

https://www.dropbox.com/home?preview=612b518e-2852-4461-b714-8f02fc477196.webm

We are facing that behavior because on mitxonline login you open edX instance was not really authenticated so that has added the next parameter because of not being logged in. That next parameter is the main cause of its redirection.

@briangrossman
Copy link
Author

@umarmughal824
Your dropbox link didn't work for me, but I was able to confirm that it's working on RC.

@umarmughal824
Copy link
Contributor

So as long as it's working in RC, once we come up with the fix of the MFE authentication issue that's what @asadiqbal08 is working on. It should be working fine on production too.

@umarmughal824 umarmughal824 removed their assignment Mar 25, 2022
@pdpinch
Copy link
Member

pdpinch commented Apr 15, 2022

@briangrossman did you find a workaround for this, by linking the mtixonline dashboard to legacy pages instead of the MFE?

@briangrossman
Copy link
Author

@pdpinch

This issue still exists. The issue you referenced above (#393) fixed the problem where learners weren't automatically signed in to edX when they accessed courses via their dashboard, but it doesn't resolve this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants