Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manque pramamétrage du proxy ajax dans mviewerstudio #186

Closed
spelhate opened this issue Mar 24, 2023 · 21 comments
Closed

Manque pramamétrage du proxy ajax dans mviewerstudio #186

spelhate opened this issue Mar 24, 2023 · 21 comments
Assignees
Labels
bug majeur Annoying but not obstructive bug
Milestone

Comments

@spelhate
Copy link
Collaborator

Actuellement, mviewerstudio génère des configurations xml avec un proxy renseigné.

<proxy url="/proxy/?url="/>

Ce paramétrage devrait être disponible via mviewerstudio (Champ texte)

@spelhate
Copy link
Collaborator Author

Comment peut-on désactiver l'utilisation du proxy avec mviewer ?
Si au moment de l'enregistrement, le champ proxy est vide, studio affecte la valeur définie par défaut. Ce qui ne convient pas dans ce cas.

Ce qu'on souhaite :

  • Dans Studio, affichage par défaut du proxy tel que défini dans le config.json de studio
  • Si l'utilisateur efface cette valeur, studio enregistre la conf avec <proxy url=""/>

@Gaetanbrl
Copy link
Contributor

@Gaetanbrl
Copy link
Contributor

Gaetanbrl commented Apr 18, 2023

  • rajouter au code ci-dessous un else pour définir un proxy vide :

https://github.com/jdev-org/mviewerstudio/blob/develop-meg/js/mviewerstudio.js#L707

A rajouter :

else {<proxy url = "">}

Traité dans branche issue-160 :

https://github.com/jdev-org/mviewerstudio/blob/issue-160/js/mviewerstudio.js#L705

@lecault
Copy link
Contributor

lecault commented May 3, 2023

Je viens de tester si on vide le proxy c'est ok.

  • Dans Studio, affichage par défaut du proxy tel que défini dans le config.json de studio

Par contre pour ce point là je ne suis pas sur. Par défaut lors de la création d'une nouvelle application, le proxy est vide :
<proxy url=""/>

@Gaetanbrl
Copy link
Contributor

Par contre pour ce point là je ne suis pas sur. Par défaut lors de la création d'une nouvelle application, le proxy est vide

Quel est l'état de la valeur proxy dans la config json du studio ?

@lecault
Copy link
Contributor

lecault commented May 3, 2023

c'est chez vous je ne sais pas :)

@Gaetanbrl
Copy link
Contributor

je ne suis pas sur de comprendre ta remarque du coup...
Chez nous c'est vide. Sinon tu verrai la valeur dans l'input je crois.. et sinon ca ne marcherait pas, vu que c'était le problème avec une valeur non vide dans le config json ;)

@lecault
Copy link
Contributor

lecault commented May 3, 2023

Tu as raison suis perdu avec ce proxy..

@spelhate
Copy link
Collaborator Author

spelhate commented May 4, 2023

Le comportement attendu :

  • définition dans config.json d'un proxy (utilisé à la fois pour studio et pour la configuration xml par défaut d'une application mviewer)
  • Si défini dans config.json, affecte la valeur du proxy au formulaire studio. ex "monproxy?url=" et produit un xml mviewer avec <proxy url="monproxy?url="/>
  • si non défini dans config.json ou si effacé dans le formulaire studio => produit un xml avec <proxy url=""/>

@Gaetanbrl
Copy link
Contributor

Gaetanbrl commented May 9, 2023

@spelhate @lecault issue à ferme je pense (si tests OK dans branche suivante).

@spelhate
Copy link
Collaborator Author

Ce n'est pas complètement ok car si un proxy est renseigné dans le config.json, ce proxy n'est pas proposé par défaut dans l'ihm de studio :
image
A noter que la publication produit malgré tout un xml avec le proxy renseigné à partir du config.json.
On souhaite une cohérence entre l'IHM et le xml produit.

@Gaetanbrl
Copy link
Contributor

On souhaite une cohérence entre l'IHM et le xml produit.

Vu, c'est noté.

@Gaetanbrl
Copy link
Contributor

@spelhate j'avais en réalité déjà apporté la modification sur ma branche issue-160 :

https://github.com/jdev-org/mviewerstudio/blob/issue-160/lib/mv.js#L1184-L1186

A reporter dans issue-161-rebase donc.

@Gaetanbrl
Copy link
Contributor

A reporter dans issue-161-rebase donc.

En fait j'ai un doute...

@Gaetanbrl
Copy link
Contributor

@spelhate correctif à tester sur branche issue-161-rebase et visible sur https://gis.jdev.fr/mviewerstudio/index.html#

@Gaetanbrl Gaetanbrl added the ready for test Wait test label May 10, 2023
@Gaetanbrl
Copy link
Contributor

En attente de retour de @spelhate ou @lecault

@Gaetanbrl Gaetanbrl added the majeur Annoying but not obstructive bug label May 19, 2023
@Gaetanbrl Gaetanbrl assigned spelhate and lecault and unassigned Gaetanbrl and Agath21 May 19, 2023
@spelhate
Copy link
Collaborator Author

Le comportement attendu :

* définition dans config.json d'un proxy (utilisé à la fois pour studio et pour la configuration xml par défaut d'une application mviewer)

* Si défini dans config.json, affecte la valeur du proxy au formulaire studio. ex` "monproxy?url="` et produit un xml mviewer avec  `<proxy url="monproxy?url="/>`

* si non défini dans config.json ou si effacé dans le formulaire studio => produit un xml avec `<proxy url=""/>`

Le dernier point ne fonctionne pas. On a toujours le proxy défini dans le config.json.
Testé sur branche develop-meg

@Gaetanbrl Gaetanbrl removed the ready for test Wait test label May 22, 2023
@Gaetanbrl Gaetanbrl assigned Gaetanbrl and unassigned spelhate and lecault May 22, 2023
@Gaetanbrl
Copy link
Contributor

Le comportement attendu :

* définition dans config.json d'un proxy (utilisé à la fois pour studio et pour la configuration xml par défaut d'une application mviewer)

* Si défini dans config.json, affecte la valeur du proxy au formulaire studio. ex` "monproxy?url="` et produit un xml mviewer avec  `<proxy url="monproxy?url="/>`

* si non défini dans config.json ou si effacé dans le formulaire studio => produit un xml avec `<proxy url=""/>`

Le dernier point ne fonctionne pas. On a toujours le proxy défini dans le config.json. Testé sur branche develop-meg

OK à corriger.

@Gaetanbrl
Copy link
Contributor

Gaetanbrl commented May 22, 2023

Cas de test :

  • Créer une app et voir la valeur du proxy reprise depuis le fichier config.json - Test OK

image

  • Sauvegarder et voir dans le XML la valeur par défaut reprise du config.json - Test OK

<proxy url="proxy/?url="/>

  • Changer la valeur et voir cette nouvelle valeur dans le XML - Test OK

image

<proxy url="nouveauproxy/?url="/>

  • Supprimer la valeur du champ et voir la bonne valeur vide dans le XML à la sauvegarde - Test OK

image

<proxy url=""/>

  • Ouvrir un XML sans proxy et le sauvegarder -> Voir que la valeur est la même qu'en entrée - Test OK

Entrée :
<proxy url=""/>
XML sauvegardé :

<proxy url=""/>

  • Ouvrir un XML sans proxy et rajouter une valeur de proxy manuellement avant de le sauvegarder -> Voir que la valeur du proxy saisie est la même que celle sauvegardée - Test OK

Champ :

image

XML sauvegardé :

<proxy url="abcdef/proxy?url="/>

  • Ouvrir un XML avec une valeur de proxy et sauvegarder le XML (clic sur le bouton "sauvegarder") -> Voir que la valeur du proxy est chargée dans le champ "URL du proxy" - Test OK

image

  • Ouvrir un XML avec une valeur de proxy et sauvegarder le XML (clic sur le bouton "sauvegarder") -> Voir que la valeur du proxy a bien été chargée dans le champ "URL du proxy". A la sauvegarde, voir que la valeur du proxy est la même que celle du XML importé - Test OK

<proxy url="/xyz?url="/>

@lecault
Copy link
Contributor

lecault commented May 24, 2023

C'est ok pour moi

@spelhate
Copy link
Collaborator Author

Idem

Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue May 25, 2023
@Gaetanbrl Gaetanbrl added this to the MEGALIS - JDEV milestone Jun 16, 2023
Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug majeur Annoying but not obstructive bug
Projects
Development

No branches or pull requests

4 participants