Opened 5 years ago

Closed 5 years ago

#736 closed New feature (fixed)

unset cookie value if set(name, val) is called with val = Null

Reported by: black silence Owned by:
Priority: major Milestone: Piwik 0.4.1
Component: Core Keywords:
Cc: Sensitive:

Description

There should be a way to unset single cookie values.

I think it should be ok to unset a cookie value if the set method is called with a Null value.

This would save some bandwidth on multipage-installs with the next version of the BlockingCookie plugin.

Attachments (1)

unset_cookie_value.patch (411 bytes) - added by black silence 5 years ago.
patch to remove a single value from a cookie

Download all attachments as: .zip

Change History (8)

comment:1 Changed 5 years ago by vipsoft (robocoder)

  • Milestone changed from 1 - Piwik 0.4 to Features requests - after Piwik 1.0
  • Resolution set to wontfix
  • Status changed from new to closed

Set the cookie expiry in the past. The browser, however, may not delete it until it exits.

In any case, we plan to implement first party cookies.

comment:2 Changed 5 years ago by black silence

single values, I don't want to delete the whole cookie.

I want to delete a specific value from a cookie, only one without affecting others.

comment:3 Changed 5 years ago by matt (mattab)

black silence, please submit a patch and we'll consider...

Changed 5 years ago by black silence

patch to remove a single value from a cookie

comment:4 Changed 5 years ago by black silence

here you go. this patch will unset a cookie value if the set() method is called with a Null value

comment:5 Changed 5 years ago by matt (mattab)

  • Milestone changed from Features requests - after Piwik 1.0 to 1 - Piwik 0.4.1

comment:6 Changed 5 years ago by matt (mattab)

  • Resolution wontfix deleted
  • Status changed from closed to reopened

looks good to me

comment:7 Changed 5 years ago by vipsoft (robocoder)

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [1238]) fixes #736 - feature request to unset cookie value via set(name, null)

Note: See TracTickets for help on using tickets.