search

Purexo / FreshRSS-Notify

Un addon pour firefox, notifiant les articles non lu sur votre instance de FreshRSS
https://addons.mozilla.org/fr/firefox/addon/freshrss-notify/
8 stars 1 forks source link

Considérer les domaines pointant sur le répertoire ./p #1

Closed marienfressinaud closed 8 years ago

marienfressinaud commented 9 years ago

L'extension ne peut pas fonctionner si le (sous-)domaine pour FRSS pointe sur le répertoire ./p. Exemple : http://demo.freshrss.org/i/

Purexo commented 9 years ago

J'ai repris le développement de mon Add-on, J'avais complètement oublié ça.

Je vais réfléchir à ajouter une variable dans les préférences pour définir le path de l'API. Après c'est un cas exceptionnel, et je doute que beaucoup essaient d'utiliser l'api d'une démo avec un arrangement de sous domaines étrange ^^

EDIT : C'est maintenant chose faite, la release arrive bientôt

marienfressinaud commented 9 years ago

Après c'est un cas exceptionnel, et je doute que beaucoup essaient d'utiliser l'api d'une démo avec un arrangement de sous domaines étrange ^^

Ce n'est justement pas censé être exceptionnel, c'est ce qui est recommandé d'un point de vue sécurité :p En fait tu n'as pas besoin d'ajouter un champ supplémentaire, il suffit que l'utilisateur ajoute le /p à l'url tout simplement

Purexo commented 8 years ago

Bon, je verrai ça un autre jour, mais avec les deux champs ça devrait fonctionner. Le je viens de passer plusieurs heures sur de la manipulation DOM "safe" pour pas utiliser de innerHTML pour la review

Je comprends pas encore bien le problème Détaille moi l'url standard pour aller lire les flux (full) l'url pour acceder à l'api (full) Et je verrai quoi faire

marienfressinaud commented 8 years ago

En fait c'est simple :

  1. Tu laisses l'utilisateur saisir l'URL "de base" (exemples : http://exemple.com/FreshRSS/p, http://rss.exemple.com)
  2. À partir de cette URL, tu détermines l'URL de l'API en ajoutant /api/greader.php (ce qui donnera au final : http://exemple.com/FreshRSS/p/api/greader.php, http://rss.exemple.com/api/greader.php)

Donc par rapport à ton code actuel, tu as juste à garder le champ "URL", tu peux supprimer celui d'accès à l'API et tu peux en faire une constante d'une valeur de /api/greader.php.

Purexo commented 8 years ago

Partant du principe, que les gens peuvent configurer leur serveur http comme ils veulent, (voir n'importe comment). Je pense que je vais rester sur 2 champs :

Et les deux champs seraient en url absolu.

Pour ma part, si je devais me configurer des sous domaines pour mon instance de FreshRSS, j'aurais un sous domaine rss. qui pointe vers la partie siteweb de FreshRSS et un sous domaine api.rss. qui pointe vers le fichier api de FreshRSS

je ferrai ça pour la version 2.1.0

Purexo commented 8 years ago

screenshot

Voila : v2.1.0 prête

URL et APIURL sont maintenant dissociés Il faut donc bien penser à les redéfinir