diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-24 14:40:03 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-24 14:40:03 -0500 |
| commit | bab5bb8aa8cadc0cf524e8638e0f9400ff5dd23b (patch) | |
| tree | 3de3800401edf06afab3ad962d678908ad735547 /tests/test-system-utils-eval-buffer.el | |
| parent | 2382ec7841920e2bef55f75c98fbc98085134a8d (diff) | |
| download | dotemacs-bab5bb8aa8cadc0cf524e8638e0f9400ff5dd23b.tar.gz dotemacs-bab5bb8aa8cadc0cf524e8638e0f9400ff5dd23b.zip | |
fix(elfeed): bound and clean up the synchronous YouTube fetch
cj/youtube-to-elfeed-feed-format called url-retrieve-synchronously with no timeout, so a hung YouTube request would block Emacs indefinitely, and it only killed the temporary URL buffer when an ID was successfully extracted — a page without the expected markers leaked the buffer.
Passed cj/elfeed-url-fetch-timeout (10s) to the synchronous fetch, and moved the fetch+parse into an unwind-protect that always kills the temp buffer (live-p guarded), including the parse-failure path. Tests mock the network boundary and cover a normal channel parse, that a timeout is passed, and that the buffer is not leaked when parsing fails.
Also added tests for the EWW user-agent advice (no code change): it already injects the desktop UA only from eww-mode buffers, so package.el and other non-EWW url callers pass through untouched — the tests pin that scoping and the replace-not-duplicate header behavior.
Diffstat (limited to 'tests/test-system-utils-eval-buffer.el')
0 files changed, 0 insertions, 0 deletions
