search

stephanediondev / readerself

Replaced by a new version
https://github.com/stephanediondev/feed
Other
158 stars 26 forks source link

Reeder (iPhone app) crashes when syncing with Fever API #73

Closed mikkelnl closed 8 years ago

mikkelnl commented 8 years ago

Not sure what I can do to debug, but when I set my installation in Reeder app, Reeder will crash when I try to sync.

stephanediondev commented 8 years ago

Do you have access to your web server log?

To check requests from Reeder and status (200, 500, 404 ...)

pawi commented 8 years ago

I found the bug! I experienced the same crashes with Reeder on OSX and i assume that Reeder has the same sync engine for OSX and iOS. When readerself catches a feed item with a unknown author field it leaves the author blank (Null). However during sync this Null field causes Reeder to crash.

To verify i set auh_id to 1 if the field "Is Null" and Reeder syncs every item without a problem.

Up for discussion is where to fix the problem. An obvious choice would be directly on the Fever.php controller and if the author is blank to return "Unknown" or similar.

Another solution would be during feed catching, if the author is unknown readerself provides a well known author when saving the feed item like auh_id = 0, auh_title = Unknown. This would work as the authors database starts with 1. However a data migration for current users is needed.

stephanediondev commented 8 years ago

I made a new release 3.5.3 to fix Fever

String "Unknown" instead of boolean "false" when author is null