Re: [Vimprobable-users] [PATCH RFC] Add command to load per-site cookie files

[Hannes S. <ha...@yl...>]

Hi Serge,

thanks for this patch!

"Serge E. Hallyn" <se...@ha...>:
> The default cookie site is ~/.config/vimprobable/cookies.  With this
> patch, you can do
> 
> :cookies gh
> or
> :cookies lp
> 
> to load a per-site cookiefile called ~/.config/vimprobable/cookies_gh
> or cookies_lp (i.e. for github and launchpad).  This allows you to
> segregate cookies by sites, and more easily keep an eye on suspect
> sites.

I basically have two comments.

1. I see no way of switching back to the original file.
2. What's the rationale for defining this as a new command rather than
   a "setting"? From your original description, I would have expected
   something like
   :set cookiefile=...
   Don't you think it could be a little confusing to have both
   :set cookies=(on|off)
   *and*
   :cookies=...

Hannes