WordPress-iOS icon indicating copy to clipboard operation
WordPress-iOS copied to clipboard

WordPress.AuthenticationService.RequestAuthCookieError.wpcomCookieNotReturned

Open AliSoftware opened this issue 3 years ago • 4 comments

Sentry Url: https://sentry.io/share/issue/16c4e855765d4b8a848f5f7439e3140b/ User Count: 2491 Count: 85027 First Release: 17.7.1.0 First Seen: 2021-07-26T07:51:14.557000Z Last Seen: 2022-05-10T17:02:56Z 24 Hours: 291 30 Days: 9428

WordPress.AuthenticationService.RequestAuthCookieError.wpcomCookieNotReturned

AliSoftware avatar May 10 '22 17:05 AliSoftware

This error, wpcomCookieNotReturned, is one I added logging for because I suspected a while back that sometimes when the app requests a cookie, the server returns a "success" response (e.g. a HTTP 200 response code), but doesn't include the requested cookie in the response payload.

Git blame shows this commit message:

Log error if preview auth cookie is not found

When the user previews a post or page, the app fetches an auth cookie from the server. It appears that there are some instances where the server sends back an empty auth cookie, so this PR adds a check there and logs an error if the auth cookie is missing from the server's response.

I think this should have used logMessage instead of logError to avoid appearing as a crash in Sentry (which changes it from a Sentry log level of Error to just Info.

guarani avatar May 18 '22 23:05 guarani

I think this is low-priority but changing the log level will help reduce noise.

guarani avatar May 18 '22 23:05 guarani

We're seeing a pretty high number of these – we'll probably want to prioritize it

jkmassel avatar May 29 '25 04:05 jkmassel

I have searched for the error in Sentry, but the error event count does not seem very high. @hannahtinkler 's recent fix of https://linear.app/a8c/issue/AINFRA-106/ should help mitigate this error.

crazytonyli avatar May 29 '25 22:05 crazytonyli