Martchus
/
arch-repo-manager
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix returning output when no offset present

This commit is contained in:
Martchus 2021-04-05 21:11:13 +02:00
parent 901092ba09
commit 338674a9e6
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
librepomgr/webapi/routes_buildaction.cpp
View File

@ -116,11 +116,13 @@ void getBuildActionOutput(const Params &params, ResponseHandler &&handler)
handler(makeBadRequest(params.request(), "the offset parameter must be specified at most once")); handler(makeBadRequest(params.request(), "the offset parameter must be specified at most once"));
return; return;
} }
try { if (!offsetParams.empty()) {
offset = stringToNumber<std::size_t>(offsetParams.front()); try {
} catch (const ConversionException &) { offset = stringToNumber<std::size_t>(offsetParams.front());
handler(makeBadRequest(params.request(), "the offset must be an unsigned integer")); } catch (const ConversionException &) {
return; handler(makeBadRequest(params.request(), "the offset must be an unsigned integer"));
return;
}
} }
auto buildActionsSearchResult = findBuildActions(params, std::move(handler), false, 1); auto buildActionsSearchResult = findBuildActions(params, std::move(handler), false, 1);
if (!buildActionsSearchResult.ok) { if (!buildActionsSearchResult.ok) {