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: set offline when receiving certain error responses #212

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

crleona
Copy link
Collaborator

@crleona crleona commented Aug 15, 2024

Summary

When receiving certain connection related failures after a failed url request, set the SDK to offline mode. Any successful NWPathMonitor updates will allow us to reconnect. This prevents us from going in a retry loop.

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No

@crleona crleona requested a review from a team August 15, 2024 19:17
@crleona crleona merged commit 056ccbf into main Aug 19, 2024
3 checks passed
@crleona crleona deleted the AMP-107196-set-offline-on-connection-failure branch August 19, 2024 18:09
github-actions bot pushed a commit that referenced this pull request Aug 23, 2024
## [1.8.1](v1.8.0...v1.8.1) (2024-08-23)

### Bug Fixes

* Delete events on an invalid api key response. ([#211](#211)) ([9329a26](9329a26))
* Fix code compatibility issues on Xcode 14.1 ([#213](#213)) ([b2022b6](b2022b6))
* fix pinchGestureRecognizer unavailability issue on tvos ([#217](#217)) ([4c5ae67](4c5ae67))
* set offline when receiving certain error responses ([#212](#212)) ([056ccbf](056ccbf))
Copy link

🎉 This PR is included in version 1.8.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants