emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emacs-bug-tracker] bug#6957: closed (url-cookie-expired-p)


From: GNU bug Tracking System
Subject: [Emacs-bug-tracker] bug#6957: closed (url-cookie-expired-p)
Date: Thu, 09 Sep 2010 05:38:02 +0000

Your message dated Thu, 09 Sep 2010 01:39:07 -0400
with message-id <address@hidden>
and subject line Re: bug#6957: url-cookie-expired-p Redux (at last)
has caused the GNU bug report #6957,
regarding url-cookie-expired-p
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
6957: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6957
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: url-cookie-expired-p Date: Tue, 31 Aug 2010 13:07:05 -0700
Hello,

Please forgive me if this list is not the correct recipient for this
message. If there is a better destination, I will be happy to send it
on.

I am working on a web services client that is mostly a wrapper around
URL Package. I am using GNU Emacs v.23.2. My client implementation
requires keeping tight rein on a session ID cookie. My sessions are
terminating abnormally. I believe I have tracked this issue to a bug
in url-cookie-expired-p.

If the cookie's expiration date is the same as today's date,
url-cookie-expired-p normalizes the times to seconds and compares the
difference. I believe the problem is in the normalization algorithm.
In pseudo-code, the current implementation does:

(+ (* 360 seconds) (* 60 minutes) (* 1 hours))

I believe this should be:

(+ (* 1 seconds) (* 60 minutes) (* 360 hours)).

As is, the result of the comparison is almost always dependent on the
number of seconds in the time strings. It is interesting how
frequently this mistaken comparison is correct.

I have attached a patch.

Thank you,

-shawn

Attachment: url-cookie.el.patch
Description: Binary data


--- End Message ---
--- Begin Message --- Subject: Re: bug#6957: url-cookie-expired-p Redux (at last) Date: Thu, 09 Sep 2010 01:39:07 -0400 User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
Version: 23.3

Thank you for the patch; applied.

BTW, something like

(if (not foo) nil bar)

can be written as (and foo bar).


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]