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"))
next prev parent 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.