nextcloud/text

Failing preview generation results in 404 istead of original image

Ouverte

#6 755 ouverte le 8 déc. 2024

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)JavaScript (125 forks)auto 404
1. to developbuggood first issuephp

Métriques du dépôt

Stars
 (644 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Describe the bug We had some traces in Sentry:

Image

Image

Since we don't catch these they will be caught on the Controller Level which then sends a 404. We should return the raw image instead. In case of the trace here it was a png and could have been served fine.

Server details: cloud.nextcloud.com from 30.0.1.1 to 30.0.2.2 - probably longer.

Guide contributeur

Direction de recherche
Inspectez le code de génération d'aperçu et la gestion des erreurs du contrôleur pour intercepter les exceptions et renvoyer l'image d'origine au lieu d'une erreur 404.
Stack technique
javascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75