all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* url-cookie.el: A cookie's localpart must not be nil
@ 2006-10-20 12:39 Klaus Straubinger
  0 siblings, 0 replies; only message in thread
From: Klaus Straubinger @ 2006-10-20 12:39 UTC (permalink / raw)


Some internet sites try to set cookies without specifying any local
part (e.g., "http://www.rad-net.de/"). The function
url-cookie-handle-set-cookie then calls url-cookie-store with nil as
its argument LOCALPART. But when this cookie is read again in function
url-cookie-retrieve, an error "Wrong type argument: stringp, nil"
occurs because it is expected to be a string.

My proposal is to make this tiny change:

--- url-cookie.el.orig	2006-09-07 08:49:14.000000000 +0200
+++ url-cookie.el	2006-10-20 14:16:53.000000000 +0200
@@ -361,7 +361,8 @@
 	 (expires (cdr-safe (assoc-ignore-case "expires" args)))
 	 (localpart (or (cdr-safe (assoc-ignore-case "path" args))
 			(file-name-directory
-			 (url-filename url-current-object))))
+			 (url-filename url-current-object))
+			""))
 	 (rest nil))
     (while args
       (if (not (member (downcase (car (car args)))

-- 
Klaus Straubinger

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2006-10-20 12:39 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-20 12:39 url-cookie.el: A cookie's localpart must not be nil Klaus Straubinger

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.