Skip to content

Changing WinHTTP flags: WINHTTP_FLAG_SECURE_DEFAULTS for HTTPS, WINHTTP_FLAG_ASYNC for HTTP#907

Merged
jasonsandlin merged 4 commits intomainfrom
user/jasonsa/winhttp-flags
Sep 4, 2025
Merged

Changing WinHTTP flags: WINHTTP_FLAG_SECURE_DEFAULTS for HTTPS, WINHTTP_FLAG_ASYNC for HTTP#907
jasonsandlin merged 4 commits intomainfrom
user/jasonsa/winhttp-flags

Conversation

@jasonsandlin
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@ScottMunroMS ScottMunroMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve w/ suggestions

Comment thread Source/HTTP/WinHttp/winhttp_provider.cpp
Comment thread Source/HTTP/WinHttp/winhttp_provider.cpp
Comment thread Source/HTTP/WinHttp/winhttp_provider.cpp Outdated
Comment thread Tests/UnitTests/Tests/HttpTests.cpp Outdated
Comment thread Tests/UnitTests/Tests/HttpTests.cpp Outdated
Comment thread Tests/UnitTests/Tests/HttpTests.cpp Outdated
Comment thread Tests/UnitTests/Tests/HttpTests.cpp
Copy link
Copy Markdown
Contributor

@ScottMunroMS ScottMunroMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jasonsandlin jasonsandlin merged commit 54172bf into main Sep 4, 2025
12 checks passed
@jasonsandlin jasonsandlin deleted the user/jasonsa/winhttp-flags branch September 4, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants