Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add authentication with Auth0 #14

Open
totti240282 opened this issue May 25, 2022 · 21 comments · May be fixed by #21
Open

Add authentication with Auth0 #14

totti240282 opened this issue May 25, 2022 · 21 comments · May be fixed by #21
Assignees
Labels
enhancement New feature or request

Comments

@totti240282
Copy link

totti240282 commented May 25, 2022

Add authentication with Auth0: https://dev.to/kasuken/securing-net-6-minimal-api-with-auth0-4h5f

@micheletolve
Copy link

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.

@marcominerva
Copy link
Owner

Ciao! Grazie per la collaborazione! Potresti fare una pull request verso il branch develop, così possiamo analizzare insieme il progetto.

@micheletolve
Copy link

@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.

@marcominerva
Copy link
Owner

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.

@micheletolve
Copy link

Ok fatto, speriamo bene :-D

@micheletolve
Copy link

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.

@marcominerva
Copy link
Owner

Grazie per la pull request! Ti ho aggiunto un po' di commenti sul codice :-)

@micheletolve
Copy link

Buongiorno @marcominerva purtroppo non riesco a vedere i commenti sul codice di cui parli :-).

@marcominerva
Copy link
Owner

Se vai qui: #21 non li vedi?

@micheletolve
Copy link

No non li vedo purtroppo

@marcominerva marcominerva linked a pull request Jun 7, 2022 that will close this issue
@micheletolve
Copy link

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

@marcominerva
Copy link
Owner

Quindi mi confermi che neanche andando esplicitamente alla pagina https://github.com/marcominerva/SimpleAuthentication/pull/21/files vedi niente?

@micheletolve
Copy link

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.

@marcominerva
Copy link
Owner

Dovresti vedere qualcosa del genere:

image

Cerco di capire perché non ti appare. Ti tengo aggiornato!

@micheletolve
Copy link

micheletolve commented Jun 7, 2022

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.
image
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.

@marcominerva
Copy link
Owner

Forse ho trovato l'inghippo. Prova ora ad andare su #21.

@micheletolve
Copy link

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

@marcominerva
Copy link
Owner

Non avevo confermato di aver terminato di inserire i commenti per la Review (perché l'opzione è un po' nascosta).

@marcominerva
Copy link
Owner

marcominerva commented Jun 7, 2022

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 ;-)

@micheletolve
Copy link

Ciao @marcominerva ho fatto un pò di modifiche.

@marcominerva marcominerva added the enhancement New feature or request label Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants