diff options
| author | Robert Adam <dev@robert-adam.de> | 2021-04-16 18:33:47 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-16 18:33:47 +0300 |
| commit | ae3cc1ea692b8377dacd85e8dab71cff8827a5b6 (patch) | |
| tree | 0ee0c0795c3c89d960439001c9933d89cf650d3b /src/mumble/OverlayConfig.cpp | |
| parent | c10d636cef79b07784318671df97dba6c1af2b83 (diff) | |
| parent | affc426c5a0871611940bd5adee9f25da068e1d0 (diff) | |
Merge PR #4929: FIX(client): CrashReporter considering 2xx codes as errors
Apparently nobody has told Qt that 2xx HTTP status codes are considered
success. Therefore checking QNetworkReply::error() in order to check
whether the transfer was successful, does not work.
Instead this commit makes sure that the raw HTTP status code is checked
instead and if it is in range [200,299], the sending of the report is
considered a success.
Furthermore this commit reworks the Strings displayed to the user as
these were excessively long. These formulations have been replaced with
(hopefully) more concise versions.
Fixes #4928
Diffstat (limited to 'src/mumble/OverlayConfig.cpp')
0 files changed, 0 insertions, 0 deletions