[ckan-dev] killing cookies

David Read david.read at hackneyworkshop.com
Fri Mar 9 17:58:12 UTC 2012


On 9 March 2012 17:45, Toby Dacre <toby.okfn at gmail.com> wrote:
> David,
>
> Thanks for the info.  Yes I'm planning to delete the cookies if they exist
> for a version or two.
>
> I'd quite like to remove the ckan_killtopbar cookie too but that would need
> further discussion and is all client side and so can be ignored for caching
> even if it makes the config slightly more effort.

I agree! Do you mean to use javascript to set the cookie? Surely the
browser will still send the cookie on future requests, which will tell
Varnish not to return from the cache?

BTW I think CKAN should be configurable to not have that the 'welcome
bar' at all, solving this problem for clients that use a cache and
have no need for the welcome bar.

Dave

>
> Toby
>
>
> On 9 March 2012 16:42, David Read <david.read at hackneyworkshop.com> wrote:
>>
>> Toby,
>>
>> These cookies derive from these commits:
>>
>>
>> https://github.com/okfn/ckan/commit/026f78a66322aaae310f178c7fe1da531454d146
>>
>> https://github.com/okfn/ckan/commit/ea92ab801fbf214b7567b8ef31f782517460854e
>>
>> which were put in during a previous effort to improve CKAN pages'
>> cacheability.
>>
>> These cookies are not used in the templates any more, so I'm pleased
>> to see you deleting their creation. May be worth leaving the
>> 'delete_cookie' commands in for a version of CKAN, though?
>>
>> David
>>
>> On 9 March 2012 11:18, Toby Dacre <toby.okfn at gmail.com> wrote:
>> > Hi,
>> >
>> > As part of the #2220 making ckan cacheable work I have removed the
>> > following
>> > client side cookies
>> >
>> > ckan_user
>> > ckan_display_name
>> > ckan_apikey
>> >
>> > From what I can see we do not use these anywhere in core is anyone aware
>> > of
>> > any use cases for them?
>> >
>> > Cheers
>> > Toby
>> >
>> > _______________________________________________
>> > ckan-dev mailing list
>> > ckan-dev at lists.okfn.org
>> > http://lists.okfn.org/mailman/listinfo/ckan-dev
>> >
>>
>> _______________________________________________
>> ckan-dev mailing list
>> ckan-dev at lists.okfn.org
>> http://lists.okfn.org/mailman/listinfo/ckan-dev
>
>
>
> _______________________________________________
> ckan-dev mailing list
> ckan-dev at lists.okfn.org
> http://lists.okfn.org/mailman/listinfo/ckan-dev
>




More information about the ckan-dev mailing list