Удалил куки сессии AspNetCore.Cookies, что делать?

Question

[Nik Faraday]

Здравствуйте
Я случайно удалил куку, название которой указано в названии вопроса (AspNetCore.Cookies) из-за чего у меня не хочет теперь работать аутентификация по кукам и выдаёт ошибку, которая изображена на скриншоте ниже.



Ошибку пытался гуглить в интернете и как-то и поиски нечего не дали. Ниже соответствующий скриншот, куки. Не знаю, поможет ли он чем-то

Comments

[insighter]

Покажите, как вы создаете-настраиваете identity.
Вообще удаление куки тут должно быть не причем, больше похоже, что вы некоторое время назад сделали успешный SignIn, кука прописалась, а потом вы стали по другому формировать параметр identity, куку удалили, а новый механизм авторизации не работает

[Nik Faraday]

insighter, это интересное решение
Скажу сразу, что с этом всё я только относительно недавно и это первый тестовый проект для настройки регистрации и авторизации, так что говорить что-то более сложно будет
Я использую в Program.cs следующую запись для настройки аутентификации

services.AddAuthentication(CookieAuthenticationDefaults.AuthenticationScheme)
    .AddCookie(options =>
    {
        options.LoginPath = "/User/SignIn";
        options.AccessDeniedPath = "/User/SignIn";
        options.ExpireTimeSpan = TimeSpan.MaxValue;
        options.SlidingExpiration = true;
    });
services.AddAuthorization();

Та же ниже показана часть сервисного класса, который я вызываю для проведения самой аутентификации

public class AuthenticationService : IAuthenticationService
    {
        private readonly IHttpContextAccessor _accessor;

        public AuthenticationService(IHttpContextAccessor accessor)
        {
            _accessor = accessor;
        }

        public void SignIn(UserEntity userEntity)
        {
            List<Claim> claims = new List<Claim>()
            {
                new Claim(ClaimsIdentity.DefaultNameClaimType, userEntity.Login),
                new Claim(ClaimsIdentity.DefaultRoleClaimType, userEntity.Type.ToString()),
                new Claim(ClaimTypes.Email, userEntity.Email),
            };

            ClaimsIdentity identity = new ClaimsIdentity(claims, "ApplicationCookie", ClaimsIdentity.DefaultNameClaimType, ClaimsIdentity.DefaultRoleClaimType);
            _accessor.HttpContext.SignInAsync(CookieAuthenticationDefaults.AuthenticationScheme, new ClaimsPrincipal(identity));
        }

И вот сама часть экшена на контроллере

[IgnoreAntiforgeryToken]
        [HttpPost]
        public async Task<IActionResult> SignIn([FromBody]SignInViewModel viewModel)
        {
            if (!ModelState.IsValid)
            {
                return BadRequest(new { responseMessage = "Invalid data" });
            }

            var userList = _userService.GetByLoginAndPass(viewModel.Login, viewModel.Password);
            if (userList.Count == 0)
            {
                return NotFound(new { responseMessage = "User not found" });
            }

            UserEntity userEntity = userList[0];
            _authenticationService.SignIn(userEntity);

            return Ok(new { responseMessage = "Success" });
        }

На механизм выборки пользователя не обращайте внимания, я его использую пока как примитивную заглушку

[insighter]

Nik Faraday,

new Claim(ClaimsIdentity.DefaultRoleClaimType, userEntity.Type.ToString()),

из за этого наверно и посыпалось - userEntity.Type.ToString() посмотрите, что отсюда возвращается,
наверно, что то такое что в куку прописать не получается

Ну и по хорошему лучше использовать метод Unauthorized вместо NotFound
Unauthorized(new ProblemDetails { Title = "User not found" });

NotFound - нужен для случаев когда клиент запрашивает некий ресурс с перманентой "пропиской" а метод его может найти у себя. Например если бы вы обрабатывали запрос типа "получить пользователя с заданным идентификатором". Лучше лишний раз не нарушать семантику HTTP ответов.

Ну и ещё :)) При неудачной аутентификации (то, что вы делаете называете аутентификацией) лучше обобщайте все ошибки в одну - "Неправильное имя пользователя или пароль, возможно пользователь не зарегистрирован". С точки зрения безопасности так лучше

[Nik Faraday]

insighter, клайма там нормально генерится, по крайней мере, всё работало )
На счёт замены NotFound нужно будет посмотреть, как оно кидается на ajax
Ну, я уточнял, что за аутентификацию относительно недавно сел, до этого такое не делал. Я стараюсь прописать свою логику и как-то это изучить. Спасибо

Ответ нашёл, причина была в конфигурации, дублирую код ниже

services.AddAuthentication(CookieAuthenticationDefaults.AuthenticationScheme)
    .AddCookie(options =>
    {
        options.LoginPath = "/User/SignIn";
        options.AccessDeniedPath = "/User/SignIn";
        options.ExpireTimeSpan = TimeSpan.MaxValue;
        options.SlidingExpiration = true;
    });
services.AddAuthorization();

Сама строчка
options.SlidingExpiration = true;
Не позволяла создать новую куку при авторизации

[insighter]

Nik Faraday, это очень косвенная причина. Я понял в чем дело, ты просто частично удалил куку (там остались ещё две части) и ASP.Core при попытке её продлить выдавала внутреннюю ошибку.

Лучше вернуть SlidingExpiration, и руками поудалять все куки включая ai_session ai_user

[Nik Faraday]

insighter, А как лучше тогда задать конфигурацию для кук ?
Я видел много примеров, но эта тема не очень хорошо раскрыта на просторах интернета

[insighter]

Nik Faraday, у тебя все правильно, просто ASP.NET не смогла нормально переварить форс-мажорную ситуацию. Разработчики не ожидают, что кто то руками будет куки частично удалять. Все ок.

По ошибке было понятно, что ASP.NET Core Identity не могла сгенерировать куку, мне было интересно в чем именно причина была.

[Nik Faraday]

insighter, напишите, пожалуйста, заключение в ответы, я помечу как правильный ответ
Спасибо

[Nik Faraday]

insighter, С отсылкой на нашу дискуссию в комментариях )

[insighter]

Nik Faraday, ок

Answer 1

[insighter]

Ошибка того, что куки были удалены не полностью, а только одна "AspNETCore.Cookies". Руками надо удалить остальные две "ai_session", "ai_user".

ASP.Core при попытке продления куки выдавала внутреннюю ошибку.