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

[WIP] implement login, verify and logout using asp.net core identity #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rizaramadan
Copy link
Collaborator

@rizaramadan rizaramadan commented Apr 18, 2023

work in progress, high level steps:

  • prepare secret
  • prepare db context
  • prepare identity
  • prepare jwt things
  • implement login
  • implement verify
  • implement logout

@aldy505
Copy link
Member

aldy505 commented Apr 19, 2023

regarding secrets, I'd like to give some information. our deployment setup is as follows:

  • for local/self-hosted deployment, we'll provide a generic docker-compose file, in which application should read from normal OS' environment variables
  • for production deployment (deployed by the TokoBapak infrastructure team on some VM), we'll use Hashicorp Nomad. for details on how application would read secrets, you might want to consult with @zasdaym

@rizaramadan
Copy link
Collaborator Author

care to give example repo for reference @aldy505 ?

@aldy505
Copy link
Member

aldy505 commented Apr 19, 2023

care to give example repo for reference?

@rizaramadan for reading from regular OS' environment variable, you can take a look at this: https://github.com/teknologi-umum/spectator/blob/9f59bb390bd2cb6d88abf462e8aae930ae6222ac/backend/Spectator/Program.cs#L29

I don't really have a reference for reading secrets when using Nomad yet. But I assume it's the same as reading from the OS' environment variable. You can read more about it here: https://developer.hashicorp.com/nomad/docs/job-specification/env

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants