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

Switch submit-passport API to use lambda functions #404

Closed
7 tasks
nutrina opened this issue Sep 25, 2023 · 0 comments · Fixed by #401 or #416
Closed
7 tasks

Switch submit-passport API to use lambda functions #404

nutrina opened this issue Sep 25, 2023 · 0 comments · Fixed by #401 or #416
Assignees

Comments

@nutrina
Copy link
Collaborator

nutrina commented Sep 25, 2023

As the new load tests have revealed, scaling the service for the submit-passport request is difficult, because these requests take quite a long to to process.
Lambda functions however seem to be able to handle spikes in traffic better (see branch https://github.com/gitcoinco/passport-scorer/tree/1646_lambda ), hence we would like to implement this API endpoint to be served by a lambda function.

  • create a lambda function to handle the submit-passport request
    • the lambda function preferably calls the same handler as the current API
    • rate limit is verified
    • authentication token is verified
  • create tests for lambda function
  • should be triggered on the requests to /registry/submit-passport and /registry/v2/submit-passport. (is possible specify a wildcard /registry/*/submit-passport)
@nutrina nutrina added this to Passport Sep 25, 2023
@nutrina nutrina converted this from a draft issue Sep 25, 2023
@tim-schultz tim-schultz self-assigned this Oct 3, 2023
@tim-schultz tim-schultz moved this from Prioritized to In Progress (WIP) in Passport Oct 3, 2023
@aminah-io aminah-io mentioned this issue Oct 9, 2023
@erichfi erichfi moved this from In Progress (WIP) to Ready to Deploy in Passport Oct 10, 2023
@erichfi erichfi moved this from Ready to Deploy to Done in Passport Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants