1622306 - Group sameSite console info logs

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P3)

Description

[:Harald Kirschner :digitarald]

Attachment: image.png

… “sameSite” policy set to “lax” because it is missing a “sameSite” attribute and other warnings can be very frequent.

We should group them and reconsider if they are a warning or should be shown in other places like Network details.

Comment 1

[:Harald Kirschner :digitarald]

Baku, bug 1454781 mentions warnings, but the logs are "info", which is unusual for platform warnings. Is there a reason for the specific severity?

Also are you able to help us group those messages for console, similar to how we did it for Tracking Protection in bug 1524276?

Comment 2

[Andrea Marchesini [:baku]]

Baku, bug 1454781 mentions warnings, but the logs are "info", which is unusual for platform warnings. Is there a reason for the specific severity?

There 2 different kinds of messages related to cookie sameSite:

• info in nightly to inform that firefox has taken the decision to treat cookie X as "lax" because the attribute "sameSite" is missing.
• in beta/release we have warning messages saying that, soon, we will treat cookies as sameSite lax if nothing has is specified.

Also are you able to help us group those messages for console, similar to how we did it for Tracking Protection in bug 1524276?

Yes. I can help here. I take this bug.

Comment 3

[:Harald Kirschner :digitarald]

in beta/release we have warning messages saying that, soon, we will treat cookies as sameSite lax if nothing has is specified.

Should we fix those as well or assume that sameSite-lax rolls out soon so we don't need the warnings? I'd leave that to you to decide, since you track it closer.

Comment 4

[Andrea Marchesini [:baku]]

Attachment: Bug 1622306 - Group cookie console info/warning logs, r?ehsan

Comment 5

[Andrea Marchesini [:baku]]

Attachment: Bug 1622306 - Group cookie console info/warning logs - tests, r?nchevobbe

Depends on D66890

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/385bf7e04abb
Group cookie console info/warning logs, r=nchevobbe
https://hg.mozilla.org/integration/autoland/rev/d972e8f4606f
Group cookie console info/warning logs - tests, r=nchevobbe

Comment 7

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/385bf7e04abb
https://hg.mozilla.org/mozilla-central/rev/d972e8f4606f