Skip to content

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

Open
Gaetanbrl wants to merge 4 commits into
developfrom
issue-398
Open

fix:398 - permet de controler l'utilisateur avant d'ouvrir le mode edition#413
Gaetanbrl wants to merge 4 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
Copy link
Copy Markdown
Collaborator

lecault commented May 11, 2026

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
Copy link
Copy Markdown
Collaborator

lecault commented May 11, 2026

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
Copy link
Copy Markdown
Collaborator

lecault commented May 28, 2026

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
Copy link
Copy Markdown
Member Author

Gaetanbrl commented May 28, 2026

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.

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