fix login bug

fixes #94 (closed)

It looks like there was an update to FastAPI recently (0.95.0 -> 0.95.2), and this updated Starlette to version 0.27.0. This was causing an error where we do a string find to determine if the application is being accessed locally. My guess is that the updated method request.url_for("authorize") now returns a non-string. This casts the type as a string before the comparison.

I am not a python developer, if you know of a better check then I can update the MR.

geomagio/api/secure/login.py

@@ -186,7 +186,7 @@ async def authorize(request: Request):
     description="Redirect to OpenID provider.",
 )
 async def login(request: Request):
-    redirect_uri = request.url_for("authorize")
+    redirect_uri = str(request.url_for("authorize"))
     if "127.0.0.1" not in redirect_uri:
         # 127.0.0.1 used for local dev, all others use https
         redirect_uri = redirect_uri.replace("http://", "https://")