-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Allows setting WinGet proxy setting as SFS Client Proxy #5032
Allows setting WinGet proxy setting as SFS Client Proxy #5032
Conversation
Requires ingesting sfs-client update from microsoft/sfs-client#218
Requires ingesting sfs-client update including microsoft/sfs-client#218
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Before merging we'll have to update the sfs subtree in src/SfsClient
(instructions in the README in that location)
ebb149d
to
5426ef4
Compare
nidietr-MSFT
commented
Dec 6, 2024
via email
•
edited by microsoft-github-policy-service
bot
Loading
edited by microsoft-github-policy-service
bot
@microsoft-github-policy-service agree company="Microsoft"
|
0e27525d Bumping version to 1.1.0 (microsoft#222) c639a506 Adding support for a custom proxy input (microsoft#218) 258d189b Improve logging when the content type is wrong (microsoft#221) 216210ab Adding required permissions to enable uploading of CodeQL results (microsoft#214) fb953d6e Bump github/codeql-action from 2 to 3 (microsoft#215) 52af7124 Enabling CodeQL scanning (microsoft#211) e555d764 Bump clang-format from 18.1.5 to 19.1.1 (microsoft#210) ab8f0e72 Setup: improving build tools installation (microsoft#207) git-subtree-dir: src/SfsClient/sfs-client git-subtree-split: 0e27525d597c730e71646fd0b15bdc8c8503f24d
…t-Update1.1.0-ForProxySupport
Pulling SfsClient subtree release 1.1.0 [SfsClient Release 1.1.0](https://github.com/microsoft/sfs-client/releases/tag/1.1.0) Which brought especially [SfsClient - Client: proxy support #84](microsoft/sfs-client#218) This subtree update is required for WinGet Download Proxy support [WinGet download doesn't work through a proxy #5030](#5030) Also requires passing the appropriate proxy to SfsClient which is done as part of separate PR [Allows setting WinGet proxy setting as SFS Client Proxy #5032](#5032)
I've merged the SfsClient update PR; please remove the SfsClient update from this PR and we can run a build. |
Closing this PR to have a clean change in |
Allows setting WinGet proxy setting as the SFS Client Proxy. Fixes the issue below [WinGet download doesn't work through a proxy #5030](#5030) This PR requires SfsClient subtree updated to 1.1.0 [SfsClient update 1.1.0 for proxy support #5052](#5052) Which brings especially [Adding support for a custom proxy input #218](microsoft/sfs-client#218) This PR replaces closed PR below [Allows setting WinGet proxy setting as SFS Client Proxy #5032](#5032) <!-- To check a checkbox place an "x" between the brackets. e.g: [x] --> - [x] I have signed the [Contributor License Agreement](https://cla.opensource.microsoft.com/microsoft/winget-pkgs). - [x] This pull request is related to an issue.
Allows setting WinGet proxy setting as the SFS Client Proxy.
Fixes the issue below
WinGet download doesn't work through a proxy #5030
This PR requires ingesting Sfs-client update with
microsoft/sfs-client#218
Microsoft Reviewers: Open in CodeFlow