bugGNU Wget - Bugs: bug #60617, POST not continued after 301, 302

 
 

bug #60617: POST not continued after 301, 302

Submitter:  Frank Heckenbach <frank>
Submitted:  Sun 16 May 2021 03:43:23 PM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  5 - Normal Status:  Ready For Test
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Open
Release:  1.20 Operating System:  GNU/Linux
Reproducibility:  Every Time Fixed Release:  None
Planned Release:  None Regression:  None
Work Required:  None Patch Included:  Yes
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 06 Aug 2022 10:03:25 AM UTC, comment #3: 

I seem unable to add myself to the "Mail Notification Carbon-Copy List" thus I've to post this comment, hopefully this worked.

Emanuel Czirai <correabuscar>
Fri 11 Mar 2022 02:55:52 PM UTC, comment #2: 

comment #1:

> I think you're right. I'll fix the code and add a new test case for it.
>
> Thanks for reporting!


Still doesn't seem to be in git. Any problems?

Frank Heckenbach <frank>
Wed 08 Sep 2021 04:14:44 PM UTC, comment #1: 

I think you're right. I'll fix the code and add a new test case for it.

Thanks for reporting!

Darshit Shah <darnir>
Group administrator
Sun 16 May 2021 03:43:23 PM UTC, original submission:  

According to the man page: "In case of a 301 Moved Permanently, 302 Moved Temporarily or 307 Temporary Redirect, Wget will, in accordance with RFC2616, continue to send a POST request."

That doesn't seem to work after 301 and 302, see debug.output.

It seems the comparison is inverted. It should keep POST if it was POST ("==", not "!="), see attached patch which seems to fix the problem for me.

Frank Heckenbach <frank>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #51444:  debug.output added by frank (3KiB - application/octet-stream)
file #51445:  redirect.patch added by frank (767B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by correabuscar (Posted a comment)
  • -email is unavailable- added by darnir (Posted a comment)
  • -email is unavailable- added by frank (Submitted the item)
  •  

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-09-08 darnir StatusNone Ready For Test
    2021-05-16 frank Attached File- Added debug.output, #51444
        Attached File- Added redirect.patch, #51445

    Back to the top

    Powered by Savane 3.13-ee12.
    Corresponding source code