Repos / pytaku / 595d990c64
commit 595d990c64d4a355019245784e480c0961154a9b
Author: Bùi Thành Nhân <hi@imnhan.com>
Date:   Tue Aug 11 20:25:09 2020 +0700

    remove ensure_session_version thing
    
    I would blow up any client that didn't support cookies, like some
    privacy-concious people and, sigh, facebook opengraph scraper.

diff --git a/pyproject.toml b/pyproject.toml
index ce9d770..dca39b9 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pytaku"
-version = "0.2.16"
+version = "0.2.17"
 description = ""
 authors = ["Bùi Thành Nhân <hi@imnhan.com>"]
 license = "AGPL-3.0-only"
diff --git a/src/pytaku/decorators.py b/src/pytaku/decorators.py
index 7bd09a9..388731f 100644
--- a/src/pytaku/decorators.py
+++ b/src/pytaku/decorators.py
@@ -5,27 +5,6 @@
 from .persistence import read, unread
 
 
-def ensure_session_version(f, CURRENT_VERSION=1):
-    """
-    Increment CURRENT_VERSION to nuke all current sessions.
-    Useful for when I try to change session structure or something. Maybe.
-    """
-
-    @wraps(f)
-    def decorated_function(*args, **kwargs):
-        if (
-            not session.get("version")
-            or not isinstance(session["version"], int)
-            or session["version"] < CURRENT_VERSION
-        ):
-            session.clear()
-            session["version"] = CURRENT_VERSION
-            return redirect("/")
-        return f(*args, **kwargs)
-
-    return decorated_function
-
-
 def require_login(f):
     @wraps(f)
     def decorated_function(*args, **kwargs):
diff --git a/src/pytaku/main.py b/src/pytaku/main.py
index cfa7e78..d55cba4 100644
--- a/src/pytaku/main.py
+++ b/src/pytaku/main.py
@@ -14,7 +14,7 @@
 )
 
 from .conf import config
-from .decorators import ensure_session_version, require_login, toggle_has_read
+from .decorators import require_login, toggle_has_read
 from .persistence import (
     follow,
     get_followed_titles,
@@ -45,7 +45,6 @@
 
 
 @app.route("/")
-@ensure_session_version
 def home_view():
     if session.get("user"):
         return redirect(url_for("follows_view"))
@@ -53,7 +52,6 @@ def home_view():
 
 
 @app.route("/following", methods=["GET"])
-@ensure_session_version
 @require_login
 def follows_view():
     titles = get_followed_titles(session["user"]["id"])
@@ -66,7 +64,6 @@ def follows_view():
 
 
 @app.route("/follow/<site>/<title_id>", methods=["POST"])
-@ensure_session_version
 @require_login
 def follow_view(site, title_id):
     follow(session["user"]["id"], site, title_id)
@@ -74,7 +71,6 @@ def follow_view(site, title_id):
 
 
 @app.route("/unfollow/<site>/<title_id>", methods=["POST"])
-@ensure_session_version
 @require_login
 def unfollow_view(site, title_id):
     unfollow(session["user"]["id"], site, title_id)
@@ -82,14 +78,12 @@ def unfollow_view(site, title_id):
 
 
 @app.route("/logout", methods=["POST"])
-@ensure_session_version
 def logout_view():
     session.pop("user")
     return redirect("/")
 
 
 @app.route("/auth", methods=["GET", "POST"])
-@ensure_session_version
 def auth_view():
     if session.get("user"):
         return redirect(url_for("home_view"))
@@ -173,7 +167,6 @@ def auth_view():
 
 
 @app.route("/m/<site>/<title_id>")
-@ensure_session_version
 @toggle_has_read
 def title_view(site, title_id):
     user = session.get("user", None)
@@ -196,7 +189,6 @@ def title_view(site, title_id):
 
 
 @app.route("/m/<site>/<title_id>/<chapter_id>")
-@ensure_session_version
 @toggle_has_read
 def chapter_view(site, title_id, chapter_id):
     chapter = load_chapter(site, title_id, chapter_id)
@@ -225,7 +217,6 @@ def chapter_view(site, title_id, chapter_id):
 
 
 @app.route("/search")
-@ensure_session_version
 def search_view():
     query = request.args.get("q", "").strip()
     results = {}