-
Notifications
You must be signed in to change notification settings - Fork 368
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
Add auth hash to external Id #1210
Conversation
c8e8cd1
to
212a5ff
Compare
212a5ff
to
e949f47
Compare
e949f47
to
10b3beb
Compare
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.
Need to add require_user_id_auth
to remote params
@@ -1565,8 +1595,20 @@ public void run() { | |||
return; | |||
} | |||
|
|||
if (remoteParams != null && remoteParams.useEmailAuth && externalIdAuthHash == null) { |
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.
remoteParams.useEmailAuth
should be remoteParams.useUserIdAuth
return; | ||
} | ||
|
||
String internalExternalIdAuthHash = externalIdAuthHash; |
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.
Can we change the name of internalExternalIdAuthHash? Is the only change we are making is making it all lower case? If that is the case maybe include the lowercase aspect in the name?
10b3beb
to
aa99221
Compare
aa99221
to
d1090fd
Compare
* Add auth to setExternalId * Add tests for new functionality
d1090fd
to
3d2ef2c
Compare
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.
external_user_id_auth_hash
is correctly added here so I think it looks good to me.
Needs name params validation
This change is