Skip to content

Angular - Fixing the cookie language provider to set proper cookie path - #25859

Merged
oykuermann merged 1 commit into
rel-10.6from
issue/25858
Jul 24, 2026
Merged

Angular - Fixing the cookie language provider to set proper cookie path#25859
oykuermann merged 1 commit into
rel-10.6from
issue/25858

Conversation

@sumeyyeKurtulus

Copy link
Copy Markdown
Contributor

Description

Resolves #25858

Screenshot 2026-07-23 at 10 51 24 Screenshot 2026-07-23 at 10 53 37

Checklist

  • I fully tested it as developer / designer and created unit / integration tests
  • I documented it (or no need to document or I will create a separate documentation issue)

How to test it?

  • You can switch to this branch for testing test/issue-25858
  • Run this command yarn copy-to:app under npm > ng-packs
  • Use this application for tests templates > app

Verify the bug is fixed

  1. Open DevTools → ApplicationCookieshttp://localhost:4200
  2. Delete .AspNetCore.Culture if it exists
  3. Change the UI language
  4. Check .AspNetCore.Culture:
    • Expected: Path=/
    • Wrong (old behavior): Path=/web
  5. Open DevTools → Network
  6. Trigger an API call (refresh or open a page that loads data)
  7. Select a request to http://localhost:4200/api/...
  8. Inspect Request HeadersCookie
    • Expected: includes .AspNetCore.Culture=...
    • Wrong (old behavior): .AspNetCore.Culture missing

@sumeyyeKurtulus sumeyyeKurtulus added this to the 10.6-final milestone Jul 23, 2026
Copilot AI review requested due to automatic review settings July 23, 2026 08:02
@sumeyyeKurtulus
sumeyyeKurtulus requested review from fahrigedik and oykuermann and removed request for Copilot July 23, 2026 08:02
@oykuermann
oykuermann merged commit e7c0b6d into rel-10.6 Jul 24, 2026
2 of 3 checks passed
@oykuermann
oykuermann deleted the issue/25858 branch July 24, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants