Skip to content

Add proxy support and configurable download location - #47

Open
siator72 wants to merge 4 commits into
mjishnu:mainfrom
siator72:main
Open

siator72 wants to merge 4 commits into
mjishnu:mainfrom
siator72:main

Conversation

@siator72

@siator72 siator72 commented Aug 25, 2026

Copy link
Copy Markdown

Changes

Proxy support (new feature)

  • IProxyService, ProxyService, ProxyEntry - manage multiple proxy entries with runtime switching
  • DownloadLocationService - configurable download location
  • Settings page UI for proxy & download location management (SettingsPage.xaml / .cs, SettingsViewModel)
  • New localized strings in Resources.resw
  • Wired up in App.xaml.cs, ActivationService, DownloadManagerService, GitHubUpdaterService, DownloadHelper

StoreListings dependency

This feature needs the proxy-routing changes in the StoreListings submodule (ProxyManager + Helpers.cs). Those are submitted separately as mjishnu/StoreListings#1. .gitmodules is untouched - once that PR is merged, this PR's submodule pointer can be bumped to the upstream commit and everything stays on mjishnu/StoreListings. Until then, this branch points the submodule at my fork commit (ddf5abf) so it remains buildable/testable.

All changes built and tested locally (WinUI 3 / .NET 10, x64).

@mjishnu mjishnu self-assigned this Aug 26, 2026
@mjishnu

mjishnu commented Aug 27, 2026

Copy link
Copy Markdown
Owner

I merged the store‑listing PR after making some edits. Specifically, I switched from destroying and rebuilding HttpClient instances to using an IWebProxy wrapper, which is thread‑safe.

This requires a small change in the Raven PR:

StoreListings.Library.ProxyManager.Proxy = webProxy;

instead of

StoreListings.Library.Internal.ProxyManager.SetProxy(webProxy);

@mjishnu mjishnu added the enhancement New feature or request label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants