Hi Team, I am facing some strange issues with Cogn...
# helpm
Mr.9715
06/01/2021, 2:02 PMHi Team, I am facing some strange issues with Cognito triggers and local testing. I created a presignup lambda trigger and directly did
context.done(null, event);a
Ashishkumar Pandey
06/01/2021, 2:06 PMcould you share the register handler, it says invalid JSON for register.
m
Mr.9715
06/01/2021, 2:09 PMHi, thanks for replying. It worked when I removed the trigger from Cognito. I am going to deploy to check if the issue persists.
a
Ashishkumar Pandey
06/01/2021, 2:10 PMokay. đ
m
Mr.9715
06/01/2021, 2:15 PMDeployed lambda worked fine. This is only happening with local testing.
a
Ashishkumar Pandey
06/01/2021, 2:16 PMwow! thatâs weird.
m
Mr.9715
06/01/2021, 2:28 PMyep, and issue exist in local testing.
Mr.9715
06/01/2021, 2:47 PMI think I kind of understand why this is happening,
below ss is how my presignup trigger logs looks like when local testing. vs what I expect and receiving after it's deployed. (Conclusion is sst.Function cannot be used with cognito triggers. and cognito triggers cannot be local tested.)
@Frank tagging you for issue visibility. I am going to replace the sst.function with CDK.function for the triggers.
f
Frank
06/01/2021, 4:42 PM
Hey @Mr.9715 this seems to be the issue
{"type":"success","data":null} Copy code
context.done(null, event)m
Mr.9715
06/01/2021, 5:23 PMHey @Frank, No idea where the event goes, I logged the event and I can see the event object received by the trigger function both locally and when deployed. for some reason, this event is lost when the message is transmitted through lambda proxy when local testing.
f
Frank
06/01/2021, 5:45 PM
Change:
Copy code
export.handler = function(event, context) {
context.done(null, event)
} Copy code
export.handler = function(event, context, callback) {
callback(null, event);
}Frank
06/01/2021, 5:47 PM
SST currently doesnât support
context.donenodejs10.xcallbackFrank
06/01/2021, 5:49 PM
Give it a try!
m
Mr.9715
06/01/2021, 7:45 PMHi @Frank, It worked. But I faced another issue. callback is not returning the errors that I am explicitly throwing. it instead returns
InvalidLambdaResponseException{ "type": "success", body: none/event }callback(new Error("an error")). or callback("Error") or callback("Error", event)Mr.9715
06/01/2021, 7:48 PMI am trying to deny creation of account for existent users in the presignup triggrr. If user creates an account with email and add /verify phone no. later. (and pool allows signin with both email, phone and username), everything works fine until another user tries to create an account with same phone to which cognito creates new account and marks phone as unverified in previous account.
f
Frank
06/01/2021, 7:54 PM
Does this work
callback(new Error("an error"), event)m
Mr.9715
06/01/2021, 7:58 PMNope, it returns success every time i return event. with or without error.
f
Frank
06/02/2021, 1:37 AM
The issue you are running into doesnât seem to be Cognito trigger related anymore. Itâs simply
callback(new Error("My Error"), event); Copy code
export function main(event, context, callback) {
const error = new Error("my error");
callback(error, event);
}Frank
06/02/2021, 1:42 AM
Maybe try with a simple Lambda and see if you can reproduce the issue?
m
Mr.9715
06/02/2021, 4:18 AMAh, You are right. I was missing a
return