vimprobable-users

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

[Matthew C. <mc...@ah...>]

Dynamically setting a new cookie file based on the host domain automatically would be amazing 

-------- Original message --------
From: "Serge E. Hallyn" <se...@ha...> 
Date: 12/07/2015  1:36 PM  (GMT-05:00) 
To: vim...@li... 
Subject: Re: [Vimprobable-users] [PATCH RFC] Add command to load per-site
  cookie files 

On Mon, Dec 07, 2015 at 07:12:13PM +0100, Hannes Schüller wrote:
> 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.

true.

> 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=...

In fact it is a little confusing :)  But I did it for lazyness'
sake.  I'll probably have 4-5 cookie files (too many to assign 
ctrl-<something> to each one), and I see no way to map, say,
f6 to ":set cookie_file=" leaving me to fill in the name.  So
':c<tab>' works nicely.

I suppose providing a way (if I'm not wrong, and such a way doesn't
already exist :) to allow such mapping, and making cookiefile a
setting, would make the most sense.

------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
_______________________________________________
Vimprobable-users mailing list
Vim...@li...
https://lists.sourceforge.net/lists/listinfo/vimprobable-users

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

[Serge E. H. <se...@ha...>]

That does sound good.  Though when site A grabs content from site B,
which site should we use?

On Mon, Dec 07, 2015 at 03:05:46PM -0500, Matthew Carter wrote:
> Dynamically setting a new cookie file based on the host domain automatically would be amazing 
> 
> -------- Original message --------
> From: "Serge E. Hallyn" <se...@ha...> 
> Date: 12/07/2015  1:36 PM  (GMT-05:00) 
> To: vim...@li... 
> Subject: Re: [Vimprobable-users] [PATCH RFC] Add command to load per-site
  cookie files 
> 
> On Mon, Dec 07, 2015 at 07:12:13PM +0100, Hannes Schüller wrote:
> > 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.
> 
> true.
> 
> > 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=...
> 
> In fact it is a little confusing :)  But I did it for lazyness'
> sake.  I'll probably have 4-5 cookie files (too many to assign 
> ctrl-<something> to each one), and I see no way to map, say,
> f6 to ":set cookie_file=" leaving me to fill in the name.  So
> ':c<tab>' works nicely.
> 
> I suppose providing a way (if I'm not wrong, and such a way doesn't
> already exist :) to allow such mapping, and making cookiefile a
> setting, would make the most sense.
> 
> ------------------------------------------------------------------------------
> Go from Idea to Many App Stores Faster with Intel(R) XDK
> Give your users amazing mobile app experiences with Intel(R) XDK.
> Use one codebase in this all-in-one HTML5 development environment.
> Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
> http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
> _______________________________________________
> Vimprobable-users mailing list
> Vim...@li...
> https://lists.sourceforge.net/lists/listinfo/vimprobable-users

> ------------------------------------------------------------------------------
> Go from Idea to Many App Stores Faster with Intel(R) XDK
> Give your users amazing mobile app experiences with Intel(R) XDK.
> Use one codebase in this all-in-one HTML5 development environment.
> Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
> http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140

> _______________________________________________
> Vimprobable-users mailing list
> Vim...@li...
> https://lists.sourceforge.net/lists/listinfo/vimprobable-users

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

[Matthew C. <m...@ah...>]

"Serge E. Hallyn" <se...@ha...> writes:

> That does sound good.  Though when site A grabs content from site B,
> which site should we use?
>

I would say, whichever site is in the URL bar is the one I'd expect to
set the cookie file (so, site A).

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

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

Matthew Carter <m...@ah...>:
> "Serge E. Hallyn" <se...@ha...> writes:
> 
> > That does sound good.  Though when site A grabs content from site B,
> > which site should we use?
> >
> 
> I would say, whichever site is in the URL bar is the one I'd expect to
> set the cookie file (so, site A).

Hm, that sounds pretty much like the existing "no third party cookies"
setting...

Hannes

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

[Serge E. H. <se...@ha...>]

On Wed, Dec 09, 2015 at 05:17:23AM +0100, Hannes Schüller wrote:
> Matthew Carter <m...@ah...>:
> > "Serge E. Hallyn" <se...@ha...> writes:
> > 
> > > That does sound good.  Though when site A grabs content from site B,
> > > which site should we use?
> > >
> > 
> > I would say, whichever site is in the URL bar is the one I'd expect to
> > set the cookie file (so, site A).
> 
> Hm, that sounds pretty much like the existing "no third party cookies"
> setting...

Well no, the idea would be your config could set

set cookies_file *.launchpad.net cookies_lp
set cookies_file *.github.com cookies_gh
set cookies_file * cookies

and vimprobable2 would just do it.

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

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

On Tue, 8 Dec 2015 22:45:43 -0600, "Serge E. Hallyn" <se...@ha...>
wrote:
> On Wed, Dec 09, 2015 at 05:17:23AM +0100, Hannes Schüller wrote:
> > Matthew Carter <m...@ah...>:
> > > "Serge E. Hallyn" <se...@ha...> writes:
> > > 
> > > > That does sound good.  Though when site A grabs content from
> > > > site B, which site should we use?
> > > >
> > > 
> > > I would say, whichever site is in the URL bar is the one I'd
> > > expect to set the cookie file (so, site A).
> > 
> > Hm, that sounds pretty much like the existing "no third party
> > cookies" setting...
> 
> Well no, the idea would be your config could set
> 
> set cookies_file *.launchpad.net cookies_lp
> set cookies_file *.github.com cookies_gh
> set cookies_file * cookies
> 
> and vimprobable2 would just do it.

Yes, of course, sorry. The original idea is fundamentally different, of
course. I'm just thinking that Matthew's proposal for automatically
handling one file per host name is similar to the existing function
(although also not completely identical).

Hannes

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

[Serge E. H. <se...@ha...>]

On Fri, Dec 11, 2015 at 03:40:14PM +0100, Hannes Schüller wrote:
> On Tue, 8 Dec 2015 22:45:43 -0600, "Serge E. Hallyn" <se...@ha...>
> wrote:
> > On Wed, Dec 09, 2015 at 05:17:23AM +0100, Hannes Schüller wrote:
> > > Matthew Carter <m...@ah...>:
> > > > "Serge E. Hallyn" <se...@ha...> writes:
> > > > 
> > > > > That does sound good.  Though when site A grabs content from
> > > > > site B, which site should we use?
> > > > >
> > > > 
> > > > I would say, whichever site is in the URL bar is the one I'd
> > > > expect to set the cookie file (so, site A).
> > > 
> > > Hm, that sounds pretty much like the existing "no third party
> > > cookies" setting...
> > 
> > Well no, the idea would be your config could set
> > 
> > set cookies_file *.launchpad.net cookies_lp
> > set cookies_file *.github.com cookies_gh
> > set cookies_file * cookies
> > 
> > and vimprobable2 would just do it.
> 
> Yes, of course, sorry. The original idea is fundamentally different, of
> course. I'm just thinking that Matthew's proposal for automatically
> handling one file per host name is similar to the existing function
> (although also not completely identical).
> 
> Hannes

Yup, I don't know if we'd want both, but I'm pretty sure I'd be happy
with just Matthew's proposal.  Happier than with just mine.