⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Relax validation on the postgres service address setting#91

Open
tankyleo wants to merge 1 commit intolightningdevkit:mainfrom
tankyleo:2026-02-relax-postgres-addr-setting
Open

Relax validation on the postgres service address setting#91
tankyleo wants to merge 1 commit intolightningdevkit:mainfrom
tankyleo:2026-02-relax-postgres-addr-setting

Conversation

@tankyleo
Copy link
Contributor

@tankyleo tankyleo commented Feb 4, 2026


    Relax validation on the postgres service address setting

    We previously parsed the postgres service address setting into a
    `std::net::SocketAddr`, which was overly restrictive, and
    prevented the use of hostnames for that setting.

    The postgres service address is logged on startup, so the user can check
    that the service address is correctly configured in case of a failure
    to connect to the postgres database on startup.

We previously parsed the postgres service address setting into a
`std::net::SocketAddr`, which was overly restrictive, and
prevented the use of hostnames for that setting.

The postgres service address is logged on startup, so the user can check
that the service address is correctly configured in case of a failure
to connect to the postgres database on startup.
@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Feb 4, 2026

👋 I see @joostjager was un-assigned.
If you'd like another reviewer assignment, please click here.

@tankyleo tankyleo requested review from tnull and removed request for joostjager February 4, 2026 21:14
Copy link
Contributor

@tnull tnull left a comment

Choose a reason for hiding this comment

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

In #73 we made the same change for bind_address. We should probably revert that, too, no?

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