search

awslabs / aws-mobile-appsync-sdk-ios

iOS SDK for AWS AppSync.
https://awslabs.github.io/aws-mobile-appsync-sdk-ios/
Other
262 stars 128 forks source link

Crash on APIKeyBasedConnectionPool #595

Closed petracvrljevic closed 3 months ago

petracvrljevic commented 5 months ago

I'm facing crashes similar to those reported in https://github.com/awslabs/aws-mobile-appsync-sdk-ios/pull/569/files. Could it be fixed?

ruisebas commented 5 months ago

Hi @petracvrljevic, thanks for opening this issue.

We will take a look and post updates here. Ideally the fix done for IAMBasedConnectionPool could be directly applied for APIKeyBasedConnectionPool as well.

petracvrljevic commented 4 months ago

@ruisebas any update about this?

ruisebas commented 4 months ago

I've opened a PR applying the same fix for this issue. Once it gets merged into the main branch, we will release a new version containing it.

If you wish, you can validate the fix in your app by pointing to the ruisebas/apikey_connection_queue branch.

petracvrljevic commented 4 months ago

@ruisebas I'm having problems with updating library though pods, getting CDN: trunk URL couldn't be downloaded: https://cdn.cocoapods.org/Specs/a/a/3/AWSCore/2.36.2/AWSCore.podspec.json Response: Failure when receiving data from the peer

petracvrljevic commented 4 months ago

Changed to use it from spm

petracvrljevic commented 4 months ago

But I don't see new version tagged?

thisisabhash commented 4 months ago

Hello @petracvrljevic , We are planning to release it early next week - We will post here when it's released.

thisisabhash commented 4 months ago

Hello @petracvrljevic, 3.7.1 has been released. You may check it out.

petracvrljevic commented 4 months ago

Hi, I get CDN: trunk URL couldn't be downloaded: https://cdn.cocoapods.org/Specs/b/e/3/AWSAppSync/3.7.1/AWSAppSync.podspec.json Response: Failure when receiving data from the peer when I try to update my pod

thisisabhash commented 4 months ago

Can you try these commands in the terminal?

pod repo remove trunk
pod install --repo-update
petracvrljevic commented 4 months ago

It works now, thank you so much!

github-actions[bot] commented 3 months ago

This issue is now closed. Comments on closed issues are hard for our team to see. If you need more assistance, please open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.