diff options
author | Kevin Ottens <kevin.ottens@enioka.com> | 2021-10-06 00:13:27 +0300 |
---|---|---|
committer | Kevin Ottens (Rebase PR Action) <er-vin@users.noreply.github.com> | 2021-10-06 13:17:38 +0300 |
commit | 5fe63a4d9a7a38c4963486dff115e71edca82cb3 (patch) | |
tree | 68272d8f17d0c0bf4988ed846bc605b715e7d258 /src | |
parent | 820ca382703a28ee7d3696a0a271cc6e71fbcdf1 (diff) |
Display the right endpoint in the warning in case of error
At that point in time _pollEndpoint isn't set yet. All the checks are
against pollEndpoint so display that one in the warning. Otherwise one
always end up with an empty URL in the logs which is not very useful for
debugging purposes.
Signed-off-by: Kevin Ottens <kevin.ottens@enioka.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/gui/creds/flow2auth.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/creds/flow2auth.cpp b/src/gui/creds/flow2auth.cpp index b94de0afd..54b7b28a8 100644 --- a/src/gui/creds/flow2auth.cpp +++ b/src/gui/creds/flow2auth.cpp @@ -100,7 +100,7 @@ void Flow2Auth::fetchNewToken(const TokenAction action) pollToken = json.value("poll").toObject().value("token").toString(); pollEndpoint = json.value("poll").toObject().value("endpoint").toString(); if (_enforceHttps && QUrl(pollEndpoint).scheme() != QStringLiteral("https")) { - qCWarning(lcFlow2auth) << "Can not poll endpoint because the returned url" << _pollEndpoint << "does not start with https"; + qCWarning(lcFlow2auth) << "Can not poll endpoint because the returned url" << pollEndpoint << "does not start with https"; emit result(Error, tr("The polling URL does not start with HTTPS despite the login URL started with HTTPS. Login will not be possible because this might be a security issue. Please contact your administrator.")); return; } |