Skip to content
This repository was archived by the owner on Dec 20, 2018. It is now read-only.
This repository was archived by the owner on Dec 20, 2018. It is now read-only.

Cookie names not set properly. #763

@JunTaoLuo

Description

@JunTaoLuo

I overlooked the fact that setting the AuthenticationScheme values does not actually achieve what we want. By default, schemes are set to .AspNetCore.{AuthenticationScheme} and this leads to names like .AspNetCore..AspNetCore.Identity.Application since cookies middleware adds the .AspNetCore. prefix by default. We can remove the .AspNetCore. from the prefix of the authentication scheme.

Still investigating the why there are also cookies like .AspNetCore.Microsoft.AspNetCore.Identity.ExternalLogin.

Edit: Ignore the ExternalLogin cookie issue. That is a really old legacy name for the cookie and the test is only checking it for non-existence.

fyi @natemcmaster

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions