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.

Sign up for GitHub

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

Jump to bottom

Favicons : ne pas dépendre d'un service externe #290

Closed
Alkarex opened this issue Nov 24, 2013 · 7 comments
Closed

Favicons : ne pas dépendre d'un service externe #290

Alkarex opened this issue Nov 24, 2013 · 7 comments
Labels
Not urgent 🐌
Milestone
0.10-dev

Comments

@Alkarex
Copy link
Member

Alkarex commented Nov 24, 2013

Actuellement, FreshRSS utilise http://g.etfv.co
Il serait bien de ne pas dépendre d'un service externe pour récupérer les favicons.
@Alkarex Alkarex mentioned this issue Dec 1, 2013
Closed
Alkarex added a commit that referenced this issue Dec 1, 2013
@Alkarex
Favicons en parallèle
b0cbc6f 
Nouvelle méthode pour afficher les favicons à la demande et en
parallèle.
Déplacement du dossier des favicons sous /data/favicons/
Devrait permettre de fermer le problème d'import OPML
#228
Voir aussi #290
Alkarex added a commit that referenced this issue Dec 1, 2013
@Alkarex
Favicons : image pour 404
0e9ed2a 
#228
#290
Alkarex added a commit that referenced this issue Dec 2, 2013
@Alkarex
Favicons : amélioration
1a27030 
* Évite de faire un accès disque systématique pour vérifier si le .ico
est présent, et ne le fait plus que lors d'un ajout ou rafraîchissement
de flux
* Corrige un bug pour les flux qui n'ont pas de site Web déclaré
* Efface le favicon lorsqu'un flux est supprimé (seulement
individuellement pour l'instant)

Voir aussi #290
@aledeg
Copy link
Member

aledeg commented Mar 5, 2014

Je pense que le remplaçant est tout trouvé :)
http://simplepie.org/wiki/reference/simplepie/get_favicon

@Alkarex
Copy link
Member Author

Alkarex commented Mar 5, 2014

Pas mal :-)
À tester dans ./p/f.php

@aledeg
Copy link
Member

aledeg commented Sep 26, 2014

:( when reading SimplePie code, it looks like getFavicon uses http://g.etfv.co and that it is deprecated.
I think we should find an other alternative.

@marienfressinaud
Copy link
Member

Seems good: https://github.com/ArthurHoaro/favicon :)

This was referenced Dec 16, 2014
Closed
Closed
@qwertygc
Copy link

FreshRSS utilise encore g.etfv.co, hors ce dernier est dans les choux... j'aurai bien dit utilisez http://www.google.com/s2/favicons?domain=example.org, mais c'est remplacer la peste par le choléra..

@marienfressinaud
Copy link
Member

Oui je sais c'est pour ça que j'ai commencé à intégrer le code d'Arthur Hoaro. J'essaye d'avoir un truc fonctionnel d'ici peu. Et on n'utilisera pas le service de Google, c'est certain !

marienfressinaud added a commit that referenced this issue Dec 29, 2014
@marienfressinaud
Add files for the new favicon system
d234304 
Use https://github.com/ArthurHoaro/favicon
See #290
marienfressinaud added a commit that referenced this issue Dec 29, 2014
@marienfressinaud
Change favicon getter behaviour
4f1fc70 
- Use Favicon library
- Fix coding style
- Change default favicon

Fix #290
marienfressinaud added a commit that referenced this issue Dec 29, 2014
@marienfressinaud
Fix typo coding style
ea79466 
See #290
@marienfressinaud
Copy link
Member

Corrigé !

marienfressinaud added a commit that referenced this issue Dec 29, 2014
@marienfressinaud
Fix behaviour when favicon url is not found
7584364 
If url is not find, we copy default favicon into favicon dir so at the next
try, we don't search the url again.

See #290
See #727
@Alkarex Alkarex mentioned this issue Jan 20, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Not urgent 🐌
Projects
None yet
Milestone
0.10-dev
Development

No branches or pull requests
4 participants
@Alkarex @marienfressinaud @aledeg @qwertygc