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.

Sign up for GitHub

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

Jump to bottom

Crash in URLSession-based HTTP client #1467

Open
jbelkins opened this issue May 1, 2024 · 4 comments
Open

Crash in URLSession-based HTTP client #1467

jbelkins opened this issue May 1, 2024 · 4 comments
Assignees
@jbelkins
Labels
bug This issue is a bug.
Milestone
GA

Comments

@jbelkins
Copy link
Contributor

jbelkins commented May 1, 2024 •
edited

Describe the bug

Original customer ticket filed in smithy-swift:
smithy-lang/smithy-swift#706
@jbelkins jbelkins added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels May 1, 2024
@jbelkins jbelkins self-assigned this May 1, 2024
This was referenced May 1, 2024
Merged
Merged
Merged
Closed
@jbelkins
Copy link
Contributor Author

jbelkins commented May 2, 2024

Unable to reproduce the customer's issue but a potential fix for this issue has been delivered to Amplify-swift in aws-sdk-swift 0.36.2. Will follow up with affected customer once this fix has been released by Amplify.

@jbelkins
Copy link
Contributor Author

jbelkins commented May 3, 2024

Customer has been informed of fix on linked smithy-swift issue:
smithy-lang/smithy-swift#706 (comment)

@jbelkins
Copy link
Contributor Author

jbelkins commented May 6, 2024

This issue is separate from the connection timeout issue seen by other Amplify customers:
#1457

Amplify has withdrawn our affected HTTP client from production use due to the latency issue above, and used their own as an interim substitute while we correct the URLSessionHTTPClient latency issues.

Final resolution of this ticket will likely have to wait until Amplify ships with our client again, since we were unable to reproduce the described crashes.

@jbelkins
Copy link
Contributor Author

jbelkins commented May 6, 2024

Potential fix merged in smithy-lang/smithy-swift#707 but has not yet been verified with customers due to withdrawal of out HTTP client from latest amplify-swift.

@jbelkins jbelkins removed the needs-triage This issue or PR still needs to be triaged. label May 6, 2024
@jbelkins jbelkins added this to the GA milestone May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jbelkins jbelkins

Labels
bug This issue is a bug.
Projects
None yet
Milestone
GA
Development

No branches or pull requests
1 participant
@jbelkins