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

Université Paul-Valéry Montpellier 3 #68

Merged
merged 3 commits into from
Aug 21, 2023
Merged

Université Paul-Valéry Montpellier 3 #68

merged 3 commits into from
Aug 21, 2023

Conversation

scdi-montpellier
Copy link
Contributor

No description provided.

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Bonjour et bienvenue, @scdi-montpellier

Il manque une URL en haut du fichier, qui commence par https://nouveau-europresse-com.

Pour vous c'est peut-être https://nouveau-europresse-com.ezpupv.scdi-montpellier.fr ?

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Je l'ai ajoutée.

Une version de test de l'extension a été construite avec les changements que vous proposez.

Pour tester l'extension, il faut décompresser le fichier zip joint, et charger le fichier qui est à l'intérieur (qui est un autre zip) comme une extension (sur firefox, dans about:debugging#/runtime/this-firefox, en cliquant sur "charger une extension temporaire").

image

image

image

Ensuite il faut sélectionner votre université dans les préférences de l'extension

image

puis ouvrir un article payant, par exemple https://www.lemonde.fr/planete/article/2023/08/18/une-canicule-tardive-s-installe-en-france-symptome-du-rechauffement-climatique_6185847_3244.html

image

Et cliquer sur le lien "lire sur europresse".

@scdi-montpellier
Copy link
Contributor Author

Bonjour,
Désolé l'url manquante est : https://nouveau-europresse-com.ezpupv.scdi-montpellier.fr/
pour l'Université Paul-Valéry Montpellier 3
Merci

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Oui, je l'ai ajoutée. Est-ce que vous pouvez tester ?

@lovasoa lovasoa changed the base branch from master to next August 21, 2023 14:31
@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Les URLs me semblent cohérentes. Je l'intègre dans la prochaine version. Si cela ne fonctionne pas, n'hésitez pas à le signaler ici.

@lovasoa lovasoa merged commit 7de7ea4 into lovasoa:next Aug 21, 2023
1 check passed
lovasoa added a commit that referenced this pull request Aug 21, 2023
* Université de Poitiers (#67)

* Université de Poitiers

* Update manifest.json

* Update README.md

---------

Co-authored-by: Ophir LOJKINE <[email protected]>
Co-authored-by: Ophir LOJKINE <[email protected]>

* Université Paul-Valéry Montpellier 3 (#68)

* Université Paul-Valéry Montpellier 3

* Update manifest.json

* Update README.md

---------

Co-authored-by: Ophir LOJKINE <[email protected]>

---------

Co-authored-by: leam-poitiers <[email protected]>
Co-authored-by: SCDI Montpellier <[email protected]>
@scdi-montpellier
Copy link
Contributor Author

scdi-montpellier commented Aug 21, 2023

Le navigateur râle pour un problème de sécurité au niveau du certificat ssl pendant la redirection car le certificats ssl n'est pas valide pour https://ezpupv.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1
Notre certificat n'est valide que pour un sous-domaine de ezpupv.scdi-montpellier.fr
exemple valide : https://login.ezpupv.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1

J'ai visiblement le même problème pour l'Université de Montpellier : https://ezpum.scdi-montpellier.fr (j'administre les machines virtuelles des ezproxy des 2 universités de Montpellier, mais je ne suis pas le principal administrateur des EZproxy).

Peut-être faut-il rajouter "login." devant les URL des ezproxy ? soit :
https://login.ezpupv.scdi-montpellier.fr
https://login.ezpum.scdi-montpellier.fr

Cordialement,

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

est-ce que vous pouvez me faire un screenshot?

Dans mon navigateur, https://ezpupv.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1 charge sans alerte de sécurité.

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Ah, effectivement, je vois le problème sur Chrome (mais pas sur Firefox).

https://www.ssllabs.com/ssltest/analyze.html?d=ezpum.scdi%2dmontpellier.fr&ignoreMismatch=on&latest

Je peux re-changer le https en http dans l'extension, mais ce n'est pas une super bonne pratique, surtout pour un site où ils faut entrer un mot de passe. Si vous avez accès à la machine, peut-être que vous pouvez faire émettre un certificat pour le domaine?

@scdi-montpellier
Copy link
Contributor Author

J'ai bien le problème sur mon firefox :
image
D'habitude on ne passe jamais par cette URL (https://ezpupv.scdi-montpellier.fr/) pour accéder à une ressource, il y a toujours un sous-domaine (https://xxxxxx.ezpupv.scdi-montpellier.fr/), donc ça ne pose pas de problème.
Je vais voir si je peux regénérer un certificat compatible.

@lovasoa
Copy link
Owner

lovasoa commented Aug 21, 2023

Mais est-ce que
https://login.ezpupv.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1

est capable de faire l'authentification depuis zéro (dans une fenêtre de navigation privée) et de rediriger automatiquement vers la page d'accueil d'Europresse ?

@scdi-montpellier
Copy link
Contributor Author

@scdi-montpellier
Copy link
Contributor Author

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

Je viens de faire le changement, ophirofox redirige maintenant vers https://login.ezpupv.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1

@scdi-montpellier
Copy link
Contributor Author

Merci, j'ai mis à jour l'extension mais ça redirige toujours vers l'URL sans le "login." avec problème de certificat, il faut attendre ?

et j'ai la même erreur de certificat pour l'Université de Montpellier (ezpum) :
image
peut-être rajouter aussi le "login." dans l'URL ?

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

Vous avez bien la version 2.1.16926.51083, publiée il y a 30 minutes ?

Le lien de téléchargement est ici: https://github.com/lovasoa/ophirofox/releases/tag/v2.1.16926.51083

@scdi-montpellier
Copy link
Contributor Author

dans manifest.json, je vois aussi qu'une URL n'est pas bonne, il y 2 fois ezpum (il devrait y avoir 1 ezpum et 1 ezpupv ?) :
{
"matches": [
"https://nouveau.europresse.com/",
"https://nouveau-europresse-com.proxy.rubens.ens.fr/
",
"https://nouveau-europresse-com.rp1.ensam.eu/",
"https://nouveau-europresse-com.ezproxy.universite-paris-saclay.fr/
",
"https://nouveau-europresse-com.ezpaarse.univ-paris1.fr/",
"https://nouveau-europresse-com.accesdistant.bu.univ-paris8.fr/
",
"https://nouveau-europresse-com.docelec.insa-lyon.fr/",
"https://nouveau-europresse-com.bnf.idm.oclc.org/
",
"https://nouveau-europresse-com.docelec.univ-lyon1.fr/",
"https://nouveau-europresse-com.ezpum.scdi-montpellier.fr/
",
"https://nouveau-europresse-com.sid2nomade-2.grenet.fr/",
"https://nouveau-europresse-com.ezpum.scdi-montpellier.fr/
",
"https://nouveau-europresse-com.ezproxy.u-bordeaux-montaigne.fr/",
"https://nouveau-europresse-com.docelec.u-bordeaux.fr/
",
"https://nouveau-europresse-com.proxy.unice.fr/",
"https://nouveau-europresse-com.acces.bibliotheque-diderot.fr/
",
"https://nouveau-europresse-com.scd-proxy.uha.fr/",
"https://nouveau-europresse-com.ezproxy.u-paris.fr/
",
"https://nouveau-europresse-com.scd1.univ-fcomte.fr/",
"https://nouveau-europresse-com.rproxy.univ-pau.fr/
",
"https://nouveau-europresse-com.ezproxy.univ-artois.fr/",
"https://nouveau-europresse-com.doc-elec.univ-lemans.fr/
",
"https://nouveau-europresse-com.lama.univ-amu.fr/",
"https://nouveau-europresse-com.portail.psl.eu/
",
"https://nouveau-europresse-com.passerelle.univ-rennes1.fr/",
"https://nouveau-europresse-com.ressources-electroniques.univ-lille.fr/
",
"https://nouveau-europresse-com.acces-distant.sciencespo.fr/",
"https://nouveau-europresse-com.ressources.univ-poitiers.fr/
",
"https://nouveau-europresse-com.bibelec.univ-lyon2.fr/",
"https://nouveau-europresse-com.gorgone.univ-toulouse.fr/
"
],

@scdi-montpellier
Copy link
Contributor Author

j'ai la version 2.1.16926.56045

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

c'est pas la bonne, donc :)

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

dans manifest.json, je vois aussi qu'une URL n'est pas bonne, il y 2 fois ezpum (il devrait y avoir 1 ezpum et 1 ezpupv ?) :

ok, j'ai ajouté ezpupv. Nouvelle version: https://github.com/lovasoa/ophirofox/releases/tag/v2.1.16926.53455

@scdi-montpellier
Copy link
Contributor Author

ok j'essaie de mettre la bonne version et je vous dis si ça marche :)

@scdi-montpellier
Copy link
Contributor Author

Alors c'est tout bon pour ezpupv (Université Paul-Valéry Montpellier 3) : plus de problème de certificat et j'arrive bien à la ressource proxifiée :)
Pour ezpum (Université de Montpellier), ça marche (accès à la ressource proxifiée) mais j'ai une alerte sur le certificat (une fois outre-passée, le navigateur n'affiche plus l'alerte...). Je pense qu'il faut rajouter le "login." dans l'URL de connexion à l'ezpum :
https://login.ezpum.scdi-montpellier.fr/login?url=https://nouveau.europresse.com/access/ip/default.aspx?un=MontpellierT_1

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

ok, c'est corrigé

@scdi-montpellier
Copy link
Contributor Author

Est-ce possible d'avoir une nouvelle release avec ce changement afin que mes collègues testent l'extension ?
Merci pour votre réactivité.

@scdi-montpellier
Copy link
Contributor Author

J'ai testé la version 2.1.16926.54767 et tout est parfait.
Merci

@lovasoa
Copy link
Owner

lovasoa commented Aug 22, 2023

Merci à vous

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants