schoolapp-api 2.1.3__tar.gz → 2.1.5__tar.gz

{schoolapp_api-2.1.3/schoolapp_api.egg-info → schoolapp_api-2.1.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: schoolapp-api
-Version: 2.1.3
+Version: 2.1.5
 Summary: Professional client for interacting with the School App API
 Author: Aferiad Kamal
 License: MIT

{schoolapp_api-2.1.3 → schoolapp_api-2.1.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "schoolapp-api"
-version = "2.1.3"
+version = "2.1.5"
 description = "Professional client for interacting with the School App API"
 readme = "README.md"
 requires-python = ">=3.7"

{schoolapp_api-2.1.3 → schoolapp_api-2.1.5}/schoolapp_api/auth.py

@@ -70,19 +70,30 @@ class AuthManager:
         return self.logged_in
     
     def check_session(self):
-        """
-        Confirm whether the session is still valid.
-        """
-        code, url, content = self.http_client.get(self.login_url)
+        """Confirm whether the session is still valid by visiting the index page"""
+        try:
+            from schoolapp_api.constants import INDEX_URL
+            code, url, content = self.http_client.get(INDEX_URL)
 
-        # If server redirects us back to login, session is dead
-        if url and "/login" in url:
+            # If url is None (exception) or it contains "/login", session is dead
+            if not url or "/login" in url.lower():
+                logger.warning(f"Session check: Redirected to {url or 'None'} - session invalid")
+                self.logged_in = False
+                return False
+            
+            # If content is very short or contains login-like keywords, it's also dead
+            if content and (len(content) < 1000 or "login" in content.lower()[:500]):
+                logger.warning("Session check: Content looks like a login page - session invalid")
+                self.logged_in = False
+                return False
+
+            self.logged_in = True
+            return True
+        except Exception as e:
+            logger.error(f"Error during check_session: {e}")
             self.logged_in = False
             return False
 
-        self.logged_in = True
-        return True
-
     def refresh_csrf_from_url(self, url):
         """Fetch a fresh CSRF token from a specific page"""
         try:

{schoolapp_api-2.1.3 → schoolapp_api-2.1.5}/schoolapp_api/http_client.py

@@ -24,13 +24,9 @@ class HTTPClient:
     def __init__(self, base_url):
         self.base_url = base_url
         self.cookie_jar = http.cookiejar.CookieJar()
-        opener = urllib.request.build_opener(
-            urllib.request.HTTPCookieProcessor(self.cookie_jar),
-            NoRedirectHandler()
+        self.opener = urllib.request.build_opener(
+            urllib.request.HTTPCookieProcessor(self.cookie_jar)
         )
-        urllib.request.install_opener(opener)
-        # opener = urllib.request.build_opener(urllib.request.HTTPCookieProcessor(self.cookie_jar))
-        # urllib.request.install_opener(opener)
         self.headers = DEFAULT_HEADERS.copy()
     
     def get(self, url, params=None):
@@ -43,17 +39,21 @@ class HTTPClient:
 
             req = urllib.request.Request(url, headers=self.headers, method="GET")
 
-            with urllib.request.urlopen(req) as response:
+            with self.opener.open(req, timeout=15) as response:
                 code = response.getcode()
                 response_url = response.geturl()
                 content = response.read().decode("utf-8")
                 return code, response_url, content
 
         except urllib.error.HTTPError as e:
-            logger.error(f"HTTP Error {e.code}: {e.reason}")
-            return e.code, None, None
+            try:
+                content = e.read().decode("utf-8")
+            except:
+                content = None
+            logger.error(f"HTTP Error {e.code} on {url}: {e.reason}")
+            return e.code, e.url, content
         except Exception as e:
-            logger.error(f"GET Error: {e}")
+            logger.error(f"GET Error on {url}: {e}")
             return None, None, None
     
     def post(self, url, data, referer=None):
@@ -66,14 +66,18 @@ class HTTPClient:
             if referer:
                 req.add_header('Referer', referer)
             
-            with urllib.request.urlopen(req) as response:
+            with self.opener.open(req, timeout=15) as response:
                 code = response.getcode()
                 response_url = response.geturl()
                 content = response.read().decode('utf-8')
                 return code, response_url, content
         except urllib.error.HTTPError as e:
-            logger.error(f"HTTP Error {e.code}: {e.reason}")
-            return e.code, None, None
+            try:
+                content = e.read().decode("utf-8")
+            except:
+                content = None
+            logger.error(f"HTTP Error {e.code} on {url}: {e.reason}")
+            return e.code, e.url, content
         except Exception as e:
-            logger.error(f"POST Error: {e}")
+            logger.error(f"POST Error on {url}: {e}")
             return None, None, None

{schoolapp_api-2.1.3 → schoolapp_api-2.1.5/schoolapp_api.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: schoolapp-api
-Version: 2.1.3
+Version: 2.1.5
 Summary: Professional client for interacting with the School App API
 Author: Aferiad Kamal
 License: MIT