shell/source/backends/kde5be/kde5access.cxx |   32 ++++++++++------------------
 1 file changed, 12 insertions(+), 20 deletions(-)

New commits:
commit cc191e4f5da18c3b223316f39aac23003ca54671
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 00:58:50 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>

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

Reply via email to