<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 12, 2015 at 5:38 AM, Benoit Chesneau <span dir="ltr"><<a href="mailto:bchesneau@gmail.com" target="_blank">bchesneau@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Mikl what is missing in the cookie implementation of hackney?  There is a new version coming next week so let's try to fix that... (any ticket is welcome).<br clear="all"></blockquote><div><br>Benoit,<br><br>First thing I stumbled upon on second minute of hackney usage is that it silently ignored cookies with Expires attribute,<br>I fixed this in my fork, but looking into code I understood that it was ported from some server codebase,<br>so it uses Cookie parser to parse Set-Cookie, and Set-Cookie builder to build Cookie. They have slightly different syntax<br>and constraints, so it need some refactoring or even full rewrite.<br>Also it seems several different representations for cookies are used inside lib, <br>like {Name, Value}, {Name, [{Name,Value}|Attrs]}, {Name, Value, Attrs}.<br>It would be great to have one unified structure for them.<br>Ticket added, see on github.<br><br>Another issue that bothers me is usage of process dictionary, I always try to avoid this.</div></div><br>-- <br><div class="gmail_signature">Mikl</div>
</div></div>