Skip to content

Commit 9c9d2dc

Browse files
BrennanConroyamcasey
authored andcommitted
Add partitioned to cookie for SignalR browser testing (#57997)
* Add partitioned to cookie for SignalR browser testing Looks like Chromium (not chrome) headless now requires 'partitioned' on the cookie when using `Secure` and `Same-Site=None` * Apply suggestions from code review Co-authored-by: Andrew Casey <[email protected]> --------- Co-authored-by: Andrew Casey <[email protected]>
1 parent d438fdf commit 9c9d2dc

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/SignalR/clients/ts/FunctionalTests/Startup.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,11 @@ public void Configure(IApplicationBuilder app, IWebHostEnvironment env, ILogger<
184184
{
185185
cookieOptions.SameSite = Microsoft.AspNetCore.Http.SameSiteMode.None;
186186
cookieOptions.Secure = true;
187+
cookieOptions.Extensions.Add("partitioned"); // Required by Chromium
187188

188189
expiredCookieOptions.SameSite = Microsoft.AspNetCore.Http.SameSiteMode.None;
189190
expiredCookieOptions.Secure = true;
191+
expiredCookieOptions.Extensions.Add("partitioned"); // Required by Chromium
190192
}
191193
context.Response.Cookies.Append("testCookie", "testValue", cookieOptions);
192194
context.Response.Cookies.Append("testCookie2", "testValue2", cookieOptions);

0 commit comments

Comments
 (0)