Skip to content
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

Concurrent transaction error while logging in #50

Open
mbarkley opened this issue Aug 13, 2020 · 0 comments
Open

Concurrent transaction error while logging in #50

mbarkley opened this issue Aug 13, 2020 · 0 comments
Assignees
Labels

Comments

@mbarkley
Copy link
Collaborator

We have automated tests that run using a small set of test users. When two instances of the tests are running simultaneously, they will sometimes fail to login this this error:

{"code":13,"message":"datastore: concurrent transaction"}

That error message is returned in the response for the last request in this series of redirects (that happens after successfully logging in through a connected IDP):

Step #2 - "run_journey_test": 	https://identity-concentrator.staging.dnastack.com/identity/loggedin?code=eyJhbGciOiJIUzI1NiJ9.eyJ0b2tlbktpbmQiOiJjb2RlIiwic3ViIjoiMjE1ODViZjQtM2VmMS00MTY4LWIyZWMtYWE2YWVlOTcyZTU5IiwiYXVkIjoiW10iLCJhenAiOiJkZGU3YzAyMS1kZjc0LTQyOWEtYmYwNC0yYmNkYjZiMGJkZmIiLCJzY29wZSI6InByb2ZpbGUgZ2E0Z2hfcGFzc3BvcnRfdjEgZW1haWwgaWRlbnRpdGllcyBvcGVuaWQiLCJleHAiOjE1OTczNTU4OTQsImlhdCI6MTU5NzM1NTI5NH0.Y9kbbQWaQVvC2ZkX-HlyiomOfoE4oB1VWA46_K-uH60&state=66574932-2180-45f5-bc64-4218aba13a54
Step #2 - "run_journey_test": 	https://identity-concentrator.staging.dnastack.com/identity/v1alpha/master/loggedin/wallet?code=eyJhbGciOiJIUzI1NiJ9.eyJ0b2tlbktpbmQiOiJjb2RlIiwic3ViIjoiMjE1ODViZjQtM2VmMS00MTY4LWIyZWMtYWE2YWVlOTcyZTU5IiwiYXVkIjoiW10iLCJhenAiOiJkZGU3YzAyMS1kZjc0LTQyOWEtYmYwNC0yYmNkYjZiMGJkZmIiLCJzY29wZSI6InByb2ZpbGUgZ2E0Z2hfcGFzc3BvcnRfdjEgZW1haWwgaWRlbnRpdGllcyBvcGVuaWQiLCJleHAiOjE1OTczNTU4OTQsImlhdCI6MTU5NzM1NTI5NH0.Y9kbbQWaQVvC2ZkX-HlyiomOfoE4oB1VWA46_K-uH60&state=66574932-2180-45f5-bc64-4218aba13a54

I suspect that this is timing issue where two of our tests are logging in as the same user at the same time. Although this is not likely to cause errors for real users, it would be appreciated if the IC could support simultaneous logins from the same user.

@chaopeng chaopeng self-assigned this Aug 14, 2020
@chaopeng chaopeng added the p3 label Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants