Spaces in Font's name are not escaped
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
Línea de trabajo
Empieza en lib/cloudinary/utils.rb, comparando el escape de la transformación de texto alrededor de la línea 399 con el manejo de font-family alrededor de la línea 452. Reproduce el ejemplo de Open Sans del issue y verifica que la URL de Cloudinary generada escape los espacios del nombre de la fuente y pueda analizarse como una URI válida.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Cloudinary supports all of Google's fonts https://support.cloudinary.com/hc/en-us/articles/203352832-What-is-the-list-of-supported-fonts-for-text-overlay-transformation- and some of them have spaces on them, for example, "Open Sans"
But unlike Text Transformation where spaces are escaped https://github.com/cloudinary/cloudinary_gem/blob/28a9ee65d24c6bac0654919a5a76de95cf356510/lib/cloudinary/utils.rb#L399 font family aren't https://github.com/cloudinary/cloudinary_gem/blob/28a9ee65d24c6bac0654919a5a76de95cf356510/lib/cloudinary/utils.rb#L452
resulting in an invalid URL
[1] pry(main)> Cloudinary::VERSION
=> "1.20.0"
[2] pry(main)> url = Cloudinary::Utils.cloudinary_url("coffee_cup.jpg", transformation: [
{overlay: {font_family: "Open Sans", text: "Text with spaces", font_size: 10}}
])
=> "http://res.cloudinary.com/cookpad/image/upload/l_text:Open Sans_10:Text%20with%20spaces/coffee_cup.jpg"
[3] pry(main)> URI.parse url
URI::InvalidURIError: bad URI(is not URI?): "http://res.cloudinary.com/cookpad/image/upload/l_text:Open Sans_10:Text%20with%20spaces/coffee_cup.jpg"
- Lenguaje dominante
- Ruby
- Estrellas
- 420
- Forks
- 282
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cloudinary/cloudinary_gem
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
cloudinary/cloudinary_gem#597 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
cloudinary/cloudinary_gem#591 · 3 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
cloudinary/cloudinary_gem#582 · 1 comentario ·
-
404's when fetching .eml files (from Action Mailbox)Quizá libre de nuevo @rnamba-cloudinary la tomó hace 400 días y no hay ningún pull request abierto. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
cloudinary/cloudinary_gem#581 · 1 comentario ·
-
BlobKey, MirrorService and Sidekiq 8 are incompatibleQuizá libre de nuevo @const-cloudinary la tomó hace 541 días y no hay ningún pull request abierto. Abierto
cloudinary/cloudinary_gem#573 · 3 comentarios · 1 asignado ·
Todos los issues de cloudinary/cloudinary_gem
Issues similares
-
area: ci benchmark build developer-experience difficulty: easy dx enhancement github-actions good first issue hacktoberfest help wanted performance quality rank: 8 kyu ruby tech: ruby
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
flisk-sh/flisk-labs#22 ·
-
1.severity: security
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
15-minutes 5-minutes algorithm any-language beginner-friendly c cpp csharp easy go good first issue hacktoberfest help wanted java javascript kotlin python rust swift typescript
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Huerte/algo-solutions-python#260 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
internal
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Kong/developer.konghq.com#8080 ·
Los mantenedores suelen responder en 1 día