bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] Conditional GET requests


From: Ander Juaristi
Subject: Re: [Bug-wget] Conditional GET requests
Date: Wed, 13 May 2015 13:28:00 +0200
User-agent: Thunderbird on Linux

Hi Hubert,

A bit of feedback here.

On 05/12/2015 10:05 PM, Hubert Tarasiuk wrote:
I would be very pleased if you could review my code and maybe point out
some mistakes or any other comments/suggestions. Maybe you have ideas
about some improvements to it?


First of all, please correct the typo at gethttp: Ommiting --> Omitting

And second, I'm not really sure whether --condget is the best name for the 
switch.
Requests that include any of If-Unmodified-Since, If-Match, If-None-Match, or 
If-Range
header fields are also "conditional GETs" as well.
We might want to implement one of those in the future and we'd be forced to 
choose a name which could easily be
inconsistent/confusing with --condget. Or maybe we won't. But we don't know 
that now, so I think
it's better to choose a switch more specific to the fact that an 
If-Modified-Since header will be sent
so as to avoid confusion.

--
Regards,
- AJ



reply via email to

[Prev in Thread] Current Thread [Next in Thread]