From 674baf6954d2a7847a87e8d98dfba888adda586a Mon Sep 17 00:00:00 2001
From: Bruno Charest <bruno.charest@gmail.com>
Date: Thu, 8 Oct 2026 19:00:13 -0400
Subject: [PATCH] fix: og:image et twitter:image en URL absolue (v7.69.1)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Les meta `og:image` / `twitter:image` ajoutés en v7.69.0 pointaient vers une
URL relative (`/static/img/logo-512.jpg`). Les crawlers — LinkedIn, Slack,
Discord, iMessage — ignorent une URL relative : la carte de partage de FlowDeck
était donc vide partout où un lien est collé.

Correction via le motif déjà en place dans `app/templating.py` : un global
Jinja `app_base_url()` construit à partir de `settings.app_base_url`, qui est
LA MÊME source que les liens de partage de pages (`collaboration.py:32`).
Si cette valeur est mal renseignée, ces liens l'étaient déjà — comportement
cohérent plutôt qu'une deuxième source de vérité à maintenir.

Lecture paresseuse de `settings` dans la fonction, à l'image de
`_plugin_enabled` : un import au chargement du module créerait un import
circulaire.

Rendu vérifié sur l'instance déployée :
  og:image" content="http://localhost:8080/static/img/logo-512.jpg

Tests : `test_og_image_url_is_absolute_via_base_url` vérifie le global ET
l'absence de la forme relative dans les deux templates. Piège évité au passage
— la forme absolue CONTIENT l'ancienne en sous-chaîne, donc l'assertion
négative porte sur `content="/static/img/` (le guillemet avant la barre) et
non sur l'URL complète.

pytest 1413 passed / 0 failed · ruff OK · OpenAPI 526 chemins / 7.69.1
e2e 14/14 verts sur l'instance redéployée
---
 CHANGELOG.md                             | 11 +++++++++++
 VERSION                                  |  2 +-
 WORKLOAD.md                              |  2 +-
 app/main.py                              |  2 +-
 app/templates/base.html                  |  4 ++--
 app/templates/landing.html               |  4 ++--
 app/templating.py                        | 12 ++++++++++++
 docs/openapi-v2.json                     |  2 +-
 tests/test_v769_selection_persistence.py | 14 +++++++++++++-
 9 files changed, 44 insertions(+), 9 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2aaa94e..dc5b4e2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,16 @@
 # Changelog - FlowDeck
 
+## v7.69.1 (2026-10-08) — `og:image` en URL absolue
+
+### Fixed
+
+- **`og:image` / `twitter:image` étaient des URL relatives** — les crawlers
+  (LinkedIn, Slack, Discord, iMessage…) ignorent une URL relative : la carte de
+  partage était vide. → global Jinja `app_base_url()` construit à partir de
+  `settings.app_base_url`, **la même source que les liens de partage de pages**
+  (`collaboration.py`) : si elle est mal renseignée, ces liens l'étaient déjà.
+  Lecture paresseuse, comme `_plugin_enabled`, pour éviter l'import circulaire.
+
 ## v7.69.0 (2026-10-08) — La sélection multi-blocs fonctionne enfin (+ identité visuelle)
 
 ### Fixed
diff --git a/VERSION b/VERSION
index 0f4b2e9..0692f00 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-7.69.0
+7.69.1
diff --git a/WORKLOAD.md b/WORKLOAD.md
index 312aab8..4fae1fb 100644
--- a/WORKLOAD.md
+++ b/WORKLOAD.md
@@ -1,6 +1,6 @@
 # WORKLOAD — FlowDeck Notion Clone
 
-> **Début**: 2026-07-08 | **Version**: v7.69.0 (sélection multi-blocs fonctionnelle : drag souris + Maj+↓, clic droit qui la préserve) | **Statut**: EN COURS 🔄
+> **Début**: 2026-07-08 | **Version**: v7.69.1 (sélection multi-blocs fonctionnelle : drag souris + Maj+↓ + Ctrl+C/X, clic droit qui préserve le surlignage ; identité visuelle logo/bannière) | **Statut**: EN COURS 🔄
 > **Cible**: parité Notion + intégration forge · **Follow-ups v7.3 livrés**: sidebar teamspaces, notif `page.updated`, charts `number` + dashboards multi-DB, unfurl forge, UI Settings → Audit — voir `ROADMAP.md § v7.3.0`
 
 ## Avancement Global
diff --git a/app/main.py b/app/main.py
index 64ca72a..e3cf24a 100644
--- a/app/main.py
+++ b/app/main.py
@@ -186,7 +186,7 @@ async def lifespan(_app: FastAPI):
 
 app = FastAPI(
     title="FlowDeck",
-    version="7.69.0",
+    version="7.69.1",
     docs_url="/docs",
     redoc_url="/redoc",
     lifespan=lifespan,
diff --git a/app/templates/base.html b/app/templates/base.html
index 67fc00a..a48f7b8 100644
--- a/app/templates/base.html
+++ b/app/templates/base.html
@@ -14,11 +14,11 @@
   <meta property="og:site_name" content="FlowDeck">
   <meta property="og:title" content="FlowDeck — votre espace Notion auto-hébergé">
   <meta property="og:description" content="Éditeur de blocs, Kanban, bases de données et intégration Gitea/GitHub, sur votre propre serveur.">
-  <meta property="og:image" content="/static/img/logo-512.jpg">
+  <meta property="og:image" content="{{ app_base_url() }}/static/img/logo-512.jpg">
   <meta name="twitter:card" content="summary_large_image">
   <meta name="twitter:title" content="FlowDeck — votre espace Notion auto-hébergé">
   <meta name="twitter:description" content="Éditeur de blocs, Kanban, bases de données et intégration Gitea/GitHub.">
-  <meta name="twitter:image" content="/static/img/logo-512.jpg">
+  <meta name="twitter:image" content="{{ app_base_url() }}/static/img/logo-512.jpg">
   <meta name="apple-mobile-web-app-capable" content="yes">
   <meta name="apple-mobile-web-app-status-bar-style" content="black-translucent">
   <link rel="stylesheet" href="/static/css/app.css?v={{ asset_version }}">
diff --git a/app/templates/landing.html b/app/templates/landing.html
index 04a26e1..5bfacfc 100644
--- a/app/templates/landing.html
+++ b/app/templates/landing.html
@@ -17,11 +17,11 @@
   <meta property="og:site_name" content="FlowDeck">
   <meta property="og:title" content="FlowDeck — votre espace Notion auto-hébergé">
   <meta property="og:description" content="Éditeur de blocs, Kanban, bases de données et intégration Gitea/GitHub, sur votre propre serveur.">
-  <meta property="og:image" content="/static/img/logo-512.jpg">
+  <meta property="og:image" content="{{ app_base_url() }}/static/img/logo-512.jpg">
   <meta name="twitter:card" content="summary_large_image">
   <meta name="twitter:title" content="FlowDeck — votre espace Notion auto-hébergé">
   <meta name="twitter:description" content="Éditeur de blocs, Kanban, bases de données et intégration Gitea/GitHub.">
-  <meta name="twitter:image" content="/static/img/logo-512.jpg">
+  <meta name="twitter:image" content="{{ app_base_url() }}/static/img/logo-512.jpg">
   <style>
     :root {
       --bg: #191919;
diff --git a/app/templating.py b/app/templating.py
index 55f3a5a..099320e 100644
--- a/app/templating.py
+++ b/app/templating.py
@@ -79,6 +79,18 @@ ENV.globals["csp_nonce"] = lambda: CSP_NONCE.get()
 ENV.globals["csrf_token"] = lambda: CSRF_TOKEN.get()
 # Plugins activés : conditionne les blocs de UI rendus côté serveur.
 ENV.globals["plugin_enabled"] = lambda slug: _plugin_enabled(slug)
+# v7.69 : URL de base absolue pour og:image / twitter:image — les crawlers
+# ignorent les URL relatives. Lecture paresseuse comme _plugin_enabled
+# (évite l'import circulaire au chargement du module). Même source que les
+# liens de partage de pages (collaboration.py) : si elle est mal renseignée,
+# ces liens le sont déjà.
+ENV.globals["app_base_url"] = lambda: _app_base_url()
+
+
+def _app_base_url() -> str:
+    from app.config import settings
+
+    return (settings.app_base_url or "").rstrip("/")
 
 
 def _plugin_enabled(slug: str) -> bool:
diff --git a/docs/openapi-v2.json b/docs/openapi-v2.json
index 8cfc007..d663b77 100644
--- a/docs/openapi-v2.json
+++ b/docs/openapi-v2.json
@@ -2,7 +2,7 @@
   "openapi": "3.1.0",
   "info": {
     "title": "FlowDeck",
-    "version": "7.69.0"
+    "version": "7.69.1"
   },
   "paths": {
     "/auth/register": {
diff --git a/tests/test_v769_selection_persistence.py b/tests/test_v769_selection_persistence.py
index 15beecd..03d1a3c 100644
--- a/tests/test_v769_selection_persistence.py
+++ b/tests/test_v769_selection_persistence.py
@@ -123,10 +123,22 @@ class TestBranding:
 
     def test_base_head_declares_open_graph(self):
         base = (ROOT / "app" / "templates" / "base.html").read_text(encoding="utf-8")
-        assert 'property="og:image" content="/static/img/logo-512.jpg"' in base
+        # URL ABSOLUE : les crawlers ignorent les URL relatives
+        assert 'content="{{ app_base_url() }}/static/img/logo-512.jpg"' in base
         assert 'name="twitter:card" content="summary_large_image"' in base
         assert '<meta name="description"' in base
 
+    def test_og_image_url_is_absolute_via_base_url(self):
+        # le global expose la même source que les liens de partage de pages
+        templating = (ROOT / "app" / "templating.py").read_text(encoding="utf-8")
+        assert 'ENV.globals["app_base_url"]' in templating
+        for tpl in ("base.html", "landing.html"):
+            src = (ROOT / "app" / "templates" / tpl).read_text(encoding="utf-8")
+            assert 'content="{{ app_base_url() }}/static/img/logo-512.jpg"' in src, tpl
+            # forme RELATIVE interdite (le guillemet avant la barre est ce qui
+            # distingue : la bonne valeur contient l'ancienne en sous-chaîne)
+            assert 'content="/static/img/' not in src, tpl
+
     def test_landing_shows_logo_and_banner(self):
         landing = (ROOT / "app" / "templates" / "landing.html").read_text(encoding="utf-8")
         # le logo remplace l'icône générique dans la barre de navigation