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

ConfiguredIdpOAuth should produce an oauth window + save JWT to storage #5444

Closed
Tracked by #5311
tomholub opened this issue Oct 5, 2023 · 4 comments · Fixed by #5795
Closed
Tracked by #5311

ConfiguredIdpOAuth should produce an oauth window + save JWT to storage #5444

tomholub opened this issue Oct 5, 2023 · 4 comments · Fixed by #5795

Comments

@tomholub
Copy link
Collaborator

tomholub commented Oct 5, 2023

part of #5311

@ioanmo226
Copy link
Collaborator

@sosnovsky , @tomholub For which domain, Custom IDP is configured?

@ioanmo226
Copy link
Collaborator

I was gonna work this one but i don't know which domain was configured to use custom IDP.

@sosnovsky
Copy link
Collaborator

We currently don't have configured custom IdP, here we just need code skeleton to support this functionality in the future.

Mart started working on implementing custom IdP support (subtasks of #5311) and now we need to finalize remaining parts.

For testing you can configure some IdP (like https://www.okta.com) and then check that authentication process will show 2 popups - first for Gmail permissions and second one for IdP authentication (in case when user has custom AuthenticationConfiguration )

@tomholub
Copy link
Collaborator Author

tomholub commented Oct 13, 2023

Ioan, We could configure an app on Auth0 or Okta and then you could include the configuration details from the IdP in the mock api - it will get returned from the mock FES.

Then you can start the extension that uses the mock - not as a test, but for regular use. Just start the mock separately, then find the extension that was built for mock and load that into the browser.

That will let you test thoroughly.

Then for writing a test, you can configure the mock fes to send configuration that is pointing to mock idp, which we already have. Just with some slightly different parameter in the url, which will make the mock idp render slightly different ui, which you will start checking for during the test. (every time the tests go through idp ui, you'll either check that it's the mock of "normal" idp, or "custom" idp, based on what you were expecting to see)

@sosnovsky sosnovsky modified the milestones: 8.5.3, 8.5.4 Dec 14, 2023
@sosnovsky sosnovsky modified the milestones: 8.5.4, 8.5.5 Mar 1, 2024
@sosnovsky sosnovsky modified the milestones: 8.5.5, 8.5.6 May 13, 2024
@sosnovsky sosnovsky modified the milestones: 8.5.6, 8.5.7 May 23, 2024
@sosnovsky sosnovsky modified the milestones: 8.5.7, 8.5.8 Jun 13, 2024
sosnovsky pushed a commit that referenced this issue Aug 2, 2024
…torage (#5795)

* WIP: add custom idp implementation

* WIP: added ui test

* fix: ui test

* fix: refactor

* fix: typo

* fix: test client id

* fix: pr reviews

* fix: pr reviews

* feat: use chrome.identity.launchWebAuthFlow and updated UI test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment