Add Context.Cookies

[frederikcreemers]

This gives easy access to request cookies, and makes it easy to set cookies on the response.

Closes #391

[markbates]

This looks great! Should we have a function that let's people set an http.Cookie? I love the helper methods for the common use case, but also being able to set a complex cookie the user defines would be great.

[frederikcreemers]

I didn't add this, because I thought these people would just use http.SetCookie directly. The whole reason for this wrapper to exist, is so people don't have to manually construct cookies, or even import the http package. If I were to add this method, it'd simply call http.SetCookie with the context's responseWriter, and the passed-in cookie. The only advantage of calling this wrapper rather than the original, is that you don't have to pass in the response, but I don't think that's a huge benefit. But ultimately, it's simple to implement, and doesn't add any complexity, so I can implement it if you want.

…

On Sat, Jul 29, 2017 at 9:08 PM Mark Bates ***@***.***> wrote: This looks great! Should we have a function that let's people set an http.Cookie? I love the helper methods for the common use case, but also being able to set a complex cookie the user defines would be great. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#569 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAtA1eYKrH4yP8x9KMUlgeHo0J4WfIFJks5sS4MXgaJpZM4OnauC> .

[markbates]

@bigblind good point.

[markbates]

@bigblind It looks like your branch is in some crazy weird state. It has huge conflicts with development and says 25 files have changed and such. Can you figure out what's going on there so I can merge? Thanks.

[frederikcreemers]

@markbates, I'm sorry, I messed up a bit. should be fixed now.