🚑️(backend) fix claim contains non user field - Githubissues

numerique-gouv / people

Teams management application

MIT License

13 stars 1 forks source link

🚑️(backend) fix claim contains non user field #548

Open qbey opened 10 hours ago

qbey commented 10 hours ago

Purpose

When we use the feature to get Organization registration number, the claim contains this value and it does not match with any user field.

BUG seen in LA-REGIE-BACKEND-16

Proposal

I switched to a whitelist instead of a blacklist (and two loops, with an if condition on each)

[x] Rewrite the update_user_if_needed for simplification
[x] Test the organization registration number in claim does not break user update