HTTP REST: minor fixes
1) const-ify internal helper ParseHashStr() 2) use HTTPError() helper when returning HTTP_NOT_FOUND
This commit is contained in:
parent
9445b876bd
commit
7715c84747
@ -56,7 +56,7 @@ static enum RetFormat ParseDataFormat(const string& format)
|
||||
return rf_names[0].rf;
|
||||
}
|
||||
|
||||
static bool ParseHashStr(string& strReq, uint256& v)
|
||||
static bool ParseHashStr(const string& strReq, uint256& v)
|
||||
{
|
||||
if (!IsHex(strReq) || (strReq.size() != 64))
|
||||
return false;
|
||||
@ -195,7 +195,7 @@ bool HTTPReq_REST(AcceptedConnection *conn,
|
||||
return false;
|
||||
}
|
||||
|
||||
conn->stream() << HTTPReply(HTTP_NOT_FOUND, "", false) << std::flush;
|
||||
conn->stream() << HTTPError(HTTP_NOT_FOUND, false) << std::flush;
|
||||
return false;
|
||||
}
|
||||
|
||||
|
@ -957,7 +957,7 @@ void ServiceConnection(AcceptedConnection *conn)
|
||||
break;
|
||||
|
||||
} else {
|
||||
conn->stream() << HTTPReply(HTTP_NOT_FOUND, "", false) << std::flush;
|
||||
conn->stream() << HTTPError(HTTP_NOT_FOUND, false) << std::flush;
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user