chore: Upgrade python

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.7-slim
+FROM python:3.11
 
 WORKDIR /app
 COPY ./install.sh ./requirements.txt ./
@@ -9,4 +9,4 @@ COPY ./core ./core
 ENTRYPOINT ["uvicorn", "--host", "0.0.0.0", "--port", "8000", "app.app:app"]
 EXPOSE 8000
 
-ENV PYTHONUNBUFFERED=1
+ENV PYTHONUNBUFFERED=1

app/app.py

@@ -1,9 +1,12 @@
+import logging
+
 from fastapi import FastAPI
-from loguru import logger
 from starlette.requests import Request
 
 from app.api import api as api_router
 
+logger = logging.getLogger(__name__)
+
 app = FastAPI(title='prayertimes',
               version='1.0.0',
               description='An API for fetching islamic prayer times from Presidency of Religious Affairs of Turkey')

core/diyanet.py

@@ -3,10 +3,17 @@ from datetime import datetime
 from typing import List
 
 import httpx
+from aiocache import cached, Cache
 from bs4 import BeautifulSoup
 from pydantic import BaseModel
 
-_http = httpx.AsyncClient()
+_http = httpx.AsyncClient(
+    timeout=20,
+    follow_redirects=True,
+    headers={
+        'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:98.0) Gecko/20100101 Firefox/98.0'
+    }
+)
 
 
 class PrayerTimes(BaseModel):
@@ -19,22 +26,30 @@ class PrayerTimes(BaseModel):
     isha: str
 
 
+@cached(ttl=3 * 60 * 60, cache=Cache.MEMORY)
 async def parse_prayer_times(url: str) -> List[PrayerTimes]:
     res = await _http.get(url)
+    res.raise_for_status()
     soup = BeautifulSoup(res.text, 'html.parser')
 
     items = []
     for row in soup.select('#tab-1 .vakit-table tbody tr'):
-        cell_texts = (c.text.strip() for c in row.select('td'))
-        headers = PrayerTimes.__fields__.keys()
-        parsed = dict(zip(headers, cell_texts))
-        parsed['date'] = datetime.strptime(parsed['date'], '%d.%m.%Y')
-        items.append(PrayerTimes(**parsed))
+        cells = [c.text.strip() for c in row.select('td')]
+        items.append(PrayerTimes(
+            date=datetime.strptime(cells[0], '%d.%m.%Y'),
+            fajr=cells[2],
+            sun=cells[3],
+            dhuhr=cells[4],
+            asr=cells[5],
+            maghrib=cells[6],
+            isha=cells[7],
+        ))
     return items
 
 
 def make_location_url(location_id: int) -> str:
-    return f'https://namazvakitleri.diyanet.gov.tr/en-US/{location_id}'
+    url = f'https://namazvakitleri.diyanet.gov.tr/en-US/{location_id}'
+    return url
 
 
 async def get_prayer_times(location_id: int) -> List[PrayerTimes]:
@@ -117,16 +132,15 @@ async def _get_countries() -> dict:
 
 if __name__ == '__main__':
     import asyncio
-    import core.diyanetdb
 
 
     async def main():
-        with core.diyanetdb.get_connection() as conn:
-            id = core.diyanetdb.get_location_id(conn,
-                                                country_name='AVUSTURYA',
-                                                city_name='PESSENDELLACH')
-        url = make_location_url(id)
-        times = await parse_prayer_times(url)
+        # with core.diyanetdb.get_connection() as conn:
+        #     id = core.diyanetdb.get_location_id(conn,
+        #                                         country_name='AVUSTURYA',
+        #                                         city_name='PESSENDELLACH')
+        # url = make_location_url(id)
+        times = await parse_prayer_times('https://namazvakitleri.diyanet.gov.tr/en-US')
         print(times)
 
 

core/diyanetdb.py

@@ -7,7 +7,7 @@ from pydantic import BaseModel
 
 from app import config
 
-_http = httpx.AsyncClient()
+_http = httpx.AsyncClient(timeout=20, follow_redirects=True)
 
 
 class Location(BaseModel):

requirements.txt

@@ -1,5 +1,6 @@
-bs4==0.0.1
-fastapi==0.47.1
-httpx==0.11.1
-loguru==0.4.1
-uvicorn==0.11.1
+beautifulsoup4==4.10.0
+fastapi==0.74.1
+httpx==0.23.3
+uvicorn==0.17.5
+aiocache==0.11.1
+pydantic==1.10.7