When strict transformations are enabled, token-based access fails with a 401
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript, nodejs
調査の方向性
lib/utils/index.js の 880 行目付近から始め、auth_token と sign_url の両方を有効にした cloudinary.url 呼び出しを再現します。生成された URL にトークンは含まれるものの、署名のパスパラメーターがない理由を追跡します。結果の URL に期待される署名が含まれ、strict transformations で動作すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Describe the bug in a sentence or two.
When I set both the auth_token and sign_url parameters on cloudinary.url, the resulting URL is given a token but is not signed. Thus, if I have strict transformations enabled, I still get a 401 from the resulting URL.
Issue Type (Can be multiple)
[ ] Build - Can’t install or import the SDK
[ ] Babel - Babel errors or cross browser issues
[ ] Performance - Performance issues
[x] Behaviour - Functions aren’t working as expected (Such as generate URL)
[ ] Documentation - Inconsistency between the docs and behaviour
[ ] Incorrect Types - For typescript users who are having problems with our d.ts files
[ ] Other (Specify)
Steps to reproduce
Any basic call to cloudinary.url, e.g.:
cloudinary.url('my_image', {
type: 'upload',
auth_token: { key: '<encryption-key>', duration: 300 },
sign_url: true,
})
This gives me:
https://res.cloudinary.com/<env>/image/upload/my_image?__cld_token__=exp=1234~hmac=<snip>&_a=BAMCkGUm0
The URL is missing the signature path param, so if I have strict transformations, the URL doesn't work.
Seems like this conditional is preventing it: https://github.com/cloudinary/cloudinary_npm/blob/d12a63fefdd0209ed71c3a641a9d3166126c09d8/lib/utils/index.js#L880
I'm not sure if there's a different reason for it, but as a new user to the platform, my gut feeling says this part might need to change.
Versions and Libraries (fill in the version numbers)
Cloudinary_NPM SDK version 2.5.1
Node - 18.13.0
NPM - 0.0.0
PNPM - 9.4.0
- 主要言語
- JavaScript
- スター
- 672
- フォーク
- 318
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cloudinary/cloudinary_npm のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
cloudinary/cloudinary_npm#746 · コメント 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
cloudinary/cloudinary_npm#739 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 58/100
cloudinary/cloudinary_npm#734 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
cloudinary/cloudinary_npm#713 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
cloudinary/cloudinary_npm#712 · コメント 1 件 ·
cloudinary/cloudinary_npm の issue をすべて見る
似ている issue
-
bug confirmed issue
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
open-webui/open-webui#34458 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
area: install-setup needs-info question
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
accessibility area: css area: web ui bug difficulty: easy frontend good first issue help wanted javascript journey: first-visit journey: mobile priority: P3 quality tech: css tech: html typescript ui user-experience
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
themactep/thingino-firmware#2018 ·
メンテナーはふだん 1 日以内に返信