feat: redirect to OIDC providers only once in registration flows #5018
Triggered via pull request
November 11, 2024 14:52
David-Wobrock
Status
Cancelled
Total duration
11m 25s
Artifacts
–
ci.yaml
on: pull_request
Generate SDKs
3m 33s
Matrix: Run end-to-end tests
Matrix: Run Playwright end-to-end tests
Build CLI docs
0s
Render version schema
0s
Release SDKs
0s
Annotations
18 errors, 18 warnings, and 3 notices
|
Run tests and lints:
selfservice/strategy/oidc/strategy_registration.go#L231
printf: (github.com/ory/herodot.DefaultError).WithReasonf does not support error-wrapping directive %w (govet)
|
|
Run tests and lints
issues found
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L1
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Test timeout of 30000ms exceeded.
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L86
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Error: page.waitForURL: Test timeout of 30000ms exceeded.
=========================== logs ===========================
waiting for navigation until "load"
============================================================
84 | await loginHydra(page)
85 |
> 86 | await page.waitForURL(
| ^
87 | new RegExp(config.selfservice.default_browser_return_url),
88 | )
89 |
at /home/runner/work/kratos/kratos/test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts:86:18
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L1
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Test timeout of 30000ms exceeded.
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L86
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Error: page.waitForURL: Test timeout of 30000ms exceeded.
=========================== logs ===========================
waiting for navigation until "load"
============================================================
84 | await loginHydra(page)
85 |
> 86 | await page.waitForURL(
| ^
87 | new RegExp(config.selfservice.default_browser_return_url),
88 | )
89 |
at /home/runner/work/kratos/kratos/test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts:86:18
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L1
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Test timeout of 30000ms exceeded.
|
|
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login:
test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts#L86
1) [chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
Error: page.waitForURL: Test timeout of 30000ms exceeded.
=========================== logs ===========================
waiting for navigation until "load"
============================================================
84 | await loginHydra(page)
85 |
> 86 | await page.waitForURL(
| ^
87 | new RegExp(config.selfservice.default_browser_return_url),
88 | )
89 |
at /home/runner/work/kratos/kratos/test/e2e/playwright/tests/desktop/identifier_first/oidc.login.spec.ts:86:18
|
|
Run end-to-end tests (postgres)
Canceling since a higher priority waiting request for 'CI-refs/pull/4064/merge' exists
|
|
Run end-to-end tests (postgres)
The operation was canceled.
|
|
Run end-to-end tests (mysql)
Canceling since a higher priority waiting request for 'CI-refs/pull/4064/merge' exists
|
|
Run end-to-end tests (mysql)
The operation was canceled.
|
|
Run end-to-end tests (cockroach)
Canceling since a higher priority waiting request for 'CI-refs/pull/4064/merge' exists
|
|
Run end-to-end tests (cockroach)
The operation was canceled.
|
|
Run Playwright end-to-end tests (cockroach)
Canceling since a higher priority waiting request for 'CI-refs/pull/4064/merge' exists
|
|
Run Playwright end-to-end tests (cockroach)
The operation was canceled.
|
|
Run end-to-end tests (sqlite)
Canceling since a higher priority waiting request for 'CI-refs/pull/4064/merge' exists
|
|
Run end-to-end tests (sqlite)
The operation was canceled.
|
|
Generate SDKs
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v2. Please update your workflow to use the latest version of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-09-16-notice-of-upcoming-deprecations-and-changes-in-github-actions-services/
|
|
Generate SDKs
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Generate SDKs
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-go@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Generate SDKs
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-go@v2, actions/cache@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run tests and lints
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Run tests and lints
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Playwright end-to-end tests (sqlite)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Run Playwright end-to-end tests (sqlite)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3, actions/checkout@v2, actions/setup-go@v4, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts took 34.9s
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts took 25.7s
|
|
Run Playwright end-to-end tests (mysql)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Run Playwright end-to-end tests (mysql)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3, actions/checkout@v2, actions/setup-go@v4, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts took 29.8s
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts took 25.1s
|
|
Run Playwright end-to-end tests (postgres)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Run Playwright end-to-end tests (postgres)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-node@v3, actions/checkout@v2, actions/setup-go@v4, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_login.spec.ts took 39.0s
|
|
Slow Test:
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts#L1
test/e2e/[mobile-chrome] › mobile/app_recovery.spec.ts took 24.1s
|
|
🎭 Playwright Run Summary
1 flaky
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
36 passed (2.5m)
|
|
🎭 Playwright Run Summary
1 flaky
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
36 passed (2.5m)
|
|
🎭 Playwright Run Summary
1 flaky
[chromium] › desktop/identifier_first/oidc.login.spec.ts:78:9 › account enumeration protection on › login
36 passed (2.5m)
|