Skip to content

Sleep if timeout is hit during default attachments processing #40

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

Merged
merged 1 commit into from
Jun 11, 2025

Conversation

patricijabrecko
Copy link
Contributor

Summary

This pull request introduces a minor version update to the library and adds a new utility for handling timeouts, which is integrated into the WorkerAdapter class. This mitigates the issue of lambda timing out. The most important changes include the addition of a sleep function and a default sleep delay constant, as well as their usage in the WorkerAdapter to handle timeouts.

Related Issues

Type of Change

  • Change doesn't affect products or customers
  • Bug fix (non-breaking change which fixes an issue)
  • Non-breaking change (the new functionality and code refactor do not require a migration strategy)
  • Breaking change (fix or feature that will require a migration plan for data or other services)
  • Documentation/comment update
  • Other (please describe):

Testing Procedure

  • Run any ADaaS snap-in where you experience problems with lambda timeout-ing during attachments extraction, using this library version: 1.5.1

Checklist

  • I used generative AI to generate this PR
  • I have self-reviewed my code for clarity and correctness
  • I have added or updated comments for complex or non-obvious logic in my code
  • I have updated relevant documentation (e.g., README, code docs)
  • My changes do not introduce new warnings or errors
  • I have added or updated tests to cover new or changed functionality
  • All tests pass locally with my changes applied

@patricijabrecko patricijabrecko merged commit 28b6921 into main Jun 11, 2025
4 checks passed
@patricijabrecko patricijabrecko deleted the attachments-processing-timeout branch June 11, 2025 13:36
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.

2 participants