We should forbid invalid userIds in the call of AssumeNewUser. However that is more complicated since
it would require an API call to do so. In that case we are only checking if the userId is a positive value.
Checklist
[x] Have you added appropriate text to the CHANGELOG.md files?
Add those to list or remove the list below altogether:
When you are merging a feature branch into main, please squash merge and make sure the final commit contains any relevent JIRA ticket number. If you are merging from main to staging, or staging to production, please use a regular merge commit.
Does this introduce tech-debt? If so, have you added an entry to the Tech-debt document?
Ticket
resolves #3249
Brief Description
We should forbid invalid userIds in the call of
AssumeNewUser
. However that is more complicated since it would require an API call to do so. In that case we are only checking if theuserId
is a positive value.Checklist
Add those to list or remove the list below altogether:
Notes
When you are merging a feature branch into
main
, please squash merge and make sure the final commit contains any relevent JIRA ticket number. If you are merging frommain
tostaging
, orstaging
toproduction
, please use a regular merge commit.Does this introduce tech-debt? If so, have you added an entry to the Tech-debt document?