Skip to content

Conversation

@sampaiodiego
Copy link
Member

2021-10-21 · 2 🐛 · 3 👩‍💻👨‍💻

Engine versions

  • Node: 12.22.1
  • NPM: 6.14.1
  • MongoDB: 3.6, 4.0, 4.2, 4.4, 5.0
  • Apps-Engine: 1.28.0

🐛 Bug fixes

  • Queue error handling and unlocking behavior (#23522)

  • SAML Users' roles being reset to default on login (#23411)

    • Remove roles field update on insertOrUpdateSAMLUser function;

    • Add SAML syncRoles event;

👩‍💻👨‍💻 Core Team 🤓

KevLehman and others added 3 commits October 21, 2021 18:03
* Error handling on queue processing

* add error handling to queue execution and lock timeout

* add error handling to queue execution and lock timeout
* Remove roles update on insertOrUpdateSAMLUser

* Check SAML Role Sync setting before updating users' roles

* Add rolesSync variable

* Add role sync event

* Remove isEnterprise check

* New code

* Updated unit test for the new behavior

Co-authored-by: Pierre Lehnen <pierre.lehnen@rocket.chat>
@sampaiodiego sampaiodiego requested review from a team October 21, 2021 21:11
@sampaiodiego sampaiodiego merged commit 17db619 into master Oct 21, 2021
@sampaiodiego sampaiodiego deleted the release-4.0.4 branch October 21, 2021 22:36
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.

5 participants