Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

main: fix incorrect proxy detection in KDE when proxy is disabled globally#1453

Merged
davvid merged 2 commits intogit-cola:mainfrom
brlin-tw:patch/fix-kde-proxy-autodetection2
Jan 18, 2025
Merged

main: fix incorrect proxy detection in KDE when proxy is disabled globally#1453
davvid merged 2 commits intogit-cola:mainfrom
brlin-tw:patch/fix-kde-proxy-autodetection2

Conversation

@brlin-tw
Copy link
Contributor

When the proxy is disabled globally in the System Settings in KDE the proxy host and address settings may still be preserved in the kioslaverc/Proxy Settings/*Proxy configuration settings.

Check the kioslaverc/Proxy Settings/httpProxy setting's value and return None when it is not set to "1"(meaning that the "Use manually specified proxy configuration" mode isn't selected).

Fixes #1452.

…bally

When the proxy is disabled globally in the System Settings in KDE the
proxy host and address settings may still be preserved in the kioslaverc/Proxy Settings/*Proxy
configuration settings.

Check the kioslaverc/Proxy Settings/httpProxy setting's value and
return None when it is not set to "1"(meaning that the "Use manually
specified proxy configuration" mode isn't selected).

Fixes git-cola#1452.

Signed-off-by: 林博仁(Buo-ren Lin) <[email protected]>
@davvid davvid merged commit 361d469 into git-cola:main Jan 18, 2025
4 checks passed
@davvid
Copy link
Member

davvid commented Jan 18, 2025

thanks!

@brlin-tw brlin-tw deleted the patch/fix-kde-proxy-autodetection2 branch January 18, 2025 22:10
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.

KDE proxy settings wrongly detected when it is previously set but disabled globally

2 participants