Cleanup

.env.example

@@ -1,3 +1 @@
 APP_DATABASE_URL="postgresql+asyncpg://user:pass@localhost:5432/${DB:-catalogage}"
-
-VITE_API_PORT="3579"

client/vite.config.js

@@ -10,12 +10,9 @@ dotenv.config({
 });
 
 function getProxy() {
+  // Return the proxy to use for the Vite dev server (not used in production).
+  // See: https://vitejs.dev/config/#server-proxy
   const API_PORT = process.env.VITE_API_PORT || "3579";
-  const shouldProxy = Boolean(API_PORT);
-
-  if (!shouldProxy) {
-    return {};
-  }
 
   return {
     // Proxy requests to /api to the local API server.

client/vitest.config.ts

@@ -1,8 +1,10 @@
+/// <reference types="vitest" />
 import path from "path";
+import { defineConfig } from "vite";
 import { svelte } from "@sveltejs/vite-plugin-svelte";
 import { config as baseConfig } from "./vite.config";
 
-const config = {
+export default defineConfig({
   ...baseConfig,
   resolve: {
     alias: {
@@ -21,6 +23,4 @@ const config = {
     globals: true,
     environment: "jsdom",
   },
-};
-
-export default config;
+});

docs/fr/demarrage.md

@@ -104,7 +104,7 @@ Le serveur d'API est configurable à l'aide des variables d'environnement suivan
 | `APP_SERVER_MODE` | Un mode d'opération qui configure Uvicorn en conséquence : <br> - `local` : pour le développement local (_hot reload_ activé, etc) <br> - `live` : pour tout déploiement tel que défini via Ansible (voir [Opérations](./ops.md)) | `local` |
 | `APP_DATABASE_URL` | URL vers la base de données PostgreSQL | `postgresql+asyncpg://localhost:5432/catalogage` |
 | `APP_PORT` | Port du server d'API | `3579` |
-| `VITE_API_PORT` | En développement local uniquement : doit être égal à `APP_PORT`. Pour plus de contexte, voir [#48](https://github.com/etalab/catalogage-donnees/pull/48) | |
+| `VITE_API_PORT` | Doit être égal à `APP_PORT` si jamais ce dernier diffère de sa valeur par défaut, afin de permettre la communication directe entre le frontend et le serveur d'API le cas échéant (SSR, proxy local...). Pour plus de contexte, voir [#48](https://github.com/etalab/catalogage-donnees/pull/48) | `3579` |
 
 Définissez les valeurs spécifiques à votre situation dans un fichier `.env` placé à la racine du projet, que vous pouvez créer à partir du modèle `.env.example`.
 

docs/fr/ops.md

@@ -209,7 +209,7 @@ Il y a probablement soit un problème de configuration de la connexion entre Ngi
 ~/catalogage $ systemctl status supervisor
 ```
 
-* Vérifier l'état du processus serveur (`server` pour Gunicorn, `client` pour le frontend Node) au sein de Supervisor :
+* Vérifier l'état du processus serveur (`server` pour Uvicorn, `client` pour le frontend Node) au sein de Supervisor :
 
 ```
 ~/catalogage $ sudo supervisorctl status server

ops/ansible/roles/web/tasks/node.yml

@@ -3,4 +3,3 @@
     path: "{{ workdir }}/client"
     ci: true
     state: present
-  become: true

ops/ansible/roles/web/templates/.env.j2

@@ -1,5 +1,4 @@
 APP_SERVER_MODE=live
 APP_PORT="{{ api_port }}"
 APP_DATABASE_URL="{{ database_url }}"
-
 VITE_API_PORT="{{ api_port }}"

ops/ansible/roles/web/templates/nginx.conf.j2

@@ -18,6 +18,7 @@ server {
     location / {
         include proxy_params;
         proxy_pass http://client/;
+    }
 
     location /api/ {
         include proxy_params;