ably / ably-asset-tracking-swift

iOS client SDKs for the Ably Asset Tracking service.
Apache License 2.0
9 stars 6 forks source link

Fix early-exit bug in `ConnectionConfiguration.getClientOptions` #598

Closed lawrence-forooghian closed 1 year ago

lawrence-forooghian commented 1 year ago

For some reason, this method was exiting early when an authCallback was specified. This definitely seems like an accident, and I can’t find anything to suggest any motivation for it — all of the skipped code seems like it should be run even in the presence of an authCallback.