apache / apache/pulsar-client-node

Consider making "type" optional for AuthenticationOauth2

オープン
#193 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
164
フォーク
98
PR マージ指標
30日以内にマージされた PR はありません

説明

The C++ client does not seem to use the property `type`. Yet [it has been marked as required](https://github.com/apache/pulsar-client-node/blob/f696d2fd958b702122ca7a21f84642ea855ec25d/index.d.ts#L180). Please verify whether it is actually required and make it optional if not.

Thanks a bunch for your work on the Node.js client!

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

index.d.ts の 180 行目付近から始めて、AuthenticationOauth2 の定義を確認します。C++ クライアントの OAuth2 の使用方法と比較して、type が必須かどうかを判断します。完了の条件は、TypeScript の宣言がその要件を正確に反映し、type が使用されていない、または必要でない場合にオプションになることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp, nodejs, typescript
領域
authentication
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。