Patch 2187: in fetchFileLazyPS also check for URI scheme "https:" - Darcs bug tracker

Title	in fetchFileLazyPS also check for URI scheme "https:"
Superseder		Nosy List	bfrk
Related Issues
Status	accepted	Assigned To
Milestone

Created on 2021-06-05.07:33:40 by bfrk, last changed 2021-06-08.05:25:52 by ganesh.

Files
File name	Status	Uploaded	Type	Edit	Remove
in-fetchfilelazyps-also-check-for-uri-scheme-_https__.dpatch		bfrk, 2021-06-05.07:33:38	application/x-darcs-patch
patch-preview.txt		bfrk, 2021-06-05.07:33:38	text/x-darcs-patch
unnamed		bfrk, 2021-06-05.07:33:38	text/plain

See mailing list archives for discussion on individual patches.

Messages
msg22847 (view)	Author: bfrk	Date: 2021-06-05.07:33:38
This is only used when fetching packs. The fallback method (download strictly to a temporary file, then read lazily) is somewhat prone to race conditions but that's a different issue. As I wrote in a comment to copyRemoteLazy, the consumer should be re-written to use proper streaming. 1 patch for repository http://darcs.net/screened: patch 59731ef16f96daa706e70c905d7a714df6993ab2 Author: Ben Franksen <ben.franksen@online.de> Date: Tue May 25 14:54:16 CEST 2021 * in fetchFileLazyPS also check for URI scheme "https:" Attachments in-fetchfilelazyps-also-check-for-uri-scheme-_https__.dpatch patch-preview.txt unnamed
msg22856 (view)	Author: ganesh	Date: 2021-06-05.09:24:36
Looks good.

History
Date	User	Action	Args
2021-06-05 07:33:40	bfrk	create
2021-06-05 07:35:41	bfrk	set	status: needs-screening -> needs-review
2021-06-05 09:24:36	ganesh	set	status: needs-review -> accepted-pending-tests messages: + msg22856
2021-06-08 05:25:52	ganesh	set	status: accepted-pending-tests -> accepted