Enable pyright reportOptionalSubscript (#611)

* Enable pyright reportPrivateImportUsage. * Enable pyright reportOptionalSubscript.
mamba-org · Feb 18, 2023 · 77d05ba · 77d05ba
1 parent 7ed1e4a
commit 77d05ba
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -50,7 +50,7 @@ reportMissingModuleSource = false
 reportMissingTypeStubs = false
 reportOptionalMemberAccess = false
 reportOptionalOperand = false
-reportOptionalSubscript = false
+reportOptionalSubscript = true
 reportPrivateImportUsage = true
 reportUnboundVariable = false
 reportUndefinedVariable = false

diff --git a/quetz/authentication/base.py b/quetz/authentication/base.py
@@ -301,7 +301,9 @@ def configure(self, config):
     async def authenticate(
         self, request: Request, data=None, dao=None, config=None, **kwargs
     ) -> Optional[str]:
-        if data["username"] == data["password"]:
+        if data is None:
+            return None
+        elif data["username"] == data["password"]:
             return data['username']
         else:
             return None
diff --git a/quetz/migrations/env.py b/quetz/migrations/env.py
@@ -73,7 +73,7 @@ def run_migrations_online():
     and associate a connection with the context.
 
     """
-    configuration = config.get_section(config.config_ini_section)
+    configuration = config.get_section(config.config_ini_section, {})
     configuration["sqlalchemy.url"] = get_url()
     connection = config.attributes.get('connection', None)