[Bug 43216] New: winhttp doesn' t support the WINHTTP_OPTION_REDIRECT_POLICY option.

https://bugs.winehq.org/show_bug.cgi?id=43216

Bug ID: 43216
           Summary: winhttp doesn't support the
                    WINHTTP_OPTION_REDIRECT_POLICY option.
           Product: Wine
           Version: 2.10
          Hardware: x86
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: winhttp
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Here is the basic code.
WinHttpOpen(....)
DWORD options = WINHTTP_OPTION_REDIRECT_POLICY_ALWAYS;
WinHttpSetOption(session, WINHTTP_OPTION_REDIRECT_POLICY, &options,
sizeof(options));
WinHttpConnect(session, L"website", 80, 0 );

WinHttpOpenRequest(connect, L"GET", L"/project", ...)
WinHttpSendRequest(.....)

When the WinHttpSendRequest receives a status HTTP_STATUS_MOVED, the requested
page isn't automatically loaded from the "Location" header field.

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

Alistair Leslie-Hughes  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|winhttp doesn't support the |winhttp fails to redirect
                   |WINHTTP_OPTION_REDIRECT_POL |from http to https on 301
                   |ICY option.                 |error.

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

--- Comment #1 from Hans Leidekker  ---
Can you attach a +winhttp,+tid trace?

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

--- Comment #2 from Alistair Leslie-Hughes  ---
Created attachment 58515
  --> https://bugs.winehq.org/attachment.cgi?id=58515
+winhttp

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

--- Comment #3 from Hans Leidekker  ---
Created attachment 58516
  --> https://bugs.winehq.org/attachment.cgi?id=58516
patch

Does this patch work for you?

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

--- Comment #4 from Alistair Leslie-Hughes  ---
(In reply to Hans Leidekker from comment #3)
> Created attachment 58516 [details]
> patch
>
> Does this patch work for you?

Yes, with the patch it works as expected.

Thanks.

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

Alistair Leslie-Hughes  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |f2bb2064bfdf0e7b369f22c34d6
                   |                            |80dec6c12493c

--- Comment #5 from Alistair Leslie-Hughes  ---
Fixed by 
http://source.winehq.org/git/wine.git/?a=commit;h=f2bb2064bfdf0e7b369f22c34d...

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 2.11.

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

Michael Stefaniuc  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |2.0.x

[Bug 43216] winhttp fails to redirect from http to https on 301 error.

https://bugs.winehq.org/show_bug.cgi?id=43216

Michael Stefaniuc  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|2.0.x                       |---

--- Comment #7 from Michael Stefaniuc  ---
Removing the 2.0.x milestone from bugs included in 2.0.3.