Add requirements

Reviewed-on: #1

.gitea/workflows/flake8.yml

@@ -0,0 +1,17 @@
+---
+name: Flake8
+on: [push]
+
+
+# XXX need to do stuff with uv
+jobs:
+  build:
+    runs-on: freebsd
+    strategy:
+      matrix:
+        python-version: ["3.11"]
+    steps:
+      - uses: actions/checkout@v4
+      - name: Analyse code with Flake8
+        run: |
+          flake8 $(git ls-files '*.py')

.gitea/workflows/mypy.yml

@@ -0,0 +1,17 @@
+---
+name: Mypy
+on: [push]
+
+
+# XXX need to do stuff with uv
+jobs:
+  build:
+    runs-on: freebsd
+    strategy:
+      matrix:
+        python-version: ["3.11"]
+    steps:
+      - uses: actions/checkout@v4
+      - name: Analyse code with Mypy
+        run: |
+          mypy --install-types --non-interactive $(git ls-files '*.py')

.gitea/workflows/pylint.yml

@@ -0,0 +1,17 @@
+---
+name: Pylint
+on: [push]
+
+
+# XXX need to do stuff with uv
+jobs:
+  build:
+    runs-on: freebsd
+    strategy:
+      matrix:
+        python-version: ["3.11"]
+    steps:
+      - uses: actions/checkout@v4
+      - name: Analyse code with Pylint
+        run: |
+          pylint $(git ls-files '*.py')

app/main.py

@@ -5,7 +5,9 @@ from ipaddress import IPv4Address, IPv4Network, IPv6Address, IPv6Network
 from typing import Annotated, Optional, Union
 
 import geoip2.database
-from fastapi import FastAPI, Path, Body
+from geoip2.errors import AddressNotFoundError
+from fastapi import FastAPI, Path, Body, Request, Response, status
+from fastapi.responses import RedirectResponse
 from pydantic import BaseModel
 
 app = FastAPI()
@@ -13,38 +15,52 @@ app = FastAPI()
 GEOLITE2_ASN_DB = '/usr/local/share/GeoIP/GeoLite2-ASN.mmdb'
 GEOLITE2_CITY_DB = '/usr/local/share/GeoIP/GeoLite2-City.mmdb'
 
+
 class IPAddressParam(BaseModel):
+    '''
+    Payload entry as used in POST
+    '''
     ip: Union[IPv6Address, IPv4Address]
 
+
 class Locality(BaseModel):
     '''
     Locality data
     '''
-    city: Optional[str]
-    country: Optional[str]
-    continent: Optional[str]
-    is_eu: bool
+    city: Optional[str] = None
+    country: Optional[str] = None
+    continent: Optional[str] = None
+    is_eu: bool = False
+
 
 class GeoLocation(BaseModel):
     '''
     Geolocation data model
     '''
-    ip: Union[IPv6Address,IPv4Address]
-    asn: Optional[int]
-    asn_org: Optional[str]
-    network: Union[IPv6Network,IPv4Network,None]
-    locality: Locality
+    ip: Optional[Union[IPv6Address, IPv4Address]] = None
+    asn: Optional[int] = None
+    asn_org: Optional[str] = None
+    network: Optional[Union[IPv6Network, IPv4Network]] = None
+    locality: Locality = Locality()
+
 
 @app.post("/")
-async def root_post(ipaddresses: Annotated[list[IPAddressParam],
-                                    Body(title="The IPAddresses to geolocate")]
+async def root_post(ipaddresses: Annotated[
+    list[IPAddressParam],
+    Body(title="The IPAddresses to geolocate")],
+                    response: Response
                     ) -> list[GeoLocation]:
+    '''
+    Return GeoLocation item(s) for a list of IPAddressParam objects
+    '''
     geolocations = []
     with (geoip2.database.Reader(GEOLITE2_ASN_DB) as reader_asn,
             geoip2.database.Reader(GEOLITE2_CITY_DB) as reader_city):
         for ipaddress in ipaddresses:
+            try:
                 asn_data = reader_asn.asn(ipaddress.ip)
                 city_data = reader_city.city(ipaddress.ip)
+
                 geolocations.append(GeoLocation(
                     ip=ipaddress.ip,
                     asn=asn_data.autonomous_system_number,
@@ -57,14 +73,36 @@ async def root_post(ipaddresses: Annotated[list[IPAddressParam],
                         is_eu=city_data.country.is_in_european_union
                     )
                 ))
+            except AddressNotFoundError:
+                geolocations.append(GeoLocation(
+                    ip=ipaddress.ip
+                    )
+                )
+    if geolocations:
+        response.headers['Cache-Control'] = 'private, max-age=604800'
     return geolocations
 
 
 @app.get("/{ipaddress}")
-async def root_get(ipaddress: Annotated[Union[IPv4Address,IPv6Address],
-                                    Path(title="The IPAddress to geolocate")]
+async def root_get(ipaddress: Annotated[
+    Union[IPv4Address, IPv6Address],
+    Path(title="The IPAddress to geolocate")],
+                   response: Response
                    ) -> GeoLocation:
     '''
     Look up geolocation for ip in path parameter
     '''
-    return (await root_post([IPAddressParam(ip=ipaddress)])).pop()
+    locations = await root_post([IPAddressParam(ip=ipaddress)], response)
+    if locations:
+        response.headers['Cache-Control'] = 'private, max-age=604800'
+        return locations.pop()
+    response.status_code = status.HTTP_404_NOT_FOUND
+    return GeoLocation()
+
+
+@app.get("/")
+def root_redirect(req: Request) -> RedirectResponse:
+    '''
+    Redirect empty request using REMOTE_ADDR
+    '''
+    return RedirectResponse(url=str(req.url) + str(req.client.host))

requirements.txt

@@ -0,0 +1,2 @@
+geoip2==4.7.0
+fastapi==0.115.6

start.sh

@@ -0,0 +1,24 @@
+#!/bin/sh
+
+NAME=ismijnipverweg
+DIR=/opt/apps/ismijnipverweg/app
+USER=www
+GROUP=nobody
+WORKERS=3
+WORKER_CLASS=uvicorn.workers.UvicornWorker
+#VENV=$DIR/.venv/bin/activate
+BIND=unix:$DIR/../run/gunicorn.sock
+LOG_LEVEL=error
+
+cd $DIR
+# source $VENV
+
+exec /usr/local/bin/gunicorn main:app \
+  --name $NAME \
+  --workers $WORKERS \
+  --worker-class $WORKER_CLASS \
+  --user=$USER \
+  --group=$GROUP \
+  --bind=$BIND \
+  --log-level=$LOG_LEVEL \
+  --log-file=-