Update Dependencies - Feb 2024

pyproject.toml

@@ -4,37 +4,27 @@ description = "An automated Slack bot to syndicate local events into Slack chann
 readme = "README.md"
 requires-python = ">=3.11"
 license = { text = "MIT" }
-version="0.1.0"
-authors = [
-  { name = "Olivia Sculley", email = "[email protected]" },
-]
-keywords = [
-  "automation",
-  "black",
-  "bolt",
-  "bot",
-  "pylint",
-  "pytest",
-  "slack"
-]
+version = "0.1.0"
+authors = [{ name = "Olivia Sculley", email = "[email protected]" }]
+keywords = ["automation", "black", "bolt", "bot", "pylint", "pytest", "slack"]
 dependencies = [
-  "aiohttp==3.9.1",
-  "fastapi==0.103.2",
+  "aiohttp==3.9.3",
+  "fastapi==0.109.2",
   "python_dateutil==2.8.2",
-  "pytz==2023.3",
-  "slack_bolt==1.18.0",
-  "uvicorn==0.23.2"
+  "pytz==2024.1",
+  "slack_bolt==1.18.1",
+  "uvicorn==0.27.0.post1",
 ]
 
 [project.optional-dependencies]
 test = [
-    "black==23.9.1",
-    "httpx==0.25.0",
-    "isort==5.12.0",
-    "pylint==2.17.5",
-    "pytest==7.4.2",
-    "pytest-asyncio==0.21.1",
-    "ssort==0.11.6"
+  "black==24.1.1",
+  "httpx==0.26.0",
+  "isort==5.13.2",
+  "pylint==3.0.3",
+  "pytest==7.4.4",
+  "pytest-asyncio==0.23.4",
+  "ssort==0.11.6",
 ]
 
 [project.urls]
@@ -43,13 +33,8 @@ Documentation = "https://github.com/hackgvl/slack-events-bot/blob/dev/README.md"
 Repository = "https://github.com/hackgvl/slack-events-bot.git"
 
 [tool.pytest.ini_options]
-pythonpath = [
-  ".",
-  "src"
-]
-norecursedirs = [
-  "tests/helpers"
-]
+pythonpath = [".", "src"]
+norecursedirs = ["tests/helpers"]
 
 [tool.isort]
 profile = "black"

src/auth.py

@@ -2,6 +2,7 @@
 Logic for restricting the use of Slack commands to specific parties
 and validating incoming requests.
 """
+
 import hashlib
 import hmac
 import logging

src/database.py

@@ -1,4 +1,5 @@
 """Contains all the functions that interact with the sqlite database"""
+
 import datetime
 import os
 import sqlite3

src/server.py

@@ -3,6 +3,7 @@
 
   Visit the /docs route for more information on the routes contained within.
 """
+
 import asyncio
 import datetime
 import logging
@@ -16,7 +17,6 @@
 import uvicorn
 from fastapi import HTTPException, Request
 from fastapi.responses import PlainTextResponse
-from starlette.types import Message
 
 import database
 from auth import validate_slack_command_source
@@ -84,39 +84,12 @@ async def update_check_api_cooldown(team_domain: str | None) -> None:
     await database.create_cooldown(team_domain, "check_api", 15)
 
 
-async def set_body(req: Request, body: bytes):
-    """
-    Overrides the Request class's __receive method as a workaround to an issue
-    where accessing a request body in middleware causes it to become blocking.
-
-    See https://github.com/tiangolo/fastapi/discussions/8187 for the discussion
-    and this post (https://github.com/tiangolo/fastapi/discussions/8187#discussioncomment-5148049)
-    for where this code originates. Thanks, https://github.com/liukelin!
-    """
-
-    async def receive() -> Message:
-        return {"type": "http.request", "body": body}
-
-    # pylint: disable=protected-access
-    req._receive = receive
-
-
-async def get_body(req: Request) -> bytes:
-    """
-    Leans into the overriden Request.__receive method seen above in set_body
-    to workaround 'await req.body()' causing the application to hang.
-    """
-    body = await req.body()
-    await set_body(req, body)
-    return body
-
-
 @API.middleware("http")
 async def rate_limit_check_api(
     req: Request, call_next: Callable[[Request], Awaitable[None]]
 ):
     """Looks to see if /check_api has been run recently, and returns an error if so."""
-    req_body = await get_body(req)
+    req_body = await req.body()
 
     if await check_api_being_requested(req.scope["path"], req_body):
         team_domain = await identify_slack_team_domain(req_body)