[req cookie] fix remove_cookie expires which was leading to expires computed to 0 in set_cookie and the Cookie class interpret that has no expires. Closes #2154654

1 
2  from time import mktime
3  from datetime import datetime
4 
5  # time delta usable to convert localized time to GMT time
6 +# XXX this become erroneous after a DST transition!!!
7  GMTOFFSET = - (datetime.now() - datetime.utcnow())
8 
9  class NoHTTPCacheManager(object):
10      """default cache manager: set no-cache cache control policy"""
11      def __init__(self, view):

12              name, value = value, name[value].value
13          if maxage: # don't check is None, 0 may be specified
14              assert expires is None, 'both max age and expires cant be specified'
15              expires = maxage + time.time()
16          elif expires:
17 +            # we don't want to handle times before the EPOCH (cause bug on
18 +            # windows). Also use > and not >= else expires == 0 and Cookie think
19 +            # that means no expire...
20 +            assert expires + GMTOFFSET > date(1970, 1, 1)
21              expires = timegm((expires + GMTOFFSET).timetuple())
22          else:
23              expires = None
24          # make sure cookie is set on the correct path
25          cookie = Cookie(str(name), str(value), self.base_url_path(),

26          """remove a cookie by expiring it"""
27          if bwcompat is not None:
28              warn('[3.13] remove_cookie now take only a name as argument',
29                   DeprecationWarning, stacklevel=2)
30              name = bwcompat
31 -        self.set_cookie(name, '', maxage=0,
32 -                        # substracting GMTOFFSET because set_cookie
33 -                        # expects a localtime and we don't want to
34 -                        # handle times before the EPOCH
35 -                        expires=date(1970, 1, 1) - GMTOFFSET) 
36 +        self.set_cookie(name, '', maxage=0, expires=date(2000, 1, 1))
37 
38      def set_content_type(self, content_type, filename=None, encoding=None):
39          """set output content type for this request. An optional filename
40          may be given
41          """