On Sat, Jun 29, 2013 at 2:44 AM, Christopher Hunt
<notifications@github.com>wrote:

> In framework/src/play/src/main/scala/play/api/libs/ws/WS.scala:
>
> > @@ -481,6 +482,58 @@ object WS {
> >  }
> >
> >  /**
> > + * A WS Cookie.
> > + */
> > +class Cookie(val ahcCookie: AHCCookie) {
>
> Why is making this a trait "doing it properly"? From a modelling
> perspective, class feels correct - cookies are not going to be mixed into
> other things...
>

I think it's so that you can swap out the implementation without touching
the interface -- You can have one Cookie interface, and then have a
NingCookie or a SprayCookie implementation.

Amended the PR to show usage of NingCookie.

Will.