Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

installer: really default to Secure Channel #586

Merged

Conversation

rimrul
Copy link
Member

@rimrul rimrul commented Jan 4, 2025

Commit 8a7ab99 (installer: switch the libcurl default to Secure Channel, 2023-05-15) attempted to change the default libcurl backend for clean new installs to Secure channel, but only adjusted one of two fallbacks in the installer. There is one default value that gets passed to ReplayChoice() to return if it finds no previous choice and one value that gets used if ReplayChoice() returns an unexpected value. 8a7ab99 changed the latter to use Secure Channel, but left the former to default to OpenSSL. Adjust the default value to match the intention of that patch.

This fixes git-for-windows/git#5295

Commit 8a7ab99 (installer: switch the libcurl default to Secure Channel,
2023-05-15) attempted to change the default libcurl backend for clean
new installs to Secure channel, but only adjusted one of two fallbacks
in the installer. There is one default value that gets passed to
ReplayChoice() to return if it finds no previous choice and one value
that gets used if ReplayChoice() returns an unexpected value. 8a7ab99
changed the latter to use Secure Channel, but left the former to default
to OpenSSL. Adjust the default value to match the intention of that
patch.

This fixes git-for-windows/git#5295

Signed-off-by: Matthias Aßhauer <[email protected]>
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Excellent find, thank you for that and the fix!

@dscho
Copy link
Member

dscho commented Jan 4, 2025

/add relnote feature New Git for Windows installation now default to the Windows-native HTTPS transport backend.

The workflow run was started

@dscho dscho merged commit a3502f0 into git-for-windows:main Jan 4, 2025
6 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 4, 2025
New Git for Windows installation [now default to the Windows-native
HTTPS transport
backend](#586).

Signed-off-by: gitforwindowshelper[bot] <[email protected]>
@rimrul rimrul deleted the installer-really-default-to-schannel branch January 4, 2025 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants