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

Router - Return 503 if Lambda Invoke gives permission error #148

Open
5 tasks
huntharo opened this issue Apr 11, 2024 · 0 comments
Open
5 tasks

Router - Return 503 if Lambda Invoke gives permission error #148

huntharo opened this issue Apr 11, 2024 · 0 comments
Labels
area-router Router portion of the app enhancement New feature or request good first issue Good for newcomers

Comments

@huntharo
Copy link
Contributor

huntharo commented Apr 11, 2024

Motivations

  • Currently, if the Lambda Invoke gets a permission error, it will just keep retrying until it times out the request
  • Need to add a test to make sure that this case ends (stops trying to invoke and removes the request as in progress)
  • Should return a 503 immediately to the caller when the error is: is not authorized to perform: lambda:InvokeFunction on resource:
    • There is no direct relationship between the PendingRequests and the Lambdas 
    • Perhaps when the LIM detects this error it should notify the Dispatcher to clear all pending requests with a 503

Acceptance Criteria

  • Add LambdaInstanceManager test that simulates lambda invoke giving a 503
  • Raise event from LIM when Lambda gives a 503
  • Subscribe to event in Dispatcher
  • Clear all pending requests when a 503 is received
  • Add Dispatcher test for the event clearing the pending requests
@huntharo huntharo added the area-router Router portion of the app label Apr 12, 2024
@huntharo huntharo moved this from Backlog to Ready in Lambda Dispatch Public Access Apr 19, 2024
@huntharo huntharo added enhancement New feature or request good first issue Good for newcomers labels Apr 20, 2024
@huntharo huntharo moved this from Ready to Backlog in Lambda Dispatch Public Access Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-router Router portion of the app enhancement New feature or request good first issue Good for newcomers
Projects
Status: Backlog
Development

No branches or pull requests

1 participant