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

Implement key-based authentication #56

Merged
merged 5 commits into from
Jun 20, 2024
Merged

Conversation

Marenz
Copy link
Contributor

@Marenz Marenz commented Jun 13, 2024

  • Add key-based authentication support
  • Update REPL client to work better with host/port/key parameters

@Marenz Marenz requested review from a team as code owners June 13, 2024 17:29
@github-actions github-actions bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests labels Jun 13, 2024
src/frequenz/client/dispatch/__main__.py Outdated Show resolved Hide resolved
src/frequenz/client/dispatch/_client.py Outdated Show resolved Hide resolved
src/frequenz/client/dispatch/test/_service.py Outdated Show resolved Hide resolved
src/frequenz/client/dispatch/test/_service.py Outdated Show resolved Hide resolved
src/frequenz/client/dispatch/test/_service.py Outdated Show resolved Hide resolved
@Marenz Marenz force-pushed the auth branch 4 times, most recently from 9b86795 to 3b4856a Compare June 18, 2024 13:48
llucax
llucax previously approved these changes Jun 19, 2024
RELEASE_NOTES.md Outdated Show resolved Hide resolved
Marenz added 5 commits June 19, 2024 13:49
Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
Before we couldn't really use "--host", "--port" and "--key" parameters
with the REPL mode

Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
Due to the new key parameter, we want to be more explicit

Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
@Marenz Marenz added this pull request to the merge queue Jun 20, 2024
@Marenz Marenz requested a review from llucax June 20, 2024 09:41
Merged via the queue into frequenz-floss:v0.x.x with commit 434960b Jun 20, 2024
14 checks passed
@Marenz Marenz deleted the auth branch June 20, 2024 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants