Produits Entr'ouvert » Passerelle

La situation actuelle n'est pas bien claire pour moi, has_pdf est toujours à False ? Si ce n'est pas le cas le code me semble ne pas faire ce pourquoi il est fait (il met bien True quand le PDF est détecté mais il ne remet pas False quand il ne l'est pas).

Et donc pour chaque ligne de facture on va en plus récupérer un fichier, chaque fois qu'on affiche cette ligne ? Ça tient la route niveau perf ou le bloc met 30s à se charger (disons si on 30 factures dans l'historique) ?

Selon moi ça ne tiendra pas, si une personne a 20 factures dans son historique, on va charger les 20, et donc paf, 20 secondes, 10 au mieux. Ca rend la page inutilisable, même avec tout l'ajax du monde, attendre plus de 5 secondes personne ne le fait, tout le monde reclique (sans compter qu'il y a "x" régies à Fontenay).

La seule solution est de régler la source du problème → balle dans la camp de la ville et de teamnet.

Alors la bonne solution c'est peut-être de gérer correctement l'erreur d'absence de fichier PDF (sans traceback, log d'erreur, etcc), et d'afficher un beau popup "Ici il devrait y avoir une facture, mais en fait non, svp prenez contact avec votre mairie 0x.xx.xx.xx.xx.".

Benjamin Dauvergne a écrit :

Alors la bonne solution c'est peut-être de gérer correctement l'erreur d'absence de fichier PDF (sans traceback, log d'erreur, etcc), et d'afficher un beau popup "Ici il devrait y avoir une facture, mais en fait non, svp prenez contact avec votre mairie 0x.xx.xx.xx.xx.".

Yep (modulo que combo présente un simple lien, et qu'il faut donc renvoyer un PDF, je crois bien).

Mais ce n'est pas la bonne solution, c'est "le moins pire des contournements" ;-)

Un simple lien vers quoi ? le portail famille ou une URL sur passerelle ?

Un simple lien (de fait c'est lingo qui dit = <a href="{% url 'download-item-pdf' regie_id=item.regie.pk item_id=item.id %}" class="icon-pdf">{% trans "Download" %}</a>).

On pourrait donc renvoyer un mini text/html dans lingo (ItemDownloadView), oui.

Ne bricolons pas dans lingo.

On ne pourrait pas en oneshot passer sur toutes les factures, stocker la liste de celles qui n'ont pas de PDF dans un fichier, et utiliser ça comme blacklist dans le connecteur ?

Frédéric Péters a écrit :

Ne bricolons pas dans lingo.

On ne pourrait pas en oneshot passer sur toutes les factures, stocker la liste de celles qui n'ont pas de PDF dans un fichier, et utiliser ça comme blacklist dans le connecteur ?

Non (il n'y a aucun webservice pour "la liste de toutes les factures")

Oublier tout ça et produire proprement une erreur 404 quand le pdf n'existe pas et attraper ça dans la "ItemDownloadView" pour afficher un message "désolé contactez la mairie" ?

Frédéric Péters a écrit :

Ne bricolons pas dans lingo.

On ne pourrait pas en oneshot passer sur toutes les factures, stocker la liste de celles qui n'ont pas de PDF dans un fichier, et utiliser ça comme blacklist dans le connecteur ?

De toute façon il faut que dans ItemDownloadView on gère le fait que peut-être le connecteur ne marche pas on ne peut pas juste faire une 500, si le fichier n'est pas disponible on le dit, surtout qu'ici ça concerne très peu de gens, autant traiter cet autre problème et ça corrigera celui de Fontenay.

Donc mon bricolage ce serait de faire que le web-service de download coté passerelle retourne une erreur 502 (après tout passerelle n'est qu'un proxy) avec une erreur formatée en JSON et que de son coté la vue ItemDownloadView pourrait retourner une redirection vers la page en cours (HTTP_REFERRER ou /) en ajoutant un messages.error() au passage à partir du message d'erreur renvoyé par passerelle, au pire juste 'Facture non disponible' (je ne sais pas comment cette vue a pu finir par s'appeler ItemDownloadView, en français ça fait quand même "Truc téléchargement"...).

Et 404 c'est bien aussi, je ne suis pas un fétichiste des codes d'erreur.

On peut gérer proprement le retour de passerelle dans le vue ItemDownloadView(afficher un message à l'utilisateur), mais ça ne nous évitera pas des traces sur admin@ à cause du raise Http404 dans le connecteur. Pour les éviter on pourrait faire un return Http404.

Je rejoins Thomas sur l'idée que la vraie solution est que la ville et Teamnet fassent leur boulot.

Ack

D'accord parce que c'est passerelle, un appel webservice, etc. mais de manière générale, il ne faut pas utiliser HttpResponseNotFound et bien privilégier le raise Http404, qui a l'avantage d'utiliser le handler 404 (par défaut l'affichage du template 404.html). (voilà, c'était juste histoire de le noter après avoir regardé la différence).

Mais là le problème c'est juste que to_json ne gère pas l'exception Http404 comme il devrait non ?

Si, to_json renvoie bien une 404, mais dans #10283 nous avons décidé de lever les exceptions afin de les avoir dans sentry et par mail.

C'est peut-être un peu excessif, sur des exceptions connus (ObjectDoesNotExist, Http404, etc..) on pourrait se contenter d'un warning.