Add authentication with Auth0
Opened this issue · 21 comments
Add authentication with Auth0: https://dev.to/kasuken/securing-net-6-minimal-api-with-auth0-4h5f
Buongiorno, io ho scaricato il progetto e qualcosina l'ho scritta e in linea di massima funziona. Potrei committare il codice e poi nel caso si cerca di affinare il tutto.
Ciao! Grazie per la collaborazione! Potresti fare una pull request verso il branch develop, così possiamo analizzare insieme il progetto.
@marcominerva sto provando a fare una pull request ma mi dice
Pushing to github.com:marcominerva/SimpleAuthentication.git
ERROR: Permission to marcominerva/SimpleAuthentication.git denied to micheletolve.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
In a case you entered incorrect password, please
update it in Keychain Access application.
Devi fare un fork del repository e poi fare una pull request verso di esso: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork. Ricordati di lavorare sul branch develop.
Ok fatto, speriamo bene :-D
Il token generato dal sistema Auth0 è lo stesso del JWT per cui in swagger non ho aggiunto nulla ma ho riutilizzato sempre il JwtBearer, nel metodo del controller invece ho differenziato indicando quale deve utilizzare, ovviamente mi aspetto che sia migliorabile sotto tutti i punti di vista.
Grazie per la pull request! Ti ho aggiunto un po' di commenti sul codice :-)
Buongiorno @marcominerva purtroppo non riesco a vedere i commenti sul codice di cui parli :-).
Se vai qui: #21 non li vedi?
No non li vedo purtroppo
Io vedo che mi hai riassegnato qualche cosa e poi c'è un link che rimanda qui #14 (a questa pagina) ma non vedo commenti sul codice :-D
Quindi mi confermi che neanche andando esplicitamente alla pagina https://github.com/marcominerva/SimpleAuthentication/pull/21/files vedi niente?
Vedo solo i files changed ma sono le modifiche che ho fatto io e vedo in cosa differisce dal ramo develop e che quindi non ci sono conflitti ma non vedo il codice diverso da come lo ho committato io.
Dovresti vedere qualcosa del genere:
Cerco di capire perché non ti appare. Ti tengo aggiornato!
Si si mi aspettavo una cosa del genere ma appunto non la vedo, ti dico di più che ho provato ad attivare e disattivare show comments ma non cambia nulla.
Comunque ora ho capito almeno che non devo usare Newtosoft.json ma System.Text.Json :-D e avrei potuto riflettere meglio perché in effetti meno è dipendente da librerie esterne meglio è visto che abbiamo il framework a disposizione.
Forse ho trovato l'inghippo. Prova ora ad andare su #21.
Si ecco ora vedo i vari commenti con la change request, nei prossimi giorni o se riesco entro oggi vedo di sistemare il tutto.
Solo per curiosità quale era l'inghippo? :-D
Non avevo confermato di aver terminato di inserire i commenti per la Review (perché l'opzione è un po' nascosta).
Quando pensi di aver sistemato qualcosa, dopo il push vai sul commento relativo e premi il pulsante Resolve conversation, eventualmente aggiungendo un tuo commento, così quando tutte sono resolved posso tornare a farne la review ;-)
Ciao @marcominerva ho fatto un pò di modifiche.