unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
@ 2013-07-30  2:29 Erik Hetzner
  2013-07-30  8:50 ` Andreas Schwab
  2019-06-20  1:20 ` Noam Postavsky
  0 siblings, 2 replies; 10+ messages in thread
From: Erik Hetzner @ 2013-07-30  2:29 UTC (permalink / raw)
  To: 14983

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


Using org-caldav-sync, which connects to an https server, I was
consistently getting errors whose root cause was
url-retrieve-synchronously returning an empty buffer. *URL-DEBUG*
looked like:

  ...
  http -> url-http-end-of-document-sentinel in buffer ( *http XXX.example.org:443*)
  http -> Marking connection as free: XXX.example.org:443 #<process XXX.example.org>
  http -> Activating callback in buffer ( *http XXX.example.org:443*)
  retrieval -> Synchronous fetching done (#<buffer  *http XXX.example.org:443*>)

The buffer returned by url-retrieve-synchronously was empty.

It seems that url-http-end-of-document-sentinel does not set the
variable url-gateway-method to 'tls before calling url-http when using
https. Changing url-http-end-of-document-sentinel to set
url-gateway-method to 'tls if the url has scheme https fixes the
issue.

Patch for a proposed fix, which works for me, is attached.

I believe this is also the issue report here:
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9592


[-- Attachment #2: url-http-end-of-document-sentinel_https_fix.patch --]
[-- Type: text/plain, Size: 676 bytes --]

diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
index 7f21a38..4ee9377 100644
--- a/lisp/url/url-http.el
+++ b/lisp/url/url-http.el
@@ -895,6 +895,8 @@ should be shown to the user."
                (let ((url-request-method url-http-method)
                      (url-request-extra-headers url-http-extra-headers)
                      (url-request-data url-http-data))
+                 (if (string= "https" (url-type url-current-object))
+                     (setq url-gateway-method 'tls))
                  (url-http url-current-object url-callback-function
                            url-callback-arguments (current-buffer)))))
 	    ((url-http-parse-headers)

[-- Attachment #3: Type: text/plain, Size: 22002 bytes --]




In GNU Emacs 24.3.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.6.4)
 of 2013-07-25 on menkalinan, modified by Debian
 (emacs-snapshot package, version 2:20130724-1~ppa1~raring1)
Windowing system distributor `The X.Org Foundation', version 11.0.11303000
System Description:	Ubuntu 13.04

Configured using:
 `configure --build x86_64-linux-gnu --host x86_64-linux-gnu
 --prefix=/usr --sharedstatedir=/var/lib --libexecdir=/usr/lib
 --localstatedir=/var --infodir=/usr/share/info --mandir=/usr/share/man
 --with-pop=yes
 --enable-locallisppath=/etc/emacs-snapshot:/etc/emacs:/usr/local/share/emacs/24.3.50/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/24.3.50/site-lisp:/usr/share/emacs/site-lisp
 --without-compress-info --with-crt-dir=/usr/lib/x86_64-linux-gnu/
 --with-x=yes --with-x-toolkit=gtk3 --with-imagemagick=yes
 CFLAGS='-DDEBIAN -DSITELOAD_PURESIZE_EXTRA=5000 -g -O2'
 CPPFLAGS='-D_FORTIFY_SOURCE=2' LDFLAGS='-g -Wl,--as-needed
 -znocombreloc''

Important settings:
  value of $LC_TIME: en_US.UTF-8
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix
  default enable-multibyte-characters: t

Major mode: Fundamental

Minor modes in effect:
  shell-dirtrack-mode: t
  iswitchb-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
<return> C-x C-f C-g C-x C-f . a u <tab> t h <tab> 
C-j M-> C-y C-x C-s M-x o r g - c a l d <tab> <return> 
s <tab> <return> <up> M-x r e <backspace> <backspace> 
e m <tab> a <tab> <tab> <M-backspace> r e p o r <tab> 
<return>

Recent messages:
Speck-mode turned off [2 times]
Using dictionary "en_US" and filter "URL" [2 times]
Speck-mode turned off [4 times]
Using dictionary "en_US" and filter "URL" [55 times]
Speck-mode turned off [31 times]
55 unique files scanned for IDs
Using dictionary "en_US" and filter "URL"
Contacting host: XXX.example.org:443
url-dav-request: Wrong type argument: integer-or-marker-p, nil
Making completion list... [2 times]

Load-path shadows:
/home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/utils/ssl hides /home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/site-lisp/wl/ssl
/home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/utils/wl-complete hides /home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/site-lisp/wl/wl-complete
/home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/utils/wl-mailto hides /home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/site-lisp/wl/wl-mailto
/home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/utils/im-wl hides /home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/site-lisp/wl/im-wl
/home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/utils/wl-addrbook hides /home/egh/.emacs.d/el-get-24.3.50.1/wanderlust/site-lisp/wl/wl-addrbook
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/.dir-locals hides /home/egh/.emacs.d/el-get-24.3.50.1/magit/.dir-locals
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/.dir-locals hides /home/egh/.emacs.d/el-get-24.3.50.1/el-get/.dir-locals
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3mhack hides /usr/share/emacs-snapshot/site-lisp/w3m/w3mhack
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-hist hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-hist
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/mime-w3m hides /usr/share/emacs-snapshot/site-lisp/w3m/mime-w3m
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-antenna hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-antenna
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-bookmark hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-bookmark
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-favicon hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-favicon
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-form hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-form
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-proc hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-proc
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-filter hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-filter
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/octet hides /usr/share/emacs-snapshot/site-lisp/w3m/octet
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/mew-w3m hides /usr/share/emacs-snapshot/site-lisp/w3m/mew-w3m
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-perldoc hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-perldoc
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-cookie hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-cookie
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-bug hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-bug
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-ccl hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-ccl
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-symbol hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-symbol
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-namazu hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-namazu
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-image hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-image
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-util hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-util
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-weather hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-weather
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-search hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-search
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-load hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-load
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-rss hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-rss
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-tabmenu hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-tabmenu
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-dtree hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-dtree
/home/egh/.emacs.d/el-get-24.3.50.1/emacs-w3m/w3m-lnum hides /usr/share/emacs-snapshot/site-lisp/w3m/w3m-lnum
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/contrib/lisp/htmlize hides /usr/share/emacs-snapshot/site-lisp/emacs-goodies-el/htmlize
/usr/share/emacs/24.3.50/site-lisp/cmake-data/cmake-mode hides /usr/share/emacs/site-lisp/cmake-mode
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/md4 hides /usr/share/emacs/24.3.50/lisp/md4
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/hex-util hides /usr/share/emacs/24.3.50/lisp/hex-util
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-plantuml hides /usr/share/emacs/24.3.50/lisp/org/ob-plantuml
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-bibtex hides /usr/share/emacs/24.3.50/lisp/org/org-bibtex
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-tangle hides /usr/share/emacs/24.3.50/lisp/org/ob-tangle
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-compat hides /usr/share/emacs/24.3.50/lisp/org/org-compat
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-gnus hides /usr/share/emacs/24.3.50/lisp/org/org-gnus
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-keys hides /usr/share/emacs/24.3.50/lisp/org/ob-keys
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/contrib/lisp/org-vm hides /usr/share/emacs/24.3.50/lisp/org/org-vm
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-loaddefs hides /usr/share/emacs/24.3.50/lisp/org/org-loaddefs
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-exp hides /usr/share/emacs/24.3.50/lisp/org/ob-exp
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-mobile hides /usr/share/emacs/24.3.50/lisp/org/org-mobile
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-timer hides /usr/share/emacs/24.3.50/lisp/org/org-timer
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-sass hides /usr/share/emacs/24.3.50/lisp/org/ob-sass
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-faces hides /usr/share/emacs/24.3.50/lisp/org/org-faces
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-C hides /usr/share/emacs/24.3.50/lisp/org/ob-C
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-version hides /usr/share/emacs/24.3.50/lisp/org/org-version
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-protocol hides /usr/share/emacs/24.3.50/lisp/org/org-protocol
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-indent hides /usr/share/emacs/24.3.50/lisp/org/org-indent
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-clojure hides /usr/share/emacs/24.3.50/lisp/org/ob-clojure
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-python hides /usr/share/emacs/24.3.50/lisp/org/ob-python
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-footnote hides /usr/share/emacs/24.3.50/lisp/org/org-footnote
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-fortran hides /usr/share/emacs/24.3.50/lisp/org/ob-fortran
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-docview hides /usr/share/emacs/24.3.50/lisp/org/org-docview
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-table hides /usr/share/emacs/24.3.50/lisp/org/ob-table
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-ref hides /usr/share/emacs/24.3.50/lisp/org/ob-ref
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-ctags hides /usr/share/emacs/24.3.50/lisp/org/org-ctags
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-scala hides /usr/share/emacs/24.3.50/lisp/org/ob-scala
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-matlab hides /usr/share/emacs/24.3.50/lisp/org/ob-matlab
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-irc hides /usr/share/emacs/24.3.50/lisp/org/org-irc
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-perl hides /usr/share/emacs/24.3.50/lisp/org/ob-perl
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-install hides /usr/share/emacs/24.3.50/lisp/org/org-install
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-element hides /usr/share/emacs/24.3.50/lisp/org/org-element
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-sh hides /usr/share/emacs/24.3.50/lisp/org/ob-sh
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-sql hides /usr/share/emacs/24.3.50/lisp/org/ob-sql
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-entities hides /usr/share/emacs/24.3.50/lisp/org/org-entities
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-list hides /usr/share/emacs/24.3.50/lisp/org/org-list
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-pcomplete hides /usr/share/emacs/24.3.50/lisp/org/org-pcomplete
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-w3m hides /usr/share/emacs/24.3.50/lisp/org/org-w3m
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-sqlite hides /usr/share/emacs/24.3.50/lisp/org/ob-sqlite
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-lilypond hides /usr/share/emacs/24.3.50/lisp/org/ob-lilypond
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-ruby hides /usr/share/emacs/24.3.50/lisp/org/ob-ruby
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/contrib/lisp/org-mew hides /usr/share/emacs/24.3.50/lisp/org/org-mew
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-R hides /usr/share/emacs/24.3.50/lisp/org/ob-R
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-macs hides /usr/share/emacs/24.3.50/lisp/org/org-macs
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-eshell hides /usr/share/emacs/24.3.50/lisp/org/org-eshell
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-java hides /usr/share/emacs/24.3.50/lisp/org/ob-java
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-bbdb hides /usr/share/emacs/24.3.50/lisp/org/org-bbdb
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-scheme hides /usr/share/emacs/24.3.50/lisp/org/ob-scheme
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-plot hides /usr/share/emacs/24.3.50/lisp/org/org-plot
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-lob hides /usr/share/emacs/24.3.50/lisp/org/ob-lob
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-latex hides /usr/share/emacs/24.3.50/lisp/org/ob-latex
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-ledger hides /usr/share/emacs/24.3.50/lisp/org/ob-ledger
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-mouse hides /usr/share/emacs/24.3.50/lisp/org/org-mouse
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-crypt hides /usr/share/emacs/24.3.50/lisp/org/org-crypt
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-clock hides /usr/share/emacs/24.3.50/lisp/org/org-clock
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-ditaa hides /usr/share/emacs/24.3.50/lisp/org/ob-ditaa
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-js hides /usr/share/emacs/24.3.50/lisp/org/ob-js
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-archive hides /usr/share/emacs/24.3.50/lisp/org/org-archive
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-colview hides /usr/share/emacs/24.3.50/lisp/org/org-colview
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-feed hides /usr/share/emacs/24.3.50/lisp/org/org-feed
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-io hides /usr/share/emacs/24.3.50/lisp/org/ob-io
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-octave hides /usr/share/emacs/24.3.50/lisp/org/ob-octave
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-emacs-lisp hides /usr/share/emacs/24.3.50/lisp/org/ob-emacs-lisp
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-awk hides /usr/share/emacs/24.3.50/lisp/org/ob-awk
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-datetree hides /usr/share/emacs/24.3.50/lisp/org/org-datetree
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org hides /usr/share/emacs/24.3.50/lisp/org/org
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-mscgen hides /usr/share/emacs/24.3.50/lisp/org/ob-mscgen
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob hides /usr/share/emacs/24.3.50/lisp/org/ob
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-css hides /usr/share/emacs/24.3.50/lisp/org/ob-css
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-rmail hides /usr/share/emacs/24.3.50/lisp/org/org-rmail
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-haskell hides /usr/share/emacs/24.3.50/lisp/org/ob-haskell
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-mhe hides /usr/share/emacs/24.3.50/lisp/org/org-mhe
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/contrib/lisp/org-mac-message hides /usr/share/emacs/24.3.50/lisp/org/org-mac-message
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-ocaml hides /usr/share/emacs/24.3.50/lisp/org/ob-ocaml
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-src hides /usr/share/emacs/24.3.50/lisp/org/org-src
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-gnuplot hides /usr/share/emacs/24.3.50/lisp/org/ob-gnuplot
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-capture hides /usr/share/emacs/24.3.50/lisp/org/org-capture
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-attach hides /usr/share/emacs/24.3.50/lisp/org/org-attach
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-shen hides /usr/share/emacs/24.3.50/lisp/org/ob-shen
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-asymptote hides /usr/share/emacs/24.3.50/lisp/org/ob-asymptote
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-inlinetask hides /usr/share/emacs/24.3.50/lisp/org/org-inlinetask
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-maxima hides /usr/share/emacs/24.3.50/lisp/org/ob-maxima
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-picolisp hides /usr/share/emacs/24.3.50/lisp/org/ob-picolisp
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-org hides /usr/share/emacs/24.3.50/lisp/org/ob-org
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-id hides /usr/share/emacs/24.3.50/lisp/org/org-id
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-lisp hides /usr/share/emacs/24.3.50/lisp/org/ob-lisp
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/contrib/lisp/org-wl hides /usr/share/emacs/24.3.50/lisp/org/org-wl
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-comint hides /usr/share/emacs/24.3.50/lisp/org/ob-comint
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-habit hides /usr/share/emacs/24.3.50/lisp/org/org-habit
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-dot hides /usr/share/emacs/24.3.50/lisp/org/ob-dot
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-calc hides /usr/share/emacs/24.3.50/lisp/org/ob-calc
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-eval hides /usr/share/emacs/24.3.50/lisp/org/ob-eval
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-agenda hides /usr/share/emacs/24.3.50/lisp/org/org-agenda
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-info hides /usr/share/emacs/24.3.50/lisp/org/org-info
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/org-table hides /usr/share/emacs/24.3.50/lisp/org/org-table
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/lisp/ob-screen hides /usr/share/emacs/24.3.50/lisp/org/ob-screen
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/sasl-ntlm hides /usr/share/emacs/24.3.50/lisp/net/sasl-ntlm
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/hmac-def hides /usr/share/emacs/24.3.50/lisp/net/hmac-def
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/sasl hides /usr/share/emacs/24.3.50/lisp/net/sasl
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/sasl-cram hides /usr/share/emacs/24.3.50/lisp/net/sasl-cram
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/hmac-md5 hides /usr/share/emacs/24.3.50/lisp/net/hmac-md5
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/sasl-digest hides /usr/share/emacs/24.3.50/lisp/net/sasl-digest
/home/egh/.emacs.d/el-get-24.3.50.1/flim/site-lisp/flim/ntlm hides /usr/share/emacs/24.3.50/lisp/net/ntlm
/home/egh/.emacs.d/el-get-24.3.50.1/org-mode/.dir-locals hides /usr/share/emacs/24.3.50/lisp/gnus/.dir-locals
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/smime hides /usr/share/emacs/24.3.50/lisp/gnus/smime
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg-parse hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg-parse
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg-def hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg-def
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg-gpg hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg-gpg
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg-pgp5 hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg-pgp5
/home/egh/.emacs.d/el-get-24.3.50.1/semi/site-lisp/semi/pgg-pgp hides /usr/share/emacs/24.3.50/lisp/obsolete/pgg-pgp

Features:
(shadow sort mail-extr emacsbug vc-git autorevert filenotify speck ox-md
ox-confluence org-wl org-w3m org-timer org-mu4e org-irc org-jsinfo
org-infojs org-html org-exp org-exp-blocks org-info org-habit
org-docview org-clock org-archive org-agenda gnutls network-stream
starttls org-caldav icalendar diary-lib diary-loaddefs org-id ox-latex
ox-icalendar ox-html ox-ascii ox-publish ox url-dav url-http tls
url-auth url-gw url-handlers windmove tramp tramp-compat tramp-loaddefs
trampver shell quail zenburn-theme iswitchb edmacro kmacro .loaddefs
twittering-mode scala-mode2 scala-mode2-map scala-mode2-fontlock
scala-mode2-indent scala-mode2-paragraph scala-mode2-syntax
scala-mode2-lib org-element org warnings advice org-macro org-footnote
org-pcomplete pcomplete org-list org-faces org-entities noutline outline
org-version ob-emacs-lisp ob ob-tangle org-src ob-ref ob-lob ob-table
ob-keys ob-exp ob-comint ob-core ob-eval org-compat org-macs
org-loaddefs find-func cal-menu calendar cal-loaddefs monky bindat
google-contacts-message google-contacts xml url-cache url url-proxy
url-privacy url-expand url-methods url-history url-cookie url-domsuf
mailcap url-util url-parse url-vars google-oauth mu4e mu4e-speedbar
speedbar sb-image ezimage dframe mu4e-main mu4e-view epa derived epg
epg-config browse-url comint ansi-color ring mu4e-headers mu4e-compose
mu4e-draft mu4e-actions rfc2368 smtpmail auth-source eieio eieio-core
gnus-util password-cache sendmail mu4e-mark mu4e-message html2text
mu4e-proc mu4e-utils doc-view jka-compr image-mode mu4e-lists mu4e-about
mu4e-vars message byte-opt format-spec rfc822 mml mml-sec mm-decode
mm-bodies mm-encode mail-parse rfc2231 rfc2047 rfc2045 ietf-drums
mm-util mail-prsvr mailabbrev mail-utils gmm-utils mailheader hl-line
mu4e-meta el-get cl-macs gv el-get-autoloads el-get-list-packages
el-get-notify help-mode easymenu el-get-dependencies el-get-build
el-get-status pp el-get-recipes el-get-byte-compile el-get-methods
el-get-fossil el-get-svn el-get-pacman el-get-github-zip
el-get-github-tar el-get-http-zip el-get-http-tar el-get-hg
el-get-git-svn el-get-fink el-get-emacswiki el-get-http
el-get-emacsmirror el-get-github el-get-git el-get-elpa el-get-darcs
el-get-cvs el-get-bzr el-get-brew el-get-builtin el-get-apt-get
el-get-custom el-get-core autoload help-fns lisp-mnt bytecomp
byte-compile cconv cl nadvice cl-loaddefs cl-lib dired package w3m-load
emacs-goodies-el emacs-goodies-custom emacs-goodies-loaddefs easy-mmode
time-date tooltip ediff-hook vc-hooks lisp-float-type mwheel x-win x-dnd
tool-bar dnd fontset image regexp-opt fringe tabulated-list newcomment
lisp-mode prog-mode register page menu-bar rfn-eshadow timer select
scroll-bar mouse jit-lock font-lock syntax facemenu font-core frame cham
georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese hebrew greek romanian slovak czech european ethiopic
indian cyrillic chinese case-table epa-hook jka-cmpr-hook help simple
abbrev minibuffer loaddefs button faces cus-face macroexp files
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote make-network-process
dbusbind gfilenotify dynamic-setting system-font-setting
font-render-setting move-toolbar gtk x-toolkit x multi-tty emacs)

-- 
Sent from my free software system <http://fsf.org/>.

[-- Attachment #4: Type: text/plain, Size: 53 bytes --]

Sent from my free software system <http://fsf.org/>.

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30  2:29 bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https Erik Hetzner
@ 2013-07-30  8:50 ` Andreas Schwab
  2013-07-30 14:01   ` Erik Hetzner
                     ` (2 more replies)
  2019-06-20  1:20 ` Noam Postavsky
  1 sibling, 3 replies; 10+ messages in thread
From: Andreas Schwab @ 2013-07-30  8:50 UTC (permalink / raw)
  To: Erik Hetzner; +Cc: 14983

Erik Hetzner <egh@e6h.org> writes:

> diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
> index 7f21a38..4ee9377 100644
> --- a/lisp/url/url-http.el
> +++ b/lisp/url/url-http.el
> @@ -895,6 +895,8 @@ should be shown to the user."
>                 (let ((url-request-method url-http-method)
>                       (url-request-extra-headers url-http-extra-headers)
>                       (url-request-data url-http-data))
> +                 (if (string= "https" (url-type url-current-object))
> +                     (setq url-gateway-method 'tls))
>                   (url-http url-current-object url-callback-function
>                             url-callback-arguments (current-buffer)))))

I think this should use url-https instead for https connections.

Andreas.

-- 
Andreas Schwab, SUSE Labs, schwab@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
"And now for something completely different."





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30  8:50 ` Andreas Schwab
@ 2013-07-30 14:01   ` Erik Hetzner
  2013-07-30 14:20   ` Erik Hetzner
  2013-07-30 14:20   ` Erik Hetzner
  2 siblings, 0 replies; 10+ messages in thread
From: Erik Hetzner @ 2013-07-30 14:01 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 14983

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

At Tue, 30 Jul 2013 10:50:56 +0200,
Andreas Schwab wrote:
> 
> Erik Hetzner <egh@e6h.org> writes:
> 
> > diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
> > index 7f21a38..4ee9377 100644
> > --- a/lisp/url/url-http.el
> > +++ b/lisp/url/url-http.el
> > @@ -895,6 +895,8 @@ should be shown to the user."
> >                 (let ((url-request-method url-http-method)
> >                       (url-request-extra-headers url-http-extra-headers)
> >                       (url-request-data url-http-data))
> > +                 (if (string= "https" (url-type url-current-object))
> > +                     (setq url-gateway-method 'tls))
> >                   (url-http url-current-object url-callback-function
> >                             url-callback-arguments (current-buffer)))))
> 
> I think this should use url-https instead for https connections.
> 
> Andreas.

Hi Andreas,

That would be a better solution, but url-https does not currently take
a retry-buffer argument:

(url-http URL CALLBACK CBARGS &optional RETRY-BUFFER)
(url-https URL CALLBACK CBARGS)

Maybe it should be modified to pass through an optional retry-buffer?

best, Erik

[-- Attachment #2: Type: text/plain, Size: 53 bytes --]

Sent from my free software system <http://fsf.org/>.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30  8:50 ` Andreas Schwab
  2013-07-30 14:01   ` Erik Hetzner
@ 2013-07-30 14:20   ` Erik Hetzner
  2015-12-25 21:03     ` Lars Ingebrigtsen
  2019-09-24  7:58     ` Lars Ingebrigtsen
  2013-07-30 14:20   ` Erik Hetzner
  2 siblings, 2 replies; 10+ messages in thread
From: Erik Hetzner @ 2013-07-30 14:20 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 14983

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

At Tue, 30 Jul 2013 10:50:56 +0200,
Andreas Schwab wrote:
> 
> Erik Hetzner <egh@e6h.org> writes:
> 
> […]
> 
> I think this should use url-https instead for https connections.

This patch modifies url-https to support an optional retry-buffer
argument, making this better approach possible.


[-- Attachment #2: url-http-end-of-document-sentinel_https_fix_2.patch --]
[-- Type: text/plain, Size: 1420 bytes --]

diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
index 7f21a38..868b6ff 100644
--- a/lisp/url/url-http.el
+++ b/lisp/url/url-http.el
@@ -895,8 +895,11 @@ should be shown to the user."
                (let ((url-request-method url-http-method)
                      (url-request-extra-headers url-http-extra-headers)
                      (url-request-data url-http-data))
-                 (url-http url-current-object url-callback-function
-                           url-callback-arguments (current-buffer)))))
+                 (if (string= "https" (url-type url-current-object))
+                     (url-https url-current-object url-callback-function
+                                url-callback-arguments (current-buffer))
+                   (url-http url-current-object url-callback-function
+                             url-callback-arguments (current-buffer))))))
 	    ((url-http-parse-headers)
 	     (url-http-activate-callback))))))
 
@@ -1432,7 +1435,7 @@ p3p
        ,@(remove '&rest (remove '&optional args))))))
 
 ;;;###autoload (autoload 'url-https "url-http")
-(url-https-create-secure-wrapper nil (url callback cbargs))
+(url-https-create-secure-wrapper nil (url callback cbargs &optional retry-buffer))
 ;;;###autoload (autoload 'url-https-file-exists-p "url-http")
 (url-https-create-secure-wrapper file-exists-p (url))
 ;;;###autoload (autoload 'url-https-file-readable-p "url-http")

[-- Attachment #3: Type: text/plain, Size: 12 bytes --]


best, Erik

[-- Attachment #4: Type: text/plain, Size: 53 bytes --]

Sent from my free software system <http://fsf.org/>.

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30  8:50 ` Andreas Schwab
  2013-07-30 14:01   ` Erik Hetzner
  2013-07-30 14:20   ` Erik Hetzner
@ 2013-07-30 14:20   ` Erik Hetzner
  2019-06-17  2:58     ` Noam Postavsky
  2 siblings, 1 reply; 10+ messages in thread
From: Erik Hetzner @ 2013-07-30 14:20 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 14983

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

At Tue, 30 Jul 2013 10:50:56 +0200,
Andreas Schwab wrote:
> 
> Erik Hetzner <egh@e6h.org> writes:
> 
> […]
> 
> I think this should use url-https instead for https connections.

This patch modifies url-https to support an optional retry-buffer
argument, making this better approach possible.


[-- Attachment #2: url-http-end-of-document-sentinel_https_fix_2.patch --]
[-- Type: text/plain, Size: 1420 bytes --]

diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
index 7f21a38..868b6ff 100644
--- a/lisp/url/url-http.el
+++ b/lisp/url/url-http.el
@@ -895,8 +895,11 @@ should be shown to the user."
                (let ((url-request-method url-http-method)
                      (url-request-extra-headers url-http-extra-headers)
                      (url-request-data url-http-data))
-                 (url-http url-current-object url-callback-function
-                           url-callback-arguments (current-buffer)))))
+                 (if (string= "https" (url-type url-current-object))
+                     (url-https url-current-object url-callback-function
+                                url-callback-arguments (current-buffer))
+                   (url-http url-current-object url-callback-function
+                             url-callback-arguments (current-buffer))))))
 	    ((url-http-parse-headers)
 	     (url-http-activate-callback))))))
 
@@ -1432,7 +1435,7 @@ p3p
        ,@(remove '&rest (remove '&optional args))))))
 
 ;;;###autoload (autoload 'url-https "url-http")
-(url-https-create-secure-wrapper nil (url callback cbargs))
+(url-https-create-secure-wrapper nil (url callback cbargs &optional retry-buffer))
 ;;;###autoload (autoload 'url-https-file-exists-p "url-http")
 (url-https-create-secure-wrapper file-exists-p (url))
 ;;;###autoload (autoload 'url-https-file-readable-p "url-http")

[-- Attachment #3: Type: text/plain, Size: 12 bytes --]


best, Erik

[-- Attachment #4: Type: text/plain, Size: 53 bytes --]

Sent from my free software system <http://fsf.org/>.

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30 14:20   ` Erik Hetzner
@ 2015-12-25 21:03     ` Lars Ingebrigtsen
  2019-05-20 18:56       ` Noam Postavsky
  2019-09-24  7:58     ` Lars Ingebrigtsen
  1 sibling, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2015-12-25 21:03 UTC (permalink / raw)
  To: Erik Hetzner; +Cc: 14983, Andreas Schwab

Erik Hetzner <erik.hetzner@ucop.edu> writes:

> This patch modifies url-https to support an optional retry-buffer
> argument, making this better approach possible.
>
> diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
> index 7f21a38..868b6ff 100644
> --- a/lisp/url/url-http.el
> +++ b/lisp/url/url-http.el
> @@ -895,8 +895,11 @@ should be shown to the user."
>                 (let ((url-request-method url-http-method)
>                       (url-request-extra-headers url-http-extra-headers)
>                       (url-request-data url-http-data))
> -                 (url-http url-current-object url-callback-function
> -                           url-callback-arguments (current-buffer)))))
> +                 (if (string= "https" (url-type url-current-object))
> +                     (url-https url-current-object url-callback-function
> +                                url-callback-arguments (current-buffer))
> +                   (url-http url-current-object url-callback-function
> +                             url-callback-arguments (current-buffer))))))
>  	    ((url-http-parse-headers)
>  	     (url-http-activate-callback))))))
>  
> @@ -1432,7 +1435,7 @@ p3p
>         ,@(remove '&rest (remove '&optional args))))))
>  
>  ;;;###autoload (autoload 'url-https "url-http")
> -(url-https-create-secure-wrapper nil (url callback cbargs))
> +(url-https-create-secure-wrapper nil (url callback cbargs &optional retry-buffer))

I think this looks sensible, but the patch it two years old, so I wonder
whether there was a reason it wasn't applied back then?  Or whether this
was fixed a different way?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2015-12-25 21:03     ` Lars Ingebrigtsen
@ 2019-05-20 18:56       ` Noam Postavsky
  0 siblings, 0 replies; 10+ messages in thread
From: Noam Postavsky @ 2019-05-20 18:56 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 14983, Andreas Schwab, Erik Hetzner

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Erik Hetzner <erik.hetzner@ucop.edu> writes:
>
>> This patch modifies url-https to support an optional retry-buffer
>> argument, making this better approach possible.
>>
>> diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
>> index 7f21a38..868b6ff 100644
>> --- a/lisp/url/url-http.el
>> +++ b/lisp/url/url-http.el
>> @@ -895,8 +895,11 @@ should be shown to the user."
>>                 (let ((url-request-method url-http-method)
>>                       (url-request-extra-headers url-http-extra-headers)
>>                       (url-request-data url-http-data))
>> -                 (url-http url-current-object url-callback-function
>> -                           url-callback-arguments (current-buffer)))))
>> +                 (if (string= "https" (url-type url-current-object))
>> +                     (url-https url-current-object url-callback-function
>> +                                url-callback-arguments (current-buffer))
>> +                   (url-http url-current-object url-callback-function
>> +                             url-callback-arguments (current-buffer))))))
>>  	    ((url-http-parse-headers)
>>  	     (url-http-activate-callback))))))
>>  
>> @@ -1432,7 +1435,7 @@ p3p
>>         ,@(remove '&rest (remove '&optional args))))))
>>  
>>  ;;;###autoload (autoload 'url-https "url-http")
>> -(url-https-create-secure-wrapper nil (url callback cbargs))
>> +(url-https-create-secure-wrapper nil (url callback cbargs &optional retry-buffer))
>
> I think this looks sensible, but the patch it two years old, so I wonder
> whether there was a reason it wasn't applied back then?  Or whether this
> was fixed a different way?

As far as I can tell, there's nothing wrong with the patch, and the
problem wasn't fixed in some other way.

Minor question on the copyright situation, it looks like Erik Hetzner is
on the list, but the email is different and there are some other middle
names.  Is it the same person? (the patch is small enough to apply
regardless though)






^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30 14:20   ` Erik Hetzner
@ 2019-06-17  2:58     ` Noam Postavsky
  0 siblings, 0 replies; 10+ messages in thread
From: Noam Postavsky @ 2019-06-17  2:58 UTC (permalink / raw)
  To: Erik Hetzner; +Cc: 14983, Andreas Schwab

Erik Hetzner <egh@e6h.org> writes:

Oh, the .ucop.edu address bounced, but maybe this one is good?

> This patch modifies url-https to support an optional retry-buffer
> argument, making this better approach possible.
>
>
>
> best, Erik
>
> Sent from my free software system <http://fsf.org/>.





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30  2:29 bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https Erik Hetzner
  2013-07-30  8:50 ` Andreas Schwab
@ 2019-06-20  1:20 ` Noam Postavsky
  1 sibling, 0 replies; 10+ messages in thread
From: Noam Postavsky @ 2019-06-20  1:20 UTC (permalink / raw)
  To: 14983; +Cc: Erik Hetzner

[bring discussion back to the list]

> From: Erik Hetzner <egh@e6h.org>
>
> I’m also not sure if this is still an issue that needs to be addressed
> with a patch. I haven’t seen this issue recently.

Hmm, well the mismatch with http and https functions is still there.
The comments talk about some particular circumstances when
url-http-end-of-document-sentinel is called:

    (defun url-http-end-of-document-sentinel (proc why)
      ;; Sentinel used to handle (i) terminated old HTTP/0.9 connections,
      ;; and (ii) closed connection due to reusing a HTTP connection which
      ;; we believed was still alive, but which the server closed on us.
      ;; We handle case (ii) by calling `url-http' again.

Perhaps due to changes in servers and clients neither of those
conditions happens for you any longer.







^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https
  2013-07-30 14:20   ` Erik Hetzner
  2015-12-25 21:03     ` Lars Ingebrigtsen
@ 2019-09-24  7:58     ` Lars Ingebrigtsen
  1 sibling, 0 replies; 10+ messages in thread
From: Lars Ingebrigtsen @ 2019-09-24  7:58 UTC (permalink / raw)
  To: Erik Hetzner; +Cc: 14983, Andreas Schwab

Erik Hetzner <erik.hetzner@ucop.edu> writes:

> -                 (url-http url-current-object url-callback-function
> -                           url-callback-arguments (current-buffer)))))
> +                 (if (string= "https" (url-type url-current-object))
> +                     (url-https url-current-object url-callback-function
> +                                url-callback-arguments (current-buffer))
> +                   (url-http url-current-object url-callback-function
> +                             url-callback-arguments (current-buffer))))))

The code has changed in the meantime, so I applied the following patch
instead:

diff --git a/lisp/url/url-http.el b/lisp/url/url-http.el
index bfc106c2a5..a6963cef70 100644
--- a/lisp/url/url-http.el
+++ b/lisp/url/url-http.el
@@ -1024,7 +1024,9 @@ url-http-end-of-document-sentinel
                    (setq url-using-proxy
                          (url-generic-parse-url url-using-proxy)))
                  (url-http url-current-object url-callback-function
-                           url-callback-arguments (current-buffer)))))
+                           url-callback-arguments (current-buffer)
+                           (and (string= "https" (url-type url-current-object))
+                                'tls)))))
 	    ((url-http-parse-headers)
 	     (url-http-activate-callback))))))
 


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply related	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2019-09-24  7:58 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-30  2:29 bug#14983: 24.3.50; url-http-end-of-document-sentinel does not work with https Erik Hetzner
2013-07-30  8:50 ` Andreas Schwab
2013-07-30 14:01   ` Erik Hetzner
2013-07-30 14:20   ` Erik Hetzner
2015-12-25 21:03     ` Lars Ingebrigtsen
2019-05-20 18:56       ` Noam Postavsky
2019-09-24  7:58     ` Lars Ingebrigtsen
2013-07-30 14:20   ` Erik Hetzner
2019-06-17  2:58     ` Noam Postavsky
2019-06-20  1:20 ` Noam Postavsky

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).