<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 12, 2015 at 2:32 PM Mikl Kurkov <<a href="mailto:mkurkov@gmail.com">mkurkov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><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></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><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></div></div><div dir="ltr"><div class="gmail_extra"><br>-- <br><div>Mikl</div></div></div></blockquote><div><br></div><div>Thanks! It's useful. I will implement changes for the next coming version aka 2.0.0 . </div><div><br></div><div>I am not sure yet what should be the API. It depends if I still want to support older versions or not. </div><div><br></div><div>- benoit </div></div></div>