Skip to content

fix:398 - permet de controler l'utilisateur avant d'ouvrir le mode edition - #413

Merged
lecault merged 5 commits into
developfrom
issue-398
Jul 1, 2026
Merged

fix:398 - permet de controler l'utilisateur avant d'ouvrir le mode edition#413
lecault merged 5 commits into
developfrom
issue-398

Conversation

@Gaetanbrl

Copy link
Copy Markdown
Member

Cette PR corrige #398 en utilisant le backend pour comparer l'utilisateur connecté et l'utilisateur propriétaire de la carte.

La comparaison par le backend permet d'éviter de modifier la valeur d'une condition via l'interface debbug du navigateur.

@Gaetanbrl Gaetanbrl self-assigned this Apr 17, 2026
@Gaetanbrl Gaetanbrl added 🔔 bug 🚩 Critique Prevents proper functioning or crash ✅ ready for test Wait test labels Apr 27, 2026
@lecault

lecault commented May 11, 2026

Copy link
Copy Markdown
Collaborator

KO pour moi. Si je bascule sur cette branche, je ne peux pas modifier un projet créé dans une version antérieure. Testé sur develop et master.

@Gaetanbrl

Copy link
Copy Markdown
Member Author

créé dans une version antérieure

Est-ce que tu peux m'en dire plus stp ?

@lecault

lecault commented May 11, 2026

Copy link
Copy Markdown
Collaborator

Quand je me branche sur cette PR, je ne peux pas éditer une application :

image

@Gaetanbrl

Copy link
Copy Markdown
Member Author

Je viens de tester sur notre instance gis.jdev.fr/mviewerstudio/ et j'arrive bien à lister les projets précédents ainsi que les modifier.

@Gaetanbrl

Copy link
Copy Markdown
Member Author

Au passage, cette erreur intervient quand ton mviewer n'est pas accessible. Il faudrait que tu vérifies ta configuration dans static/config.json.

@lecault

lecault commented May 28, 2026

Copy link
Copy Markdown
Collaborator

J'ai toujours le souci.
Je suis dans mon dossier mviewerstudio sur develop => ça marche
Je change de branche vers issue-398 et je redémarre le service => ça bug
Je suis branché sur une instance GeOrchestra.

Voici la requête en erreur sur la branche issue-398 : https://kartenn.region-bretagne.fr/mvs/api/app/load?url=https%3A%2F%2Fkartenn.region-bretagne.fr%2Fkartoviz%2Fapps%2Fstore%2Frb%2F8c1787b835c7%2Fwooww2.xml

Il utilise le nom de l'app au lieu de l'id j'ai l'impression : https://kartenn.region-bretagne.fr/kartoviz/apps/store/rb/8c1787b835c7/8c1787b835c7.xml

@Gaetanbrl

Copy link
Copy Markdown
Member Author

Il utilise le nom de l'app au lieu de l'id j'ai l'impression

Ca me semble lié à #417 .

Je viens de faire un rebase.

@Gaetanbrl

Gaetanbrl commented May 28, 2026

Copy link
Copy Markdown
Member Author

Vu avec @lecault, l'erreur rencontré dans la plateforme kartenn est la suivante :

{"name": "Bad Request", "description": "Could not retrieve XML: HTTPSConnectionPool(host='kartenn.region-bretagne.fr', port=443): Max retries exceeded with url: /kartoviz/apps/store/rb/8c1787b835c7/8c1787b835c7.xml (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1137)')))"}

SSLCertVerificationError:
certificate verify failed: self signed certificate in certificate chain

Cette erreur est dû au changement apporté par cette PR : récupérer le XML dans le backend, que si l'utilisateur à les droits (avec vérification dans le backend selon le mode d'authent).

C'est ce code qui réalise l'appel (GET) :

response = requests.get(
url,
timeout=timeout,
headers={
"User-Agent": "Kartenn-MVS/1.0",
"Accept": "application/xml,text/xml,*/*",
},
allow_redirects=True,
)

"User-Agent": "Kartenn-MVS/1.0", n'est pas obligatoire

Si on rajoute le paramètre verify=False, il n'y a plus l'erreur.

Le problème semble donc côté Région Bretagne / certificat.

@lecault

lecault commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Résolu côté région :

  • modification du fichier /.venv/lib/python3.9/site-packages/certifi/cacert.pem
  • ajout de la root CA région

TODO : à ajouter dans la doc

@Gaetanbrl

Copy link
Copy Markdown
Member Author

TODO : à ajouter dans la doc

je te laisse rajouter cette partie, étant donné que vous venez de résoudre :)

@lecault

lecault commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Je le mets dans les notes de migration 4.3 vers 4.4 ?

@lecault lecault added this to the 4.4 milestone Jul 1, 2026
@lecault
lecault merged commit 95412db into develop Jul 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔔 bug 🚩 Critique Prevents proper functioning or crash ✅ ready for test Wait test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants