Skip to content

Backport: removed invalid RealmName check (#24001) - #24065

Merged
jonkoops merged 1 commit into
keycloak:release/22.0from
edewit:backport-24001
Oct 17, 2023
Merged

Backport: removed invalid RealmName check (#24001)#24065
jonkoops merged 1 commit into
keycloak:release/22.0from
edewit:backport-24001

Conversation

@edewit

@edewit edewit commented Oct 17, 2023

Copy link
Copy Markdown
Contributor

back to having the server check this
now that we have #23667 there are no issues with spaces

back to having the server check this
now that we have keycloak#23667 there are no issues with spaces
@edewit
edewit requested a review from a team as a code owner October 17, 2023 17:37
@ghost ghost added the team/ui label Oct 17, 2023
@jonkoops
jonkoops enabled auto-merge (squash) October 17, 2023 17:56
@cypress

cypress Bot commented Oct 17, 2023

Copy link
Copy Markdown

1 flaky test on run #9454 ↗︎

0 527 48 0 Flakiness 1

Details:

Merge 4159e20 into b42ea8f...
Project: Keycloak Admin UI Commit: 6f5489032f ℹ️
Status: Passed Duration: 16:56 💡
Started: Oct 17, 2023 5:45 PM Ended: Oct 17, 2023 6:02 PM
Flakiness  cypress/e2e/sessions_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Sessions test > revocation > Check if notBefore saved Output Screenshots

Review all test suite changes for PR #24065 ↗︎

@jonkoops
jonkoops merged commit 087bc41 into keycloak:release/22.0 Oct 17, 2023
@cypress

cypress Bot commented Oct 17, 2023

Copy link
Copy Markdown

2 flaky tests on run #9456 ↗︎

0 527 48 0 Flakiness 2

Details:

removed invalid RealmName check (#24001) (#24065)
Project: Keycloak Admin UI Commit: 087bc41bd9
Status: Passed Duration: 16:30 💡
Started: Oct 17, 2023 6:14 PM Ended: Oct 17, 2023 6:30 PM
Flakiness  authentication_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Authentication test > should add a condition Output Screenshots
Flakiness  sessions_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Sessions test > revocation > Check if notBefore saved Output Screenshots

Review all test suite changes for PR #24065 ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants