- Issue created by @marvil07
- Merge request !260Add Keycloak/Drupal user integrity check drush command → (Merged) created by marvil07
- Status changed to Needs work
10 months ago 3:44am 29 May 2024 - Issue was unassigned.
- Status changed to Needs review
10 months ago 1:42am 4 June 2024 - 🇵🇪Peru marvil07
This is ready for review.
The3448936-kc-integrity-check
and related MR !260 contains the proposed code.The
3448936-kc-integrity-check-batch
branch is there just for comparison.
Using Batch API helped a bit with the performance of the implementation, but the cost was a lot higher complexity.
I would say that performance gain may not be worth in this case.
Hence, no MR opened related to this branch. - First commit to issue fork.
-
drumm →
committed 900b84a5 on 7.x-3.x authored by
marvil07 →
Issue #3448936: Add keycloak user integrity check command
-
drumm →
committed 900b84a5 on 7.x-3.x authored by
marvil07 →
- Status changed to Fixed
9 months ago 10:44pm 25 June 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Active
8 months ago 5:38am 1 August 2024 - Merge request !275Add a processed-keycloak-users-json option to integrity check command → (Merged) created by marvil07
- Status changed to Needs review
8 months ago 5:43am 1 August 2024 - 🇵🇪Peru marvil07
Added new MR https://git.drupalcode.org/project/drupalorg/-/merge_requests/275 with an extra option to use a local file instead of KC REST API.
-
drumm →
committed 3635bc9b on 7.x-3.x authored by
marvil07 →
Issue #3448936 by marvil07, drumm: Add keycloak user integrity check...
-
drumm →
committed 3635bc9b on 7.x-3.x authored by
marvil07 →
- Status changed to Fixed
8 months ago 9:01pm 7 August 2024 - 🇺🇸United States drumm NY, US
We may add more special handling for edge cases, but this is all good to close.
Automatically closed - issue fixed for 2 weeks with no activity.