Skip to content
Snippets Groups Projects

#3470408 Add route to process keycloak user delete event relay

Merged #3470408 Add route to process keycloak user delete event relay
All threads resolved!
All threads resolved!

Then, add it to a new drupalorg_keycloak_events_user_delete queue if it is valid.

Finally, on queue processing, query KC one more time, and only remove the user from Drupal if the user on KC does not really exist.

Closes #3470408

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Neil Drumm
  • Neil Drumm requested changes

    requested changes

  • Marco Villegas added 7 commits

    added 7 commits

    • 89da64f8...4ef3af93 - 3 commits from branch project:7.x-3.x
    • 6b48c191 - Add route to process keycloak user delete event relay
    • 05d45550 - Avoid full HTML response
    • 739889ab - Skip processing the queue if not enough data is present
    • 78d42933 - Refactor endpoint for Keycloak to be more generic

    Compare with previous version

  • Marco Villegas requested review from @fjgarlin and @drumm

    requested review from @fjgarlin and @drumm

  • Neil Drumm added 5 commits

    added 5 commits

    • 67bf44ce - 1 commit from branch project:7.x-3.x
    • 9f69754e - Add route to process keycloak user delete event relay
    • 3b180953 - Avoid full HTML response
    • edc322eb - Skip processing the queue if not enough data is present
    • 69ae606a - Refactor endpoint for Keycloak to be more generic

    Compare with previous version

  • Neil Drumm added 1 commit

    added 1 commit

    • c157c7c8 - Make queue processing more generic too

    Compare with previous version

  • Neil Drumm resolved all threads

    resolved all threads

  • Neil Drumm approved this merge request

    approved this merge request

  • merged

  • Marco Villegas mentioned in merge request !281 (merged)

    mentioned in merge request !281 (merged)

  • Please register or sign in to reply
    Loading