Merge pull request #948 from girder/rest-rate-limiting

Adjust logging rate for region endpoints
girder · Aug 30, 2022 · 7515336 · 7515336
2 parents f37f32d + 7b170b9
commit 7515336
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,9 @@
 - Better ignore tiff directories that aren't part of the pyramid ([943](../../pull/943))
 - Fix an issue with styling frames in ome tiffs ([945](../../pull/945))
 
+### Changes
+- Adjusted rest request logging rates for region endpoint ([948](../../pull/948))
+
 ## 1.16.1
 
 ### Improvements

diff --git a/girder/girder_large_image/rest/tiles.py b/girder/girder_large_image/rest/tiles.py
@@ -165,12 +165,19 @@ def __init__(self, apiRoot):
                            self.getDZITile)
         apiRoot.item.route('GET', (':itemId', 'tiles', 'internal_metadata'),
                            self.getInternalMetadata)
+        # Logging rate limiters
         filter_logging.addLoggingFilter(
             'GET (/[^/ ?#]+)*/item/[^/ ?#]+/tiles/zxy(/[^/ ?#]+){3}',
-            frequency=250)
+            frequency=250, duration=10)
+        filter_logging.addLoggingFilter(
+            'GET (/[^/ ?#]+)*/item/[^/ ?#]+/tiles/fzxy(/[^/ ?#]+){3}',
+            frequency=250, duration=10)
         filter_logging.addLoggingFilter(
             'GET (/[^/ ?#]+)*/item/[^/ ?#]+/tiles/dzi_files(/[^/ ?#]+){2}',
-            frequency=250)
+            frequency=250, duration=10)
+        filter_logging.addLoggingFilter(
+            'GET (/[^/ ?#]+)*/item/[^/ ?#]+/tiles/region',
+            frequency=100, duration=10)
         # Cache the model singleton
         self.imageItemModel = ImageItem()