fix hook name

wlanslovenija · Jun 30, 2020 · dab746c · dab746c
1 parent 5812a06
commit dab746c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/broker/src/tunneldigger_broker/broker.py b/broker/src/tunneldigger_broker/broker.py
@@ -91,7 +91,7 @@ def create_tunnel(self, broker, address, uuid, remote_tunnel_id, client_features
             return False
 
         # Rate limit creation of new tunnels with the same IP address.
-        # Runs broker.tunnel-connection-threshold hook if threshold exceeded:
+        # Runs broker.connection-rate-limit hook if threshold exceeded:
         if self.connection_rate_limit_per_ip_count > 0 and self.connection_rate_limit_per_ip_time > 0:
             if address[0] not in self.last_tunnel_created_per_ip:
                 # Create deque with max size if IP address does not exist in dict
@@ -108,7 +108,7 @@ def create_tunnel(self, broker, address, uuid, remote_tunnel_id, client_features
                         int(delta),
                     ))
                     broker.hook_manager.run_hook(
-                        'broker.tunnel-connection-threshold',
+                        'broker.connection-rate-limit',
                         address[0],
                         address[1],
                         uuid,

diff --git a/broker/src/tunneldigger_broker/main.py b/broker/src/tunneldigger_broker/main.py
@@ -59,7 +59,7 @@
     event_loop=event_loop,
     log_arguments=config.getboolean('log', 'log_ip_addresses'),
 )
-for hook in ('session.up', 'session.pre-down', 'session.down', 'session.mtu-changed', 'broker.tunnel-connection-threshold'):
+for hook in ('session.up', 'session.pre-down', 'session.down', 'session.mtu-changed', 'broker.connection-rate-limit'):
     try:
         script = config.get('hooks', hook).strip()
         if not script:

diff --git a/docs/server.rst b/docs/server.rst
@@ -204,7 +204,7 @@ There are currently four different hooks, namely:
 * ``session.mtu-changed`` is called after the broker's path MTU discovery determines that the tunnel's MTU has changed
   and should be adjusted (example is found under ``scripts/mtu_changed.sh``)
 
-* ``broker.tunnel-connection-threshold`` is called when a IP address tries to connect ``connection_rate_limit_per_ip_count``
+* ``broker.connection-rate-limit`` is called when a IP address tries to connect ``connection_rate_limit_per_ip_count``
   times within ``connection_rate_limit_per_ip_time`` seconds.
 
 Please look at all the example hook scripts carefully and try to understand