Fix next page after Cognito login about odm2datasharingportal HOT 4 CLOSED

@aufdenkampe

I was not able to reproduce this error. That might have something to do with my fix in commit ecc4097, but that code should be on staging was we added prior to you opening this issue.

I definitely want to make sure we have this resolved before we release though. Would you mind trying to reproduce this issue again?

from odm2datasharingportal.

@aufdenkampe,

I think I figured out part of the issue. My new commit 7c80506 fixes an incorrect path in the login_failed endpoint. That likely explains the 500 response that you received, however that doesn't explain why why your first logging after setting the new password failed. So I think there is probably still a need for additional testing.

from odm2datasharingportal.

@ptomasula, it might explain my error. I experience the error immediately after setting my account password, so it wasn't from the normal pathway. It worked fine as soon as I tried logging in from a fresh start. If you can't reproduce it, I'm OK with moving forward with having others test the process on Staging.

from odm2datasharingportal.

Some of the Stroud Center team also reported similar issues with the version presently on staging. I tracked the error down to the update method of our ORM implementation. The update method drops the the key value pair from the dictionary provided as an argument if that dictionary key equals the primary field name. This results in a KeyError being raised a few lines later when the cognito code was trying to reference the primary key field in the same dictionary.

I fixed this with commit 9a74fba which adds back in the dropped primary key after the update occurs.

from odm2datasharingportal.