Updated recovery error message as per suggestion

ably · Jan 14, 2024 · 22f1e54 · 22f1e54
1 parent f3d3e70
commit 22f1e54
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/ably/realtime_client.go b/ably/realtime_client.go
@@ -42,7 +42,9 @@ func NewRealtime(options ...ClientOption) (*Realtime, error) {
 	if !empty(c.opts().Recover) {
 		recoverKeyContext, err := DecodeRecoveryKey(c.opts().Recover)
 		if err != nil {
+			// Ignoring error since no recover will be used for new connection
 			c.log().Errorf("Error decoding recover with error %v", err)
+			c.log().Errorf("Trying a fresh connection instead")
 		} else {
 			c.Channels.SetChannelSerialsFromRecoverOption(recoverKeyContext.ChannelSerials) // RTN16j
 			c.Connection.msgSerial = recoverKeyContext.MsgSerial                            // RTN16f

diff --git a/ably/realtime_conn.go b/ably/realtime_conn.go
@@ -289,7 +289,9 @@ func (c *Connection) params(mode connectionMode) (url.Values, error) {
 	case recoveryMode:
 		recoveryKeyContext, err := DecodeRecoveryKey(c.recover)
 		if err != nil {
-			c.log().Errorf("error decoding recovery key, %v", err)
+			// Ignoring error since no recover will be used for new connection
+			c.log().Errorf("Error decoding recovery key, %v", err)
+			c.log().Errorf("Trying a fresh connection instead")
 		} else {
 			query.Set("recover", recoveryKeyContext.ConnectionKey) // RTN16k
 		}