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

Add BM Lyon #174

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Add BM Lyon #174

merged 1 commit into from
Nov 9, 2023

Conversation

vr
Copy link
Contributor

@vr vr commented Nov 7, 2023

url param is encoded with base64 but include the same code as other one

url param is encoded with base64 but include the same code as other one
@lovasoa
Copy link
Owner

lovasoa commented Nov 8, 2023

Bonjour ! Vous n'avez pas ajouté d'url de proxy dans optional_permissions. Est-ce un oubli ?

@vr
Copy link
Contributor Author

vr commented Nov 9, 2023

Bonjour !

non, il utilise apparement l'url generique nouveau.europresse.com, donc pas de proxy nécessaire ?

@lovasoa lovasoa merged commit 7e342f8 into lovasoa:master Nov 9, 2023
1 check passed
@vr vr deleted the patch-1 branch November 10, 2023 14:17
@vr
Copy link
Contributor Author

vr commented Nov 10, 2023

apparement ca ne marche pas, le bouton n'est pas cliquable, est ce parce que justement il n'y a pas de proxy ?

@lovasoa
Copy link
Owner

lovasoa commented Nov 10, 2023

Oui, effectivement, j'ai été trop rapide à intégrer le changement, ça ne fonctionne pas.

Jusqu'ici tous les partenaires fonctionnaient de l'une des deux manières suivantes:

  • avec un proxy pour l'identification et pour la consultation (l'identification et la consultation se font sur le même nom de domaine, par exemple bm-lyon.fr).
  • sans proxy ni pour l'identification (qui se fait sur une URL du type: https://nouveau.europresse.com/Login/Esidoc?sso_id=qqch) ni pour la consultation (qui se fait sur https://nouveau.europresse.com/).

êtes vous sûrs que vous n'avez aucun moyen

  • de vous identifier directement sur une url du type https://nouveau.europresse.com/Login/Esidoc?sso_id=qqch ?
  • de consulter europresse sur une url du type qqch.bm-lyon.fr ?

@vr
Copy link
Contributor Author

vr commented Nov 13, 2023

non, l'authent ne marche pas en validant l'ip du proxy comme les autres, ni en utilisant l'authentification direct d'europresse, c'est via une validation de referer, on y accède via le site https://www.bm-lyon.fr/les-offres-numeriques-de-la-bibliotheque/article/lire-la-presse-de-chez-vous qui pointe ensuite sur l'url que j'avais spécifié.

En vrai si je choisi "pas d'intermédiaire", ca marche, modulo la partie authentification que je dois gérer manuellement du coup.

@lovasoa
Copy link
Owner

lovasoa commented Nov 13, 2023

Ok, alors il faut changer la logique de demande de permission. Peut-être ajouter un clef dans le manifest à côté de AUTH_URL pour indiquer sur quel domaine demander des permissions dans le cas où ça ne peut pas être détecté automatiquement.

Actuellement c'est cette fonction qui va trouver sur quoi demander une permission: https://github.com/lovasoa/ophirofox/blob/master/ophirofox/content_scripts/config.js#L105-L118

Elle cherche juste dans les permissions optionnelles une URL dont le nom de domaine correspond.

@Write
Copy link
Collaborator

Write commented Dec 22, 2024

@vr Bonjour, pourriez-vous m'envoyer par mail des identifiants que je puisse tester une solution viable ?

github@socialspill.com

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.

3 participants