Closed harishv7 closed 1 year ago
Note: Tests will pass once #928 is merged into this
lgtm. Thanks @harishv7 for addressing the comments quickly. Should merge #928 first and make sure the CI testing passed before merging this one.
@QiluXie all checks have passed. merging now, thanks
Problem
We need to have context of user attributes for us to do targeted rollouts/experiments.
Closes IS-312
Solution
We inject information at both the authentication and authorization middlewares based on what we know about the user at that point in time.
Note: To be reviewed in conjunction with IS-310 which introduces GrowthBook for BE.
Breaking Changes