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

docker lnd: add support for custom params RPCHOST and RPCCRTPATH #7429

Conversation

eldios
Copy link
Contributor

@eldios eldios commented Feb 20, 2023

Change Description

In this commit we add support for two more custom parameters in the main entrypoint for the lnd Docker image script (start-lnd.sh)

The two parameters are:

  • RPCHOST to set a custom endpoint for the RPC server
  • RPCCRTPATH to set a custom location for the certificate used when
    communicating with the RPC server

Steps to Test

Change should be 100% backward compatible, but it's as easy as creating a new deployment and set the two new variables via environment variables or leave them empty for default values.

Pull Request Checklist

Testing

  • Your PR passes all CI checks.
  • Tests covering the positive and negative (error paths) are included.
  • Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

📝 Please see our Contribution Guidelines for further guidance.

In this commit we add support for two more custom parameters in the
main entrypoint for the `lnd` Docker image script (`start-lnd.sh`)

The two parameters are:
* RPCHOST    to set a custom endpoint for the RPC server
* RPCCRTPATH to set a custom location for the certificate used when
  communicating with the RPC server
Add link to current release notes for this commit PR
Copy link
Collaborator

@ellemouton ellemouton left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM 🔥

@ellemouton
Copy link
Collaborator

Oh ,actually - it is still missing an entry in the release notes @eldios

@eldios
Copy link
Contributor Author

eldios commented Feb 20, 2023

I was still adding it cause I needed the URL 😝

@guggero
Copy link
Collaborator

guggero commented May 12, 2023

Replaced by/included in #7689, original commit author credits are preserved.

@guggero guggero closed this May 12, 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.

3 participants