pnp/cli-microsoft365

[Bug] Without the required scope errors returned from 'tenant people profilecardproperty' commands are vague

Aperta

#5629 aperta il 3 nov 2023

Vedi su GitHubGuida contributor
 (14 commenti) (0 reazioni) (0 assegnatari)TypeScript (399 fork)auto 404
bughelp wanted

Metriche repository

Star
 (1390 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I don't have the required scope added yet to my app reg but the error returned is a bit vague:

image

That's probably because the error object coming in from the Graph has an empty message. :( image not sure how to solve this here... The MS Graph team should make this more descriptive, or we should implement checks based on the HTTP status code.. But that seems a change that would impact a lot of other code.

Originally posted by @martinlingstuyl in https://github.com/pnp/cli-microsoft365/issues/5624#issuecomment-1791486402

Guida contributor

Direzione di ricerca
Indaga come la CLI gestisce le risposte di errore dell'API Graph. Cerca pattern in cui il messaggio di errore è vuoto e proponi messaggi di fallback basati sui codici di stato HTTP o sui requisiti di ambito. Controlla la gestione degli errori esistente in comandi simili.
Tech stack
typescript
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60