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

Microsoft Edge and forgotten password failure #1065

Open
LeePorte opened this issue Aug 2, 2019 · 7 comments
Open

Microsoft Edge and forgotten password failure #1065

LeePorte opened this issue Aug 2, 2019 · 7 comments
Labels
closed due to age Close issue or PR soon because no reaction unscheduled waiting-4-pr Pull-Request is wanted in order to proceed

Comments

@LeePorte
Copy link

LeePorte commented Aug 2, 2019

What version of UAA are you running?

4.35

How are you deploying the UAA?

I am deploying the UAA

  • using cf-deployment

What did you do?

Triggered the forgotten password flow by using the link on UAA. This was then opened by Microsoft Edge (system default browser). This loads the password reset page as expected.

What did you expect to see? What goal are you trying to achieve with the UAA?

I expected to see that the password which met the complexity requirements was accepted.

What did you see instead?

The following error message

There is a problem “Invalid login attempt, the request does not meet our security standards and may indicate that the action was not originated by you. Please try again.”

Additional info

We have no issues with Chrome, Firefox, Safari. Just with Edge. Unfortunately some of our users are on Enterprise managed Windows estates and are unable to change browsers. Once the password has been reset normal functionality continues with Edge.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/167668383

The labels on this github issue will be updated when the story is started.

@LeePorte LeePorte changed the title Microsoft Edge andforgotten password failure Microsoft Edge and forgotten password failure Aug 2, 2019
@LeePorte
Copy link
Author

We have had a further report from a user trying to log in

When trying to login using corporate device from Edge I'm getting the message below. I'm unable to login to the PaaS.

There is a problem 
Invalid login attempt, the request does not meet our security standards and may indicate that the action was not originated by you. Please try again.

OS - Windows 10 Enterprise 1809 
Browser - Edge 44.17763.1.0

@mwdb
Copy link
Contributor

mwdb commented Sep 20, 2019 via email

@LeePorte
Copy link
Author

Hi Martijn,

I assume you mean the browser console, is this correct?

Thanks

Lee

@LeePorte
Copy link
Author

LeePorte commented Sep 23, 2019

Hi Martijn,

There is nothing related to a session cookie in the console, only a minor CSS error.

Thanks

Lee

@paroxp
Copy link

paroxp commented Nov 8, 2019

o/

We did some investigation to establish what the problem is and how can we fix that for our valued users and I'd like to give a bit of an update on that issue.

The actual issue here is, that UAA is always returning 302 with Location: /login instead of for instance 404.

It was a little bit of red herring, as in our case it is only visible on Microsoft Edge and Internet Explorer.

Here's what's happening:

  • The user visits a page served by UAA (say /login, or /accept/invitation - anything that returns a form)
  • The page contains an SVG, with an image fallback
  • Most browsers are happy with the SVG and don't request the image, but IE / Edge quirkily request both
  • The path to the image is wrong, but because the user is not logged in it returns a 302 to /login instead of a 404
  • The browser follows the 302 redirect, now looking for the image at /login
  • Because the response for /login is a form, it has a Set-Cookie header that overwrites the csrf token for the form the user is actually looking at.
  • When the user submits the form, the token in the HTML doesn't match the token in their cookie, so their request is denied.

More details: alphagov/paas-uaa-customized-boshrelease#14

@strehle strehle added the waiting-4-pr Pull-Request is wanted in order to proceed label Jan 20, 2022
@strehle
Copy link
Member

strehle commented Jan 20, 2022

@paroxp can you please provide a PR to this repo

@strehle strehle added the closed due to age Close issue or PR soon because no reaction label Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed due to age Close issue or PR soon because no reaction unscheduled waiting-4-pr Pull-Request is wanted in order to proceed
Projects
Development

No branches or pull requests

5 participants