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

Use SSL by default #67

Merged
merged 3 commits into from
Aug 5, 2024
Merged

Use SSL by default #67

merged 3 commits into from
Aug 5, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Aug 2, 2024

This PR enable SSL by default and makes the default SSL option configurable for parse_grpc_uri().

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
This also prepare the test to test changing the default SSL more easily
in future commits.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax requested a review from a team as a code owner August 2, 2024 09:07
@llucax llucax requested a review from Marenz August 2, 2024 09:07
@llucax llucax self-assigned this Aug 2, 2024
@llucax llucax added this to the v0.6.0 milestone Aug 2, 2024
@github-actions github-actions bot added part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests part:code Affects the code in general labels Aug 2, 2024
@llucax llucax added the type:enhancement New feature or enhancement visitble to users label Aug 2, 2024
@Marenz
Copy link
Contributor

Marenz commented Aug 5, 2024

CI failure

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
Copy link
Contributor Author

llucax commented Aug 5, 2024

Fixed.

@llucax llucax enabled auto-merge August 5, 2024 14:55
@llucax llucax added this pull request to the merge queue Aug 5, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit 3fd2eed Aug 5, 2024
14 checks passed
@llucax llucax deleted the ssl-default branch August 5, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:code Affects the code in general part:docs Affects the documentation part:tests Affects the unit, integration and performance (benchmarks) tests type:enhancement New feature or enhancement visitble to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants