Increased number of errors after updating to 6.3.0+ about malibu HOT 5 CLOSED

Alright then. Things are a bit clearer now, and I sort of know how to investigate it further, I'll close this.

from malibu.

Hey @guilhermearaujo.

I found that "Error Domain=Malibu.NetworkError Code=8" is actually noDataInResponse by doing print(NetworkError.noDataInResponse as NSError).

In the last two released we fixed a couple of issues related to ETags: https://github.com/hyperoslo/Malibu/pull/94 and https://github.com/hyperoslo/Malibu/pull/97. Could it be that you're getting 304 Not Modified for some of your requests? If it so, try to disable Etags functionality if 304 is not supposed to be handled in your app.

from malibu.

Hi @vadymmarkov,
That does seem to be the problem, however it still seems a bit weird:

Prior to #97, the default behaviour for ETag Policy was enabled. This PR changed it to be disabled, and my app does not override it.
Therefore, my app requests (using 6.4.0) should never include the ETag header, I suppose, and consequently, the back-end should not respond with 304.

Is this correct or am I missing a point here?

Update:
I was wrong: the collapsed code made me misinterpret the changes.
GET requests were already supposed to have ETags enabled, and I believed that after these changes they now do work as intended.

from malibu.

Another thing that I haven't understood yet:
Some users reported that the content wouldn't load some times. It kind makes sense if the app gets a 304, but has no cache (or is not using the cache as it should) to complete the promises.

But in my tests, performing the exact same requests, using debug and release builds, I could not reproduce a single error like their reports.

Why isn't if failing consistently? Is it possible that the cache is storing the ETags, but has lost (or corrupted?) the data?

from malibu.

It's hard to say why it isn't failing consistently. You can test ETags by doing the same request multiple times and inspecting the status code. You can also try to clear ETags storage Malibu.clearStorages() to start from the clean state.

from malibu.