search

Igalia / cog

WPE launcher and webapp container
MIT License
236 stars 61 forks source link

launcher: set preferred language list from system locale #732

Open rawoul opened 1 month ago

rawoul commented 1 month ago

This changes the Accept-Language header to match the system locale by default.

rawoul commented 1 month ago

Thanks for the review, I've fixed the issues you raised and factored the code in a separate function for clarity.

TingPing commented 1 month ago

I have one concern, this exposes system information directly to the web which adds information for fingerprinting. Browsers like Safari specifically filter and reduce this list to make it less unique.

If this does or doesn't matter to Cog is probably for another dev to decide.