facebook/fresco

SvgDecoderExample limit <svg tag

Open

#2 490 ouverte le 20 avr. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Java (17 072 stars) (3 750 forks)batch import
enhancementgood first issuehelp wanted

Description

We use GitHub Issues for bugs.

If you have a non-bug question, please ask on Stack Overflow: http://stackoverflow.com/questions/tagged/fresco

--- Please use this template, and delete everything above this line before submitting your issue ---

Description

I want use SimpleDraweeView to .setImageURI(Uri.parse("https://www.ups.com/assets/resources/images/UPS_logo.svg")); but the svg file is start with <?xml tag, so SvgDecoderExample.SvgFormatChecker() is block it.

Solution

I try update SvgDecoderExample.java line 33 : private static final String HEADER_TAG = "<svg"; to private static final String HEADER_TAG = "<?xml"; let it thought the cheker, it is visible now.
why you want to block the tag?
The AndroidSVG('com.caverock:androidsvg:1.2.1') lib is work find.

Additional Information

  • Fresco version: 'com.facebook.fresco:fresco:2.0.0'

Guide contributeur

Stack technique
javaandroid
Domaine
mobile
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.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaAndroid
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The issue is in SvgDecoderExample.java on line 33 where the constant HEADER TAG checks only for '<svg'. The reporter suggests changing it to '<?xml' or accepting both. The maintainer should confirm if files starting with <?xml are valid SVG and whether the fix should allow both patterns. Check the AndroidSVG library documentation for supported SVG root tags.