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

Allow loopd autogenerated TLS cert validity override with a new flag #614

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

gcaracuel
Copy link
Contributor

Add a new flag --tlsvalidity that allows override default ~14months loopd auto generated certificate validity period.

Use case example:

When using lndconnect string to reach loopd some user will like to have a valid string for longer of 14 months. For those that has a system to rotate this config it might be useful to change this validity period.

@gcaracuel
Copy link
Contributor Author

Guys a compilation of loopserver for arm64 platform will be really welcome :) it has been painful to test this thing on M1/2 chipsets.

Copy link
Member

@sputn1ck sputn1ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the useful PR!

LGTM, pending line length (see: https://github.com/lightningnetwork/lnd/blob/master/docs/code_formatting_rules.md#80-character-line-length for reference)

We will have a loopserver arm image out soon (PR on serverside is pending), thanks for the reminder.

loopd/config.go Outdated Show resolved Hide resolved
@sputn1ck sputn1ck requested a review from bhandras July 27, 2023 10:11
Copy link
Member

@GeorgeTsagk GeorgeTsagk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm ✅

loopd/config.go Outdated Show resolved Hide resolved
loopd/config.go Outdated Show resolved Hide resolved
loopd/config.go Outdated Show resolved Hide resolved
@gcaracuel gcaracuel force-pushed the tlsvalidity_flag branch 2 times, most recently from 9ac8db6 to 96b2d83 Compare July 27, 2023 12:47
@sputn1ck
Copy link
Member

sputn1ck commented Jul 27, 2023

Guys a compilation of loopserver for arm64 platform will be really welcome :) it has been painful to test this thing on M1/2 chipsets.

we now have an arm64 image available!

https://hub.docker.com/r/lightninglabs/loopserver/tags

@gcaracuel gcaracuel force-pushed the tlsvalidity_flag branch 2 times, most recently from 2241137 to b3a61d6 Compare July 27, 2023 13:06
@gcaracuel
Copy link
Contributor Author

Refactored PR with your suggestion, it looks way better now, thanks!

Copy link
Member

@sputn1ck sputn1ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 reviewers and no one noticed the wrong comment 😄

loopd/config.go Outdated Show resolved Hide resolved
… flag

Co-authored-by: George Tsagkarelis <34623190+GeorgeTsagk@users.noreply.github.com>
loopd/config.go Outdated Show resolved Hide resolved
Co-authored-by: George Tsagkarelis <34623190+GeorgeTsagk@users.noreply.github.com>
@sputn1ck sputn1ck merged commit 8986763 into lightninglabs:master Jul 27, 2023
5 checks passed
@gcaracuel
Copy link
Contributor Author

3 reviewers and no one noticed the wrong comment 😄

It was added in the last commits. Copy pasting issues

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.

4 participants