Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

fix: resolve cookie handling for large number of Notes sync requests

Description

For syncing a large number of notes in Notes app, the requests often fall into inconsistent cookie state, where some requests contain cookies where others do not.

These cookies make the server create new sessions, forcing it to frequently overwhelm with too many sessions.

In this fix, instead of only clearing the cookie state, the Cookie request header is now explicitly removed for requests where cookies are disabled. (Notes app, for now). The cookie policy is temporarily overridden to IGNORE_COOKIES during the request execution and then restored.

Screenshots

Technical details

Tests

Issues

https://gitlab.e.foundation/e/os/backlog/-/issues/3544

10 commandments of code review

👪 ❤️ code review guidelines

Edited by Fahim M. Choudhury

Merge request reports

Loading