LearnersGuild / idm

identity management service
MIT License
2 stars 24 forks source link

Filters createUser feed change - fixes #201 #203 #202

Closed lumodon closed 7 years ago

lumodon commented 7 years ago

Fixes #201, #203

Overview

userCreated worker queue was receiving queue elements upon updating a user - this lead to a big bug where everyone who had their role changed from 'member' to 'learner' fired off workers.

  1. We don't want to trigger workers on updates, only on creation / insert.
  2. We don't care about filtering for 'learner' roles only, but rather want to fire off workers for any creation at all.

Solution: By removing the getAll token the filter for old_val now functions properly (is not null when updating, but is null when newly created) allowing us to only trigger workers when creating new users.

Data Model / DB Schema Changes

None

Environment / Configuration Changes

None

Notes