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

Fix: Random Crash #9 #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Andrea-Scuderi
Copy link

@Andrea-Scuderi Andrea-Scuderi commented Aug 3, 2019

Hi,
It seems the lambda crashes because of the unwrapped optional.

@Andrea-Scuderi
Copy link
Author

Sorry, it did't solve. It happened again.

@Andrea-Scuderi Andrea-Scuderi reopened this Aug 4, 2019
@Andrea-Scuderi
Copy link
Author

The crash shown in #9 is due to an unhandled try catch.
Fixing this will show the error is related to URLRequest.
Increasing the configuration.timeoutIntervalForRequest to 3600 sec, the lambda will wait for an hour before throwing an error. In theory, if there are no events for more than 20 min the lambda will be killed, so the next start will be cold start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant