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

Session management #214

Merged
merged 1 commit into from
Jun 25, 2023
Merged

Session management #214

merged 1 commit into from
Jun 25, 2023

Conversation

rdnt
Copy link
Owner

@rdnt rdnt commented Jun 25, 2023

This PR adds session identifiers that should be used by the user of the application (e.g. separate HTTP REST clients, or a CLI client) to authenticate against the application. Sessions will have a time to live that resets using the HealthCheck method. If no sessions are present the argon key is purged from in-memory.

This adds proper security locally in case some program installed does queries against the application while the user is authenticated, which previously would return all the decrypted data.

@rdnt rdnt merged commit fee40f8 into main Jun 25, 2023
@rdnt rdnt deleted the sessions branch June 25, 2023 11:37
rdnt added a commit that referenced this pull request Aug 30, 2023
Former-commit-id: fee40f8
Former-commit-id: c83ba085d9a932ba6d57eb0a82864cd6a77d9897
rdnt added a commit that referenced this pull request Aug 30, 2023
Former-commit-id: c727288be312bac6bb32d2be1e9a8a572f1f1339 [formerly fee40f8]
Former-commit-id: c83ba085d9a932ba6d57eb0a82864cd6a77d9897
Former-commit-id: f0d8ece
rdnt added a commit that referenced this pull request Aug 30, 2023
Former-commit-id: c727288be312bac6bb32d2be1e9a8a572f1f1339 [formerly ea3f15214cf83701b44a909a20443f2cbd7cb6bb]
Former-commit-id: c83ba085d9a932ba6d57eb0a82864cd6a77d9897
Former-commit-id: f0d8ece
Former-commit-id: a876cd589b5f0f993605b8df38264a0e4f0afb40
rdnt added a commit that referenced this pull request Aug 30, 2023
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.

1 participant