UHF-8720: update cookies

[Jussiles]

UHF-8720

UHF-9223

What was done

• Remove outdated cookies from cookie settings
• Remove marketing cookie category
• Add new cookie: GUEST_LANGUAGE_ID
• Update cookie policy version to 1.2.0

How to install

• Make sure Strategia instance is up and running on latest dev branch.
  • git pull origin dev
  • make fresh
  • make drush-cr
• Go to the site and give full cookie consent when cookie banner appears. It's still the old cookie policy version. You need to give consent so you're able to test that cookie banner will reappear after cookie policy version is increased.
• Update the Helfi Platform config
  • composer require drupal/helfi_platform_config:dev-UHF-8720-update-cookies
• Run make drush-updb drush-cr

How to test

• Go to some page. Cookie consent banner should appear
• Check that giving cookie consent with different settings works
• Check that page has correct cookies: https://strategia.docker.so/fi/paatoksenteko-ja-hallinto/evasteasetukset
• Check that cookie consent page doesn't have the removed cookies: https://helsinkisolutionoffice.atlassian.net/wiki/spaces/HEL/pages/8321335444/Hel.fi+keksit#Poistetut-ev%C3%A4steet
• Check "Cookie handling" section of the cookie settings page: https://strategia.docker.so/fi/paatoksenteko-ja-hallinto/admin/config/system/eu-cookie-compliance/settings. Check that is has the update cookies.
• Check that cookies are set after the consent is given and you browse the pages. NOTE: it's hard to test all the cookies in one instance but you can e.g. add remote video paragraph to page for testing Youtube and Helsinkikanava cookies. Kartta.hel.fi and palvelukartta.hel.fi cookies can be tested by adding kartta paragraphs to a page.
• Test that videos and maps are loaded when consent is given and not loaded when consent is not given.
• Test with some other instance too. For testing Chats SOTE should be good.
• Check that code follows our standards

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2b892c3) 13.75% compared to head (32d2a19) 13.75%.

❗ Current head 32d2a19 differs from pull request most recent head 78db1bb. Consider uploading reports for the commit 78db1bb to get more accurate results

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #624   +/-   ##
=========================================
  Coverage     13.75%   13.75%           
  Complexity      238      238           
=========================================
  Files            27       27           
  Lines           880      880           
=========================================
  Hits            121      121           
  Misses          759      759           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[khalima]

Awesome job with the documentation! 🥇
However there seems to be something 🐟 in the allowed_cookies. Can you check it out?

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[khalima]

This is good to be merged. Good job! 🚀

[khalima]

@Jussiles As a reminder; We need to rebuild the cookie explanations when the 4.x branch and/or tags based on that branch have been released. The markup of the <table> tags will change to <figure><table>

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication