mirror
/
s3proxy
kopia lustrzana https://github.com/gaul/s3proxy
1
0
Forkuj 0
Kod Zgłoszenia Packages Projekty Wydania Wiki Aktywność

Improve list bucket max-keys error handling 

Found with Ceph s3-tests.  References #5.
pull/16/head
Andrew Gaul 2014-07-29 07:34:36 -07:00
rodzic f698bd68e8
commit cf1376387c
1 zmienionych plików z 13 dodań i 4 usunięć
Pokaż statystyki Ściągnij plik aktualizacji Ściągnij plik porównania Expand all files Collapse all files

17
src/main/java/org/gaul/s3proxy/S3ProxyHandler.java
Wyświetl plik

@ -309,10 +309,19 @@ final class S3ProxyHandler extends AbstractHandler {
if (marker != null) {
options = options.afterMarker(request.getParameter("marker"));
}
String maxKeys = request.getParameter("max-keys");
if (maxKeys != null) {
options = options.maxResults(Integer.valueOf(maxKeys));
int maxKeys = 1000;
String maxKeysString = request.getParameter("max-keys");
if (maxKeysString != null) {
try {
maxKeys = Integer.valueOf(maxKeysString);
} catch (NumberFormatException nfe) {
sendSimpleErrorResponse(response,
HttpServletResponse.SC_BAD_REQUEST, "InvalidArgument",
"Bad Request", Optional.<String>absent());
return;
}
}
options = options.maxResults(maxKeys);
PageSet<? extends StorageMetadata> set;
try {
@ -337,7 +346,7 @@ final class S3ProxyHandler extends AbstractHandler {
writer.write("</Prefix>\r\n");
}
writer.write(" <MaxKeys>");
writer.write(String.valueOf(set.size()));
writer.write(String.valueOf(maxKeys));
writer.write("</MaxKeys>\r\n");
if (marker == null) {
writer.write(" <Marker/>\r\n");