bugGNU Wget - Bugs: bug #59320, wget does not obey timestamping...

 
 

bug #59320: wget does not obey timestamping option when destination file is owned by "nobody"

Submitter:  None
Submitted:  Thu 22 Oct 2020 04:01:38 PM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  5 - Normal Status:  None
Privacy:  Public Assigned to:  None
Originator Name:  Jonathan Bondhus Originator Email:  -email is unavailable-
Open/Closed:  Open Release:  1.19.5
Operating System:  GNU/Linux Reproducibility:  Every Time
Fixed Release:  None Planned Release:  None
Regression:  Yes Work Required:  None
Patch Included:  No
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 22 Oct 2020 04:01:38 PM UTC, original submission:  

This behavior does not occur on wget 1.14 bundled with CentOS 7, however it does occur on wget 1.19.5.

When a destination file already downloaded (same size and timestamp) is owned by "root:wheel", wget honors the timestamping flag and does not re-download. When this same destination file is owned by "nobody:wheel", wget *does not* honor the timestamping flag, and always re-downloads the file, regardless of it's modified date and size. I expected wget to honor the timestamping flag regardless of the owner of the file.

See attached command output from wget 1.14 on CentOS 7 and wget 1.19.5 on CentOS 8.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #50053:  no_clobber.txt added by None (2KiB - text/plain - This seems to also cause "--no-clobber" to fail.)
file #50052:  patch.c added by None (268B - text/plain - It seems that this may be special for the root user, the file in the problem case was owned by "nobody:wheel", and root isn't in the wheel group. This issue seems to pertain to "file_exists_p". Here's a suggested patch, see "getuid() == 0".)
file #50050:  old@behavior.txt added by None (7KiB - text/plain)
file #50051:  new@behavior.txt added by None (8KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by None (Submitted the item)
  •  

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-10-22 None Attached File- Added no_clobber.txt, #50053
    2020-10-22 None Attached File- Added patch.c, #50052
    2020-10-22 None Attached File- Added old@behavior.txt, #50050
        Attached File- Added new@behavior.txt, #50051

    Back to the top

    Powered by Savane 3.13-599c.
    Corresponding source code