Cookie module issues

Ouverte
  • Quelque chose ne va pas ?
#194 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
25/100
Type d'issue
Bug
Clarté
À clarifier
Activité
À l'abandon
Stack technique
erlang
Domaine
api, backend

Piste de recherche

Commencez par examiner le module hackney_cookie et le point d’entrée hackney:cookies(Headers), y compris le comportement traité par PR #193. Comparez le parser et le builder avec RFC6265, puis examinez les différentes représentations des cookies utilisées dans le projet. Le travail est considéré comme terminé lorsque l’analyse de Set-Cookie et la construction de Cookie sont correctes, et qu’une représentation cohérente a été définie.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

api enhancement http http11 lib rfc

It seems that cookie parser in hackney_cookie uses 'Cookie' parser to parse 'Set-Cookie' header and 'Set-Cookie' builder to build 'Cookie' header. Also code seems to be outdated as it tries to parse comma as valid separator, that not the case in last RFC6265.
As one particular problem that I'm run into is parser failing to proceed set-cookie with Expires attribute. If you try to request https://ya.ru it sets yandexuid cookie and removes S cookie (with Expires in past), but hackney:cookie silently ignores this:

> {ok, Status, Headers, Client} = hackney:request(get, "https://ya.ru", [], <<>>, []).
...
> Headers.
[{<<"Server">>,<<"nginx">>},
 {<<"Date">>,<<"Fri, 12 Jun 2015 11:21:05 GMT">>},
 {<<"Content-Type">>,<<"text/html; charset=UTF-8">>},
 {<<"Content-Length">>,<<"9627">>},
 {<<"Connection">>,<<"close">>},
 {<<"Cache-Control">>,
  <<"no-cache,no-store,max-age=0,must-revalidate">>},
 {<<"Expires">>,<<"Fri, 12 Jun 2015 11:21:05 GMT">>},
 {<<"Last-Modified">>,<<"Fri, 12 Jun 2015 11:21:05 GMT">>},
 {<<"P3P">>,
  <<"policyref=\"/w3c/p3p.xml\", CP=\"NON DSP ADM DEV PSD IVDo OUR IND STP PHY P"...>>},
 {<<"Set-Cookie">>,
  <<"S=; Expires=Tue, 14-Jun-2005 11:21:05 GMT; Path=/">>},
 {<<"Set-Cookie">>,
  <<"yandexuid=6091834691434108065; Expires=Mon, 09-Jun-2025 11:21:05"...>>},
 {<<"X-Frame-Options">>,<<"DENY">>}]

> hackney:cookies(Headers).
[]

This particular problem addressed in PR #193.
But I think cookie module need more attention to implement proper 'Set-Cookie' parser and 'Cookie' builder with respect to RFC6265.
Another issue is that different cookies representations are used across project like {Name, Value}, {Name, [{Name,Value}|Attrs]}, {Name, Value, Attrs}. This adds complexity to lib usage and can be source of errors, maybe it make sense to go with some unified representation based on record or map.

Langage dominant
Erlang
Étoiles
1.4k
Forks
458
Merge moyen
11 h 32 min
PR mergées (30 j)
47

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de benoitc/hackney

Toutes les issues de benoitc/hackney

Issues similaires

Plus d'issues Backend & API Design

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.