search

ably / ably-js

Javascript, Node, Typescript, React, React Native client library SDK for Ably realtime messaging service
https://ably.com/download
Apache License 2.0
310 stars 55 forks source link

`FetchRequest` doesn't work with `checkConnectivity` #1843

Closed michalzaq12 closed 5 days ago

michalzaq12 commented 1 month ago

Ably ver: 2.3.1

FetchRequest doesn't work with checkConnectivity.

No random query parameter is added to connectivityCheckUrl, so any request to connectivityCheckUrl is returned from the browser cache. XHRRequest works properly.

┆Issue is synchronized with this Jira Task by Unito

sacOO7 commented 1 month ago

@michalzaq12 thanks for raising the issue! Currently, we only check for active internet connection when existing connection fails and tries to use a fallback host. Can you confirm if this is causing issue for you in some way ?

VeskeR commented 3 weeks ago

Hi @michalzaq12 ! We were able to reproduce the issue locally, and we can see the bug in the code for FetchRequest. We're going to fix this as soon as possible.

VeskeR commented 5 days ago

This has been fixed in ably-js 2.3.2 release. Thank you for reporting this issue!