bugGNU Wget - Bugs: bug #56755, Saved cookies file doesn't pass...

 
 

bug #56755: Saved cookies file doesn't pass cookielib's valid cookie format test

Submitter:  None
Submitted:  Wed 14 Aug 2019 05:35:40 AM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Status:  Wont Fix
Privacy:  Public Assigned to:  rockdaboot
Originator Name:  Ashley Harvey Originator Email:  -email is unavailable-
Open/Closed:  Closed Release:  1.20
Operating System:  None Reproducibility:  Every Time
Fixed Release:  None Planned Release:  None
Regression:  None Work Required:  None
Patch Included:  No
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 14 Aug 2019 11:17:13 AM UTC, comment #1: 

That first line is not mandatory as of my reading of http://www.cookiecentral.com/faq/. The regex just seems to comply with cookie files generated by curl.

So if cookielib checks for such a line, it should at least relax to be completely case insensitive.

That line has been checked in at 2001-04-08 15:25:24 (commit 2cfb2d2ef).

Please report at cookielib.

Tim Ruehsen <rockdaboot>
Group administrator
Wed 14 Aug 2019 05:35:40 AM UTC, original submission:  

This may be an issue of strict regexing where it needn't be - please let me know if I should report this in a bug repo for cookielib instead.

I'm on macOS 10.14.6, wget 1.20.3.

Command line:

> $ wget --save-cookies cookies.txt --keep-session-cookies --post-data 'username=myUserName&password=myPassword' --delete-after <url>


However, line 39 of _MozillaCookieJar.py (cookielib) shows it looking for

> magic_re = "#( Netscape)? HTTP Cookie File"

in order to validate the passed cookie file.  wget, however, produces a cookies file that begins with

> # HTTP cookie file


The lack of upper-case c and f cause cookielib to choke and stop processing the cookies file.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-08-14 rockdaboot StatusNone Wont Fix
        Assigned toNone rockdaboot
        Open/ClosedOpen Closed
        Operating SystemMac OS None

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code