-[ARTRest time:] will fail, but without an error, if no decoder found for response’s MIME type

[lawrence-forooghian]

If self->_encoders[response.MIMEType] is nil, then we'll end up calling the callback with both time and error being nil, which is not a valid behaviour.

ably-cocoa/Source/ARTRest.m

Line 519 in 0fe3c59

ARTErrorInfo *dataError = [self->_encoders[response.MIMEType] decodeErrorInfo:data error:&decodeError];

We need to handle this (if the feature spec has something to say about how this should be handled, then we should do that, else we should probably define it in the spec – the SDK should generate some sort of error).

┆Issue is synchronized with this Jira Task by Unito

[maratal]

As in case of Accept header in this comment #1420 (comment) I think it's a legacy code in the _time function. Error creation is handled for all requests in the executeRequest:... method:

ably-cocoa/Source/ARTRest.m

Line 372 in 0fe3c59

NSError *dataError = [self->_encoders[response.MIMEType] decodeErrorInfo:data error:&decodeError];

Issue to update _time function - #1445