search

owncloud / client

🖥️ Desktop Syncing Client for ownCloud
GNU General Public License v2.0
1.4k stars 665 forks source link

[QA] 2.9 warning sync.networkjob.jsonapi: invalid JSON! #8752

Closed gabi18 closed 3 years ago

gabi18 commented 3 years ago

Seen with 2.9daily20210616 (build 4404) on Mac BigSur 11.4

A json warning is written to the log file for JsonApiJob of QUrl( oCIS server )

06-17 13:13:56:662 [ info sync.networkjob.jsonapi ]:    JsonApiJob of QUrl("https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json") FINISHED WITH STATUS "OK"
06-17 13:13:56:662 [ warning sync.networkjob.jsonapi ]: invalid JSON! "" "Ungültiger Wert"
06-17 13:13:56:662 [ warning gui.servernotification ]:  Notifications failed with status code  0
06-17 13:13:56:662 [ debug sync.networkjob ]    [ OCC::AbstractNetworkJob::slotFinished ]:      Network job finished OCC::JsonApiJob("https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json", "959f7cd7-0868-4023-aa0c-852a29b02157", "959f7cd7-0868-4023-aa0c-852a29b02157")

20210617_1313_owncloud.log.1.gz

TheOneRing commented 3 years ago

Please enable the http logger and provide the messages content.

gabi18 commented 3 years ago 
06-17 13:13:56:628 [ info sync.httplogger ]:    "959f7cd7-0868-4023-aa0c-852a29b02157: Request: GET https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json Header: { OCS-APIREQUEST: true, Authorization: Bearer [redacted], User-Agent: Mozilla/5.0 (Macintosh) mirall/2.9.0daily20210616 (build 4404) (ownCloud, osx-20.5.0 ClientArchitecture: x86_64 OsArchitecture: x86_64), Accept: */*, X-Request-ID: 959f7cd7-0868-4023-aa0c-852a29b02157, Original-Request-ID: 959f7cd7-0868-4023-aa0c-852a29b02157, } Data: []"
06-17 13:13:56:629 [ info sync.networkjob ]:    OCC::JsonApiJob("https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json", "959f7cd7-0868-4023-aa0c-852a29b02157", "959f7cd7-0868-4023-aa0c-852a29b02157") created for "https://ocis.team.owncloud.works/" + "ocs/v2.php/apps/notifications/api/v1/notifications" "OCC::ServerNotificationHandler"
TheOneRing commented 3 years ago

and the response to that request?

gabi18 commented 3 years ago

Line is missing in initial comment...

06-17 13:13:56:662 [ info sync.httplogger ]:    "959f7cd7-0868-4023-aa0c-852a29b02157: Response: GET 200 https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json Header: { Content-Length: 0, Date: Thu, 17 Jun 2021 11:13:56 GMT, Vary: Origin, } Data: []"
06-17 13:13:56:662 [ info sync.networkjob.jsonapi ]:    JsonApiJob of QUrl("https://ocis.team.owncloud.works/ocs/v2.php/apps/notifications/api/v1/notifications?format=json") FINISHED WITH STATUS "OK"
06-17 13:13:56:662 [ warning sync.networkjob.jsonapi ]: invalid JSON! "" "Ungültiger Wert"
TheOneRing commented 3 years ago

@butonic are notifications already implemented in ocis?

michaelstingl commented 3 years ago

I wouldn't expect a HTTP 200 status code at an invalid endpoint?

TheOneRing commented 3 years ago

Its an ocis issue and they are aware of it https://github.com/cs3org/reva/issues/1805