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

Incorrect user login #955

Closed
rich-hart opened this issue Mar 12, 2015 · 6 comments
Closed

Incorrect user login #955

rich-hart opened this issue Mar 12, 2015 · 6 comments
Assignees

Comments

@rich-hart
Copy link

In beta.cnx.org, webview will automatically sign in under the wrong user name without a password. For example, when visiting beta.cnx.org my browser will automatically sign into 'amwebb' without any prompt.
wrong_login

@reedstrm suggests that this is a problem of incorrectly caching with varnish. Note: clearing my browser cache removes the 'amwebb' login. This issue was first reported in openstax/rhaptos.cnxmlutils/issues/120

@reedstrm reedstrm self-assigned this Mar 12, 2015
@reedstrm
Copy link
Contributor

likely a caching issue just presenting the page. Not sure anyone actually tried say the workspace link to see if you got Amber's stuff.

@amberwebb
Copy link
Contributor

@reedstrm I believe @karenc said she could see my workspace

@rich-hart
Copy link
Author

I could access amber's workspace as well

@mmulich
Copy link
Contributor

mmulich commented Mar 13, 2015

This is a devops issue, not webview. But it's here now...

I'll throw my question here, since I hadn't heard a response:

13:42:58 pumazi : It's likely that the resource is cached with the session info in the response.
...
13:56:42 pumazi : reedstrm: Is it possible for nginx to append a header (maybe something like x-no-cache) when falling back to a protected app?
13:57:52 pumazi : Just trying to figure this out now, rather than implement this cascading /contents/ only to find that we have auth leaks like the one karen just experienced.

The idea with the x-no-cache is to have the caching server see it and ignore it. But there is likely a few ways to achieve the same thing.

@amberwebb
Copy link
Contributor

Closing this as a webview issue.

@reedstrm
Copy link
Contributor

We need a header to tell varnish not to cache pages that are authenticated. Ideally, the app itself should add them, making this an authoring issue.

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

No branches or pull requests

4 participants