cityofaustin / atd-data-tech

Austin Transportation Data & Technology Services
17 stars 2 forks source link

Research Data Tracker Login Errors in Chrome #2121

Closed johnclary closed 4 years ago

johnclary commented 4 years ago

Some users are getting an error when they try to sign in to the Data Tracker with their COACD credentials in Chrome.

Please work with Billy Howland to reproduce the issue (#2110).

From a user who experienced the issue:

When I tried to sign in with my COACD, it shows "Cannot GET /v1/applications/5815f29f7f7252cc2ca91c4f/auth/COACD/undefined" on a blank page When I tried to log in with my email and password, it logs me in but I cannot open anything.

Some test cases:

mateoclarke commented 4 years ago

Response from BH today:

It actually seems to be resolved at this time, I was able to get in via Chrome today! So nothing to worry about.

mateoclarke commented 4 years ago

i've reached out to BVDW to set a time for screen sharing.

dianamartin commented 4 years ago

@mateoclarke @johnclary @mddilley My computer restarted after the software upgrade and I tried to log into Data Tracker Prod and still am receiving an error (using Chrome). I tried logging into our other COACD enabled apps and they worked just fine, SMB and Finance & Purchasing (in Chrome). I think something is up with the app, maybe we need to check the code?

mddilley commented 4 years ago

After screen sharing with Brian, I was able to replicate the same message that Mateo and I saw on his machine but only with the themed app in a Chrome Incognito window.

Confirm the issue occurs when using our themed Data Tracker page: http://transportation.austintexas.io/data-tracker**

  • Unable to replicate in Firefox with/without private window
  • Unable to replicate in Chrome
  • Replicated in Chrome Incognito (worked on personal MBP with same Chrome version & Incognito, though)

Confirm if the issue occurs when using the Knack out-of-box app: https://atd.knack.com/amd

  • Unable to replicate in Firefox with/without private window-
  • Unable to replicate in Chrome
  • Unable to replicate in Chrome Incognito

Confirm whether clearing cache/incognito work affects anything

  • See above for Incognito results for themed and out of the box Knack app

Confirm what happens if user resets their COACD password

  • Unable to test on a Mac

Confirm if issue happens offsite/off-network

  • All testing was offsite and off-network

Confirm whether the user can login in Chrome using a named user account (not a COACD login)

  • Brian was unable to log in with a name user account through the login form here, need to check with @dianamartin if this is possible at all

Further resources

Microsoft note about Chrome 80

(might be relevant but doesn't explain why SSO works in Chrome outside of incognito) https://docs.microsoft.com/en-us/office365/troubleshoot/miscellaneous/chrome-behavior-affects-applications

Knack SSO Docs

https://support.knack.com/hc/en-us/articles/231796207-Single-Sign-On-with-Google-Facebook-Twitter-and-Custom

DTS Knack SSO docs

https://atd-dts.gitbook.io/austin-transportation-knack-guides/knack/open-data-portal-socrata-integration

johnclary commented 4 years ago

thanks @mddilley. this is a head-scratcher, huh? does anything catch your eye in our themed HTML template? E.g., looks we're using a fairly old version of Bootstrap 4.

The only other thing I can think of that makes this app special is in the SAML config. The issuer is urn:oasis:names:tc:SAML:2.0:metadata. That's a really weird issuer name, which I used at the time because i was fumbling through my first config. It shouldn't matter, but I could ask CTM to set up a new auth profile for this app with a less weird issuer name.

dianamartin commented 4 years ago

@mddilley @johnclary I had my issue happen using normal Chrome, not Chrome incognito. I experienced the same error message the users have been getting.

Using Chrome

image

Using Chrome Incognito

It prompted me to go into the City of Austin log in - I put in credentials and then I get this screen image

mateoclarke commented 4 years ago

~Confirmed with Brian that the out of the box https://atd.knack.com/amd worked (forgot to check this in our call yesterday) and he's been working at Cameron Rd on the network.~

mateoclarke commented 4 years ago

Brian followed up and said the following:

I lied. Had changed my default viewer to Firefox, so it didn’t open in Chrome when I clicked on the link. Below is what I get when I log in with Chrome.

image

amenity commented 4 years ago

@SurbhiBakshi and I just got another report from Paloma at ROW. Chrome didn't work, Firefox was fine.

johnclary commented 4 years ago

ugh. also note that @dianamartin is getting this error on the new HR app, which does not use a custom wrapper (in contrast to AMD Data Tracker).

this is gross and i'm fairly stumped.

johnclary commented 4 years ago

i think it's time for a knack ticket. what do y'all think @dianamartin @mddilley @mateoclarke ?

amenity commented 4 years ago

Agreed. I'm nervous about Parking needing to log in to the HR app with Brazos/Android. Will they even have another browser?

mateoclarke commented 4 years ago

yeah, if it weren't happening using the OOTB Knack app page, then I was going to start investigating the wrapper code, but that seems ruled out now. I think we should escalate with Knack.

dianamartin commented 4 years ago

I agree with @mateoclarke. @amenity We'll have to find out what can go on the Brazos devices

johnclary commented 4 years ago

pinning this here for future debugging: https://chrome.google.com/webstore/detail/saml-chrome-panel/paijfdbeoenhembfhkhllainmocckace?hl=en

amenity commented 4 years ago

Submitted a support ticket (101489) to Knack:

Note that I was wrong about the SMB app having this behavior. Screen Shot 2020-03-24 at 5 59 43 PM

johnclary commented 4 years ago

i compared two POST requests from the SAML exchange. for some users, the redirectUrl is undefined in the response. when working successfully, the redirectUrl is defined correctly, e.g https://atd.knack.com/hr?state=p%3DCOACD%26s%3Dhome%26h%3Dhome.

i won't post the request JSON here in case it contains tokens.

at first glance it appears to be an issue with the Knack server response, but i suppose it could be happening earlier in the chain, with Azure ADFS

amenity commented 4 years ago

@johnclary - did you want to reach out to someone at CTM to see whether something is happening on the ADFS end of things? Or other next steps? If not, this could go in the backlog. Workaround (using Firefox) has been working for everyone. 🤷‍♀️

amenity commented 4 years ago

We haven't heard any reports of this lately - probably because everyone switched to Firefox? @dianamartin is going to check with Jenny, B VdW, et al. and test herself.

dianamartin commented 4 years ago

Testing Chome ADFS/COACD log-in

Knack apps - previously earmarked with issues

Testing Results

dianamartin commented 4 years ago

Just messaged Brian VDW this morning about it, since I saw him "available".

dianamartin commented 4 years ago

Users to Ask to test Chrome again

Emailed users

We’ve followed up with testing AMD Data Tracker on the chrome browser and it appears that the error messages have resolved themselves. I wanted to ask if everyone could try it again since you folks were the only ones that had “error messages” a few months back.

Please reply and let me know the results via email or Teams message.

amenity commented 4 years ago

Haven't heard any reports of this being a problem; potentially resolved on the CTM side of things.