* bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect
@ 2019-02-21 16:58 Bad Blue Bull
2019-02-22 1:28 ` Bad Blue Bull
2019-02-22 12:59 ` bug#34607: (no subject) Bad Blue Bull
0 siblings, 2 replies; 8+ messages in thread
From: Bad Blue Bull @ 2019-02-21 16:58 UTC (permalink / raw)
To: 34607
[-- Attachment #1: Type: text/html, Size: 880 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect
2019-02-21 16:58 bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect Bad Blue Bull
@ 2019-02-22 1:28 ` Bad Blue Bull
2019-02-22 12:59 ` bug#34607: (no subject) Bad Blue Bull
1 sibling, 0 replies; 8+ messages in thread
From: Bad Blue Bull @ 2019-02-22 1:28 UTC (permalink / raw)
To: 34607
[-- Attachment #1: Type: text/html, Size: 552 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2019-02-21 16:58 bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect Bad Blue Bull
2019-02-22 1:28 ` Bad Blue Bull
@ 2019-02-22 12:59 ` Bad Blue Bull
2019-05-15 4:08 ` Lars Ingebrigtsen
1 sibling, 1 reply; 8+ messages in thread
From: Bad Blue Bull @ 2019-02-22 12:59 UTC (permalink / raw)
To: 34607
[-- Attachment #1: Type: text/html, Size: 231 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2019-02-22 12:59 ` bug#34607: (no subject) Bad Blue Bull
@ 2019-05-15 4:08 ` Lars Ingebrigtsen
2019-05-15 13:27 ` Dmitry Gutov
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-05-15 4:08 UTC (permalink / raw)
To: Bad Blue Bull; +Cc: 34607
Bad Blue Bull <ibmbull@yandex.ru> writes:
> sorry, elisp line intented intended to be
> (with-current-buffer (url-retrieve-synchronously "http://gnu.org" nil t 3) (kill-buffer))
This is surprisingly difficult to fix -- the reason being that URL
library is asynchronously following a redirect, and possibly opening a
new connection, and then marking that new connection with
`process-query-on-exit-flag'. So depending on where you are in that
cycle, the buffer may have a query-enabled process when you try to kill
it.
Which sucks.
On timeouts, url-retrive-synchronously should abort the entire thing,
but url-retrieve (which is used in the background) doesn't really
support that...
Hm.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2019-05-15 4:08 ` Lars Ingebrigtsen
@ 2019-05-15 13:27 ` Dmitry Gutov
2020-07-19 20:42 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Dmitry Gutov @ 2019-05-15 13:27 UTC (permalink / raw)
To: Lars Ingebrigtsen, Bad Blue Bull; +Cc: 34607
On 15.05.2019 7:08, Lars Ingebrigtsen wrote:
> On timeouts, url-retrive-synchronously should abort the entire thing,
> but url-retrieve (which is used in the background) doesn't really
> support that...
I think that's one of the reasons behind url-retrieve-synchronously's
two delete-process calls. They don't exactly seem to work as intended,
though.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2019-05-15 13:27 ` Dmitry Gutov
@ 2020-07-19 20:42 ` Lars Ingebrigtsen
2020-07-19 21:05 ` Dmitry Gutov
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2020-07-19 20:42 UTC (permalink / raw)
To: Dmitry Gutov; +Cc: 34607, Bad Blue Bull
Dmitry Gutov <dgutov@yandex.ru> writes:
> On 15.05.2019 7:08, Lars Ingebrigtsen wrote:
>> On timeouts, url-retrive-synchronously should abort the entire thing,
>> but url-retrieve (which is used in the background) doesn't really
>> support that...
>
> I think that's one of the reasons behind url-retrieve-synchronously's
> two delete-process calls. They don't exactly seem to work as intended,
> though.
Yeah, they don't trigger on timeouts, apparently.
I've now added some more process deletions, which fixes this use case:
(with-current-buffer (url-retrieve-synchronously "http://gnu.org" nil t 0.1)
(kill-buffer))
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2020-07-19 20:42 ` Lars Ingebrigtsen
@ 2020-07-19 21:05 ` Dmitry Gutov
2020-07-19 21:26 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Dmitry Gutov @ 2020-07-19 21:05 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: 34607, Bad Blue Bull
On 19.07.2020 23:42, Lars Ingebrigtsen wrote:
> Yeah, they don't trigger on timeouts, apparently.
Thanks!
> I've now added some more process deletions, which fixes this use case:
>
> (with-current-buffer (url-retrieve-synchronously"http://gnu.org" nil t 0.1)
> (kill-buffer))
That's a neat little way to reproduce the problem. Might be worth it to
add a test like this.
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#34607: (no subject)
2020-07-19 21:05 ` Dmitry Gutov
@ 2020-07-19 21:26 ` Lars Ingebrigtsen
0 siblings, 0 replies; 8+ messages in thread
From: Lars Ingebrigtsen @ 2020-07-19 21:26 UTC (permalink / raw)
To: Dmitry Gutov; +Cc: 34607, Bad Blue Bull
Dmitry Gutov <dgutov@yandex.ru> writes:
>> I've now added some more process deletions, which fixes this use case:
>> (with-current-buffer (url-retrieve-synchronously"http://gnu.org"
>> nil t 0.1)
>> (kill-buffer))
>
> That's a neat little way to reproduce the problem. Might be worth it
> to add a test like this.
For a non-automatic test? I don't think we want to hit that web site
during the normal tests. :-)
But it's brittle in any case -- it's a good test site now because it
uses a redirect, but that may not be the case in the future, so I don't
know how useful it is as a test in the long term...
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-07-19 21:26 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-21 16:58 bug#34607: url-retrieve-synchronously: TIMEOUT option value takes no effect Bad Blue Bull
2019-02-22 1:28 ` Bad Blue Bull
2019-02-22 12:59 ` bug#34607: (no subject) Bad Blue Bull
2019-05-15 4:08 ` Lars Ingebrigtsen
2019-05-15 13:27 ` Dmitry Gutov
2020-07-19 20:42 ` Lars Ingebrigtsen
2020-07-19 21:05 ` Dmitry Gutov
2020-07-19 21:26 ` Lars Ingebrigtsen
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).