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

[Regression][Logout] On Safari 8 Can't Logout #86

Closed
2jour opened this issue Oct 20, 2015 · 24 comments
Closed

[Regression][Logout] On Safari 8 Can't Logout #86

2jour opened this issue Oct 20, 2015 · 24 comments
Assignees
Milestone

Comments

@2jour
Copy link

2jour commented Oct 20, 2015

Reporter: Ladybug
OS: mac ox 10.10.5
Browser: Safari 8
server: staging 2.2.2

  1. login to staging via homepage
  2. logout of via about
    Observed (if cache clear you should be able to logout the first time)
  3. login via homepage
  4. logout via about

Observed you can't you are stuck. The only way to logout is via homepage

Expected: You should be able to logout

@2jour 2jour added this to the 2.2.2 - www milestone Oct 20, 2015
@2jour
Copy link
Author

2jour commented Oct 20, 2015

For the sauce fans .. this one is for you :)
https://saucelabs.com/tests/e0842c308227473ea34de6a5f154845f

@2jour
Copy link
Author

2jour commented Oct 22, 2015

@rschamp let me know if you need help with repro

@2jour
Copy link
Author

2jour commented Oct 22, 2015

Ok now its getting really hotel california

I am in a situation where i can't logout even from home page.

aaaaagh

@thisandagain
Copy link
Contributor

Can't repro when pointing at scratch.mit.edu. @rschamp can you confirm / deny?

@2jour
Copy link
Author

2jour commented Oct 23, 2015

Can repro with host file pointing at scratch.mit.edu

@thisandagain i can walk you through test case

@2jour
Copy link
Author

2jour commented Oct 23, 2015

Could we leave prio high tag until its conclusive its a ghost?

@rschamp
Copy link
Contributor

rschamp commented Oct 23, 2015

I can't repro

@2jour
Copy link
Author

2jour commented Oct 23, 2015

@wyrobnik able to repro on safari 8

@2jour
Copy link
Author

2jour commented Oct 23, 2015

I even tried this link /accounts/logout/?next=/ on safari does not logout

However /accounts/logout does

@2jour
Copy link
Author

2jour commented Oct 23, 2015

actually it does redirect for /accounts/logout
so fix use accounts/logout?

@rschamp rschamp removed their assignment Oct 23, 2015
@2jour 2jour removed the triage label Oct 23, 2015
@2jour
Copy link
Author

2jour commented Oct 23, 2015

@rschamp still an issue even after hotfix change. I am using discuss to logout and still notice issue. discuss has accounts/logout in sign out

However if i type accounts/logout on discuss page it works. I can logout

I am using host to point to scratch.mit.edu

@2jour
Copy link
Author

2jour commented Oct 23, 2015

One thought

Could session/ be the cause of the Safari 8 signout issues?

It is issued roughly after logout I think. If it is issued before logout or at the same time would this cause a problem?
@thisandagain @rschamp ?

@2jour
Copy link
Author

2jour commented Oct 23, 2015

could we console.log the calls for this to find out? safari has bad network tools

@2jour
Copy link
Author

2jour commented Oct 24, 2015

Is this gating? @thisandagain ?

@2jour
Copy link
Author

2jour commented Oct 25, 2015

I marked prio high because didn't want to forget this issue. If this is not gating then remove prio high :)

@2jour 2jour changed the title [Regression][www][Logout] On Safari Can't Logout [Regression][www][Logout] On Safari 8 Can't Logout Oct 25, 2015
@thisandagain thisandagain modified the milestones: 2.2.3 - HOC, 2.2.2 - www Oct 25, 2015
@2jour
Copy link
Author

2jour commented Oct 28, 2015

Getting a few complaints on Bugs and Glitches .. I am bumping to high

@2jour 2jour changed the title [Regression][www][Logout] On Safari 8 Can't Logout [Regression][Logout] On Safari 8 Can't Logout Oct 30, 2015
@thisandagain
Copy link
Contributor

Marked as "needs verification" and assigning to @2jour for confirmation / repro.

@2jour
Copy link
Author

2jour commented Nov 2, 2015

@rschamp and I both are still able to repro

@2jour
Copy link
Author

2jour commented Nov 2, 2015

Just tried it again and can't sign out

@2jour
Copy link
Author

2jour commented Nov 2, 2015

From Slack

Oct 30

ray [2:37 PM]
Everything seems ok except that the hotel california bug persists

@2jour 2jour removed their assignment Nov 9, 2015
@2jour
Copy link
Author

2jour commented Nov 9, 2015

https://scratch.mit.edu/discuss/topic/166913/?page=1#post-1592931 i think this user may be impacted too. Safari 7

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

3 participants