Spaces in Font's name are not escaped
まだ誰も着手していません。
評価
調査の方向性
lib/cloudinary/utils.rb から始め、399 行目付近のテキスト変換のエスケープ処理と、452 行目付近の font-family の処理を比較します。Issue にある Open Sans の例を再現し、生成された Cloudinary URL でフォント名のスペースがエスケープされ、正しい URI としてパースできることを確認します。
索引モデルが issue の本文から書いたものです。
説明
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"
- 主要言語
- Ruby
- スター
- 420
- フォーク
- 282
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cloudinary/cloudinary_gem のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
cloudinary/cloudinary_gem#597 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 52/100
cloudinary/cloudinary_gem#591 · コメント 3 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
cloudinary/cloudinary_gem#582 · コメント 1 件 ·
-
404's when fetching .eml files (from Action Mailbox)再び着手できるかも @rnamba-cloudinary が 400 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 30/100
cloudinary/cloudinary_gem#581 · コメント 1 件 ·
-
BlobKey, MirrorService and Sidekiq 8 are incompatible再び着手できるかも @const-cloudinary が 541 日前に担当しましたが、オープン中のプルリクエストはありません。 オープン
cloudinary/cloudinary_gem#573 · コメント 3 件 · 担当者 1 名 ·
cloudinary/cloudinary_gem の issue をすべて見る
似ている issue
-
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
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
flisk-sh/flisk-labs#22 ·
-
1.severity: security
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
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
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
Huerte/algo-solutions-python#260 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
internal
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Kong/developer.konghq.com#8080 ·
メンテナーはふだん 1 日以内に返信