Rend les forum https friendly

[artragis]

Q	R
Correction de bugs ?	oui
Nouvelle Fonctionnalité ?	non
Tickets (issues) concernés	-

mais cette PR permet de rendre la page de listing des forum https friendly.

J'ajouterai qu'il est fortement probable que cette ligne puisse être totalement supprimée puisque l'avatar n'est jamais affiché.

J'ajouterai que selon la décision de @SpaceFox et @GerardPaligot ou @pierre-24 il est probable que cette PR passe en hotfix.

si vous désirez, je peux faire une commande qui permettra de nettoyer les messages du forum de toute trace d'image chargée en https.

QA

Assurez vous qu'un utilisateur possède un avatar qui ne soit pas un gravatar.
Ensuite, allez sur les forums et vérifiez que lorsqu'il poste un message, cela ne génère pas de <img src="http://whatever"/>.

[Situphen]

Par contre si on navigue en HTTPS et que certaines images ne sont disponibles qu'en HTTP, elles ne seront pas affichées, c'est ça ?

[artragis]

Je ne parle que des images issues de zds. pas des images externes.

Le 20/02/2016 13:54, Situphen a écrit :

Par contre si on navigue en HTTPS et que certaines images ne sont
disponibles qu'en HTTP, elles ne seront pas affichées, c'est ça ?

—
Reply to this email directly or view it on GitHub
#3384 (comment).

[Situphen]

Sauf que l'avatar d'un membre ne vient pas nécessairement de Zeste de Savoir !

[artragis]

Oui et du coup je modifie rien.

Le 20/02/2016 15:43, Situphen a écrit :

Sauf que l'avatar d'un membre ne vient pas nécessairement de Zeste de
Savoir !

—
Reply to this email directly or view it on GitHub
#3384 (comment).

[Situphen]

Ah OK, j'avais pas regardé le fichier. Le nom est un peu trompeur ne trouve. Bref, au temps pour moi ! ;)

[gustavi]

La QA ça donne quoi ici ?

[SpaceFox]

Hmmm j'ai l'impression que la QA est KO, mais je ne suis pas du tout certain de ce que je dois QA en fait :(

[artragis]

Pourquoi KO la QA?

Pour la QA, c'est sur la preprod que ça peut se faire puisque seule la preprod possède un certif https.

[SpaceFox]

Parce que si le nom du filtre est logique, il rend les URLs d'avatar protocol-relative, et ce n'est pas le cas.

Donc :

• Soit je ne sais pas tester parce qu'il n'y a pas de note de QA claire,
• Soit le filtre a un nom foireux et il faut le renommer,
• Soit la QA est HS.

[artragis]

pour faire plus simple : sur quelle url te bases-tu pour le test?
Car, a priori le filtre qui existe depuis assez longtemps et qui passe
les TU fonctionne bien.

Le 20/03/2016 14:18, SpaceFox a écrit :

Parce que si le nom du filtre est logique, il rend les URLs d'avatar
protocol-relative, et ce n'est pas le cas.

Donc :

• Soit je ne sais pas tester parce qu'il n'y a pas de note de QA claire,
• Soit le filtre a un nom foireux et il faut le renommer,
• Soit la QA est HS.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#3384 (comment)

[SpaceFox]

@artragis si tu pouvais donner des instructions de QA, ce serait mieux.

[artragis]

Les instructions de QA ont été données.

[SpaceFox]

En local, je crée un message avec un compte dont l'avatar est une image de la galerie. Il poste un nouveau topic, dans le code source de la page de ce topic, j'ai :

        <a href="/membres/voir/admin/" class="avatar-link">
            <img src="http://127.0.0.1:8000/media/galleries/1/bc2c4c8c-44bc-4e4a-93e5-a0e53475ea11.png" alt="" class="avatar">
        </a>

[artragis]

Je pense avoir compris le bug : c'est un problème de config je cherche à remplacer toutes les mensions à "http://" + ZDS_APP['site']['dns'] et par défaut les urls valent "http://127.0.0.1:8000" et non http://zestedesavoir.om ou https://zestedesavoir.com.
J'ai changé le templatetags pour qu'il prenne ça en compte.

[coveralls]

Coverage decreased (-1.3%) to 87.035% when pulling f2cddc5 on artragis:patch-4 into 721dc7a on zestedesavoir:dev.

[gustavi]

QA : OK !

Je merge, merci :)