Add a default value for the /repository/files wait parameter

Author: dsotirho-ucsc

lambdas/service/app.py

@@ -1661,7 +1661,7 @@ def generate_manifest(event: AnyJSON, _context: LambdaContext):
         ),
         params.query(
             'wait',
-            schema.optional(int),
+            schema.optional(schema.with_default(0)),
             description=fd('''
                 If 0, the client is responsible for honoring the waiting period
                 specified in the Retry-After response header. If 1, the server

lambdas/service/openapi.json

@@ -11169,7 +11169,8 @@
                         "required": false,
                         "schema": {
                             "type": "integer",
-                            "format": "int64"
+                            "format": "int64",
+                            "default": 0
                         },
                         "description": "\nIf 0, the client is responsible for honoring the waiting period\nspecified in the Retry-After response header. If 1, the server\nwill delay the response in order to consume as much of that\nwaiting period as possible. This parameter should only be set to\n1 by clients who can't honor the `Retry-After` header,\npreventing them from quickly exhausting the maximum number of\nredirects. If the server cannot wait the full amount, any amount\nof wait time left will still be returned in the Retry-After\nheader of the response.\n"
                     },
@@ -11304,7 +11305,8 @@
                         "required": false,
                         "schema": {
                             "type": "integer",
-                            "format": "int64"
+                            "format": "int64",
+                            "default": 0
                         },
                         "description": "\nIf 0, the client is responsible for honoring the waiting period\nspecified in the Retry-After response header. If 1, the server\nwill delay the response in order to consume as much of that\nwaiting period as possible. This parameter should only be set to\n1 by clients who can't honor the `Retry-After` header,\npreventing them from quickly exhausting the maximum number of\nredirects. If the server cannot wait the full amount, any amount\nof wait time left will still be returned in the Retry-After\nheader of the response.\n"
                     },

src/azul/service/repository_controller.py

@@ -4,6 +4,9 @@
 )
 import json
 import logging
+from math import (
+    ceil,
+)
 from typing import (
     Optional,
     TYPE_CHECKING,
@@ -263,7 +266,7 @@ def download_file(self,
                     pass
                 elif wait == '1':
                     time_slept = self.server_side_sleep(float(retry_after))
-                    retry_after = round(retry_after - time_slept)
+                    retry_after = ceil(retry_after - time_slept)
                 else:
                     assert False, wait
                 query_params['wait'] = wait