Separate "cookie not found" and "invalid cookie" paths.

(0 comentário) (0 reação) (0 responsável)Rust (1 fork)auto 404

good first issue

Métricas do repositório

Now we just wrap 'cookie problems' in a CookieError but there is a distinct difference between two situations:

There is no cookie with a given name (the cookie was never set, or has expired). This situation is normal and would require the user to login again or restart the oauth flow. This should be a user friendly error.
There is a cookie, but we can't deserialize it. This is not normal and is probably a development problem, because we are the one serializing the cookie and because they are encrypted and authenticated, users are not able to tamper with them. This should be an internal server error.

The work here will have to be done in ephermeral/cookieable.rs and errors.rs.

Direção de pesquisa: Examine o enum CookieError em errors.rs e a lógica de manipulação de cookies em ephemeral/cookieable.rs. Identifique onde o erro é atualmente levantado e divida o em duas variantes distintas: uma para cookie ausente e outra para cookie inválido. Em seguida, atualize todos os pontos de chamada de acordo.
Pilha de tecnologia: rust
Domain: backend
Tipo Issue: Refatoração
Difficulty: 3
Tempo estimado: Meio dia
Status da atividade: Recente
Clarity: Claro
Prerequisites: Rust
Simpatia para novatos: 60