-
Notifications
You must be signed in to change notification settings - Fork 47
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
fix(archive): dburl check #2071
Conversation
This PR may contain changes to configuration options of one of the apps. If you are introducing a breaking change (i.e. the set of options in latest release would no longer be applicable) make sure the original option is preserved with a deprecation note for 2 following releases before it is actually removed. Please also make sure the label |
You can find the image built from this PR at
Built from 3453b30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sorry for the mess. I need first to submit a PR where the This can't be merged yet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!
623c64a
to
23fd599
Compare
Description
This is to make the database URL regex to accept
.
and-
within the hostname.Also, the user and password fields can be anything not empty.
notes:
apps/wakunode2/external_config.nim
.How to test
nwaku
with--store-message-db-url = "postgres://nim-waku-user:nim-waku-pass@store-db-01.do-ams3.shards.test.wg:5432/postgres"
.Issue
status-im/infra-shards#8 (comment)