apache / apache/pulsar-client-node
Consider making "type" optional for AuthenticationOauth2
- Vorherrschende Sprache
- C++
- Sterne
- 164
- Forks
- 98
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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!
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne bei index.d.ts ungefähr in Zeile 180 und untersuche die Definition von AuthenticationOauth2. Vergleiche die OAuth2-Nutzung des C++-Clients, um festzustellen, ob type erforderlich ist; abgeschlossen bedeutet, dass die TypeScript-Deklaration diese Anforderung korrekt widerspiegelt und type optional gemacht wird, wenn es nicht verwendet wird oder nicht erforderlich ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp, nodejs, typescript
- Bereich
- authentication
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100