search

Quick-Event / quickbox

Quick Event - Orienteering Software
GNU General Public License v2.0
53 stars 41 forks source link

OResults service - Better error message when request fails to be delivered. #811

Closed otahirs closed 2 years ago

otahirs commented 2 years ago

Tušíš jak kvalitní připojení jste měli? LTE, 4G? My jsme dnes pozorovali problémy při uploadu pouze přes službu OResults. Po chvíli upload proběhl zase úspěšně. Chování jsme přisuzovali slabšímu mobilnímu připojení. Vliv na chod vyčítání jsem nepozoroval. Možná to spolu nesouvisí a je to spíš na jinou issue. Fotku logu přímo z akce: log_oresults

Originally posted by @lukaskett in https://github.com/Quick-Event/quickbox/issues/809#issuecomment-1264719245

fvacek commented 2 years ago

Koukal jsem do kodu a problem je, ze se neloguje errorString(), takze nevime, proc to vlastne nedopadlo, opravil jsem to, tak jestli nekdo muze, at to vyzkousi, mohli bychom byt chytrejsi. https://github.com/Quick-Event/quickbox/commit/33f00f45755b73cecb977ecf8a10809d17d2a08f

otahirs commented 2 years ago

kdybych se mohl assignout, tak se assignu. server vraci chybovy text, kdyz se mu na poslanych datech neco nelibi, proto zde bylo QString(reply->readAll())
pokud vsak request selze jiz drive, tak dava smysl vypsat error z reply->errorString() vyzkousim a asi prepisu na QString(reply->readAll()), pokud je prazdne tak na reply->errorString(). musim otestovat

fvacek commented 2 years ago

kdybych se mohl assignout, tak se assignu. server vraci chybovy text, kdyz se mu na poslanych datech neco nelibi, proto zde bylo QString(reply->readAll()) pokud vsak request selze jiz drive, tak dava smysl vypsat error z reply->errorString() vyzkousim a asi prepisu na QString(reply->readAll()), pokud je prazdne tak na reply->errorString(). musim otestovat

zkusim to nejak nakonfigurovat, abys mohl, zkus to ted?