all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Brian McKenna <brian@brianmckenna.org>
To: 17785@debbugs.gnu.org
Subject: bug#17785: 24.4.50; EWW: Bales out during form submission
Date: Tue, 21 Oct 2014 20:56:11 -0600	[thread overview]
Message-ID: <CAJ1EMJ4yTz894o4mhWM2BqxbitCbB1UJ64zYxLhFAre9n6QTUg@mail.gmail.com> (raw)
In-Reply-To: <p91r42qqiq6.fsf@lapdog.roam.corp.google.com>

[-- Attachment #1: Type: text/plain, Size: 510 bytes --]

I got problems with the same chunk of code. My problem was that I was
using a form which didn't have a specific value, here's Twitter, for
example:

    <input type="hidden" name="scribe_log">

When trying to do a URL encode of that field, we'd try to construct a
pair but end up with:

    '(scribe_log)

Which would fail at encoding. It should encode to the same as
scribe_log having an empty value, that is "scribe_log=" and so should
be parsed as '(scribe_log "") - I've attached a patch which fixes
that.

[-- Attachment #2: eww-empty-form-value.patch --]
[-- Type: application/octet-stream, Size: 708 bytes --]

*** eww.el	2014-10-21 20:44:56.000000000 -0600
--- eww.el	2014-10-21 20:36:07.000000000 -0600
***************
*** 1001,1007 ****
        (when (and (consp elem)
  		 (eq (car elem) 'hidden))
  	(push (cons (plist-get (cdr elem) :name)
! 		    (plist-get (cdr elem) :value))
  	      values)))
      (if (and (stringp (cdr (assq :method form)))
  	     (equal (downcase (cdr (assq :method form))) "post"))
--- 1001,1007 ----
        (when (and (consp elem)
  		 (eq (car elem) 'hidden))
  	(push (cons (plist-get (cdr elem) :name)
! 		    (or (plist-get (cdr elem) :value) ""))
  	      values)))
      (if (and (stringp (cdr (assq :method form)))
  	     (equal (downcase (cdr (assq :method form))) "post"))

  reply	other threads:[~2014-10-22  2:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-15 15:45 bug#17785: 24.4.50; EWW: Bales out during form submission T.V Raman
2014-10-22  2:56 ` Brian McKenna [this message]
2014-11-02 21:37   ` Ted Zlatanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAJ1EMJ4yTz894o4mhWM2BqxbitCbB1UJ64zYxLhFAre9n6QTUg@mail.gmail.com \
    --to=brian@brianmckenna.org \
    --cc=17785@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.