-
Notifications
You must be signed in to change notification settings - Fork 428
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multitenancy for the Client REST API for MAM and MUC-light #3147
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3147 +/- ##
=======================================
Coverage 79.55% 79.55%
=======================================
Files 397 397
Lines 32322 32327 +5
=======================================
+ Hits 25714 25719 +5
Misses 6608 6608
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good in general, I have a few comments.
Log request params
Pass rest_client_SUITE:messages group
Pass muc testcases
Because mod_roster not supported. Because mim2 is not configured.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
This PR addresses "Multitenancy for the Client REST API for MAM and MUC-light".
Proposed changes include:
rooms and messages endpoints
roster not included, because roster does not support MT yet.
We also log more in rest_helper in tests now (similar to the way we log stanzas).