Spaces in Font's name are not escaped
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 45/100
Direção de pesquisa
Comece em lib/cloudinary/utils.rb, comparando o escaping da transformação de texto por volta da linha 399 com o tratamento de font-family por volta da linha 452. Reproduza o exemplo de Open Sans do issue e verifique se a URL do Cloudinary gerada faz o escaping dos espaços no nome da fonte e pode ser analisada como uma URI válida.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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"
- Linguagem predominante
- Ruby
- Estrelas
- 420
- Forks
- 282
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de cloudinary/cloudinary_gem
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
cloudinary/cloudinary_gem#597 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 52/100
cloudinary/cloudinary_gem#591 · 3 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
cloudinary/cloudinary_gem#582 · 1 comentário ·
-
404's when fetching .eml files (from Action Mailbox)Talvez livre de novo @rnamba-cloudinary assumiu há 400 dias e não há nenhum pull request aberto. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 30/100
cloudinary/cloudinary_gem#581 · 1 comentário ·
-
BlobKey, MirrorService and Sidekiq 8 are incompatibleTalvez livre de novo @const-cloudinary assumiu há 541 dias e não há nenhum pull request aberto. Aberta
cloudinary/cloudinary_gem#573 · 3 comentários · 1 responsável ·
Todas as issues de cloudinary/cloudinary_gem
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
metacall/core#964 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
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
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
flisk-sh/flisk-labs#22 ·
-
1.severity: security
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
Mantenedores costumam responder em até 1 dia
-
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
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
Huerte/algo-solutions-python#260 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia