SSO login fail for chrome/chromium/apps #3

Closed
opened 2022-12-22 02:02:54 +01:00 by pratyush · 6 comments
Owner
Author
Owner

reported by @nish

Login via sso auth is functional as expected using firefox but fails when using Chrome/Chromium Holds true for mobile browsers (checked Android)

Other browser tests pending!

Furthermore, according to the intercompatibility doc, it should work with the mobile application Tusky.

The issue here is Authentication but it only occurs with this service.

reported by @nish Login via sso auth is functional as expected using `firefox` but fails when using `Chrome`/`Chromium` Holds true for mobile browsers (checked Android) Other browser tests pending! Furthermore, according to the [intercompatibility](https://github.com/jointakahe/takahe/blob/main/docs/intercompatibility.rst) doc, it should work with the mobile application [Tusky](https://f-droid.org/en/packages/com.keylesspalace.tusky/). The issue here is Authentication but it only occurs with this service.
pratyush changed title from Accountz ooo[]]]]]]]]]]]] to SSO login fail for chrome/chromium/apps 2022-12-22 05:03:44 +01:00
Owner

Please provide * client side error messages * screenshots of the error * steps to reproduce the issue

as

but fails

is not descriptive.

Please provide * client side error messages * screenshots of the error * steps to reproduce the issue as > but fails is not descriptive.
Owner

The issue here is Authentication but it only occurs with this service.

That will indeed be an issue if the application tries to authenticate purely to Django (presumably using some headers). The correct solution is for the application to support API tokens, although this not being a user friendly solution, as Takahe is not exposing the functionality for user self-service.

> The issue here is Authentication but it only occurs with this service. That will indeed be an issue if the application tries to authenticate purely to Django (presumably using some headers). The correct solution is for the application to support API tokens, although this not being a user friendly solution, as Takahe is not exposing the functionality for user self-service.
Owner

I was able to reproduce the issue in Chromium. The error can be found upon inspecting the browser console for the POST request to /saml2/acs after submitting the login through the IDP (SSO):

image

Hence the issue has been resolved by changing

SESSION_COOKIE_SECURE = False

to

SESSION_COOKIE_SECURE = True

in settings.py.

Let me know if the issue is equally resolved on your end.

I was able to reproduce the issue in Chromium. The error can be found upon inspecting the browser console for the POST request to /saml2/acs after submitting the login through the IDP (SSO): ![image](/attachments/63316c50-be18-42a3-8a13-014283c30ee6) Hence the issue has been resolved by changing ``` SESSION_COOKIE_SECURE = False ``` to ``` SESSION_COOKIE_SECURE = True ``` in `settings.py`. Let me know if the issue is equally resolved on your end.
129 KiB
Author
Owner

I knew it would resolve that way, just by reading the Network Tab in both since it gives the solution in chrome inspect tool. But I wasnt certain so I left it to you

Agreed, I was to follow up with comment on the two contrasting browser consoles images and suggested soln, I was sidetracked into another issue and we can discuss that

I knew it would resolve that way, just by reading the Network Tab in both since it gives the solution in chrome inspect tool. But I wasnt certain so I left it to you Agreed, I was to follow up with comment on the two contrasting browser consoles images and suggested soln, I was sidetracked into another issue and we can discuss that
Author
Owner

Let me know if the issue is equally resolved on your end.

Aye.

> Let me know if the issue is equally resolved on your end. Aye.
This repo is archived. You cannot comment on issues.
No Milestone
No project
No Assignees
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: LibertaCasa/takahe#3
No description provided.