shell/source/backends/kde5be/kde5access.cxx | 32 ++++++++++------------------ 1 file changed, 12 insertions(+), 20 deletions(-)
New commits: commit 37dd169a54ec1c8e0298ce0f839b1e53ab3dbc51 Author: Katarina Behrens <katarina.behr...@cib.de> AuthorDate: Tue Apr 16 08:58:38 2019 +0200 Commit: Thorsten Behrens <thorsten.behr...@cib.de> CommitDate: Wed Apr 17 01:00:10 2019 +0200 Fix wrong proxy setting URLs in kde5backend this copy'n'pasta blunder looks innocuous but can slow down loading of JVM (which reads proxy settings among others) in KDE environment significantly Change-Id: I98f0fb107a2ee825bcd47731a5a9d15017ba63bc Reviewed-on: https://gerrit.libreoffice.org/70806 Tested-by: Jenkins Reviewed-by: Thorsten Behrens <thorsten.behr...@cib.de> (cherry picked from commit cc191e4f5da18c3b223316f39aac23003ca54671) Reviewed-on: https://gerrit.libreoffice.org/70849 Tested-by: Thorsten Behrens <thorsten.behr...@cib.de> diff --git a/shell/source/backends/kde5be/kde5access.cxx b/shell/source/backends/kde5be/kde5access.cxx index 0f2218551d5f..6147c8f0a83d 100644 --- a/shell/source/backends/kde5be/kde5access.cxx +++ b/shell/source/backends/kde5be/kde5access.cxx @@ -106,8 +106,7 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. @@ -134,8 +133,7 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. @@ -162,13 +160,12 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. aHTTPProxy = KProtocolManager::proxyForUrl( - QUrl(QStringLiteral("ftp://ftp.libreoffice.org"))); + QUrl(QStringLiteral("http://www.libreoffice.org"))); break; default: // No proxy is used break; @@ -190,13 +187,12 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. aHTTPProxy = KProtocolManager::proxyForUrl( - QUrl(QStringLiteral("ftp://ftp.libreoffice.org"))); + QUrl(QStringLiteral("http://www.libreoffice.org"))); break; default: // No proxy is used break; @@ -218,13 +214,12 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. aHTTPSProxy = KProtocolManager::proxyForUrl( - QUrl(QStringLiteral("ftp://ftp.libreoffice.org"))); + QUrl(QStringLiteral("https://www.libreoffice.org"))); break; default: // No proxy is used break; @@ -246,13 +241,12 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) break; case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables // In such cases, the proxy address is not stored in KDE, but determined dynamically. // The proxy address may depend on the requested address, on the time of the day, on the speed of the wind... // The best we can do here is to ask the current value for a given address. aHTTPSProxy = KProtocolManager::proxyForUrl( - QUrl(QStringLiteral("ftp://ftp.libreoffice.org"))); + QUrl(QStringLiteral("https://www.libreoffice.org"))); break; default: // No proxy is used break; @@ -272,8 +266,7 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) case KProtocolManager::ManualProxy: // Proxies are manually configured case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables aNoProxyFor = KProtocolManager::noProxyFor(); break; default: // No proxy is used @@ -296,8 +289,7 @@ css::beans::Optional<css::uno::Any> getValue(OUString const& id) case KProtocolManager::ManualProxy: // Proxies are manually configured case KProtocolManager::PACProxy: // A proxy configuration URL has been given case KProtocolManager::WPADProxy: // A proxy should be automatically discovered - case KProtocolManager:: - EnvVarProxy: // Use the proxy values set through environment variables + case KProtocolManager::EnvVarProxy: // Proxy values set through environment variables nProxyType = 1; break; default: // No proxy is used _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits