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

Overriding bootnode config param using default format in .env file leads to parsing error #2494

Closed
rvcryp opened this issue Sep 11, 2021 · 2 comments
Labels
issue wontfix This will not be worked on

Comments

@rvcryp
Copy link

rvcryp commented Sep 11, 2021

⚠️ Requests for support in an issue-format will be closed immediately. For support questions, we welcome you to our Discord.

Summary

This relates to docker-compose setup and specifically addresses the config in the .env file. The default bootnode address for testnet is not correct (identified earlier in #2437 and solved in master). If, however, in release 1.1.0 you want to override the currently incorrect bootnode address for testnet by using/adapting the default shown in the comment in the .env file ('default [/dnsaddr/boot.node.ethswarm.org]') it leads to a parsing error when starting the bee node.

Steps to reproduce

Start a fresh new node. Open .env file for a docker-compose setup, uncomment the BEE_BOOTNODE config line and enter the corrected default boot node address in the way suggested in the comments of the file: BEE_NODE=[/dnsadr/testnode.ethswarm.org]. Make sure log file is at least 'debug' and start the bee node

Expected behavior

Expected the bee to discover the bootnodes and start connecting to peers

Actual behavior

The bee continued to log that it had no peers and was trying the bootnodes. Before that, the log file indicated a parsing error: level=debug msg="multiaddress fail [/dnsaddr/testnet.ethswarm.org]: failed to parse multiaddr "[/dnsaddr/testnet.ethswarm.org]": must begin with /". Tried to also enter is as BEE_BOOTNODE="[/dnsaddr/testnet.ethswarm.org]", but that did not work either. Eventually it worked by not making it multiaddr, but just simply BEE_BOOTNODE=/dnsaddr/testnet.ethswarm.org. Maybe special characters are not correctly parsed/escaped?

@rvcryp rvcryp added the needs-triaging new issues that need triaging label Sep 11, 2021
@bee-runner bee-runner bot added the issue label Sep 11, 2021
@ldeffenb
Copy link
Collaborator

I suspect you need quotes inside and outside the []. Not sure the proper syntax, but something like:

BEE_BOOTNODE="['/dnsaddr/testnet.ethswarm.org']"
BEE_BOOTNODE="[""/dnsaddr/testnet.ethswarm.org""]"
BEE_BOOTNODE="["/dnsaddr/testnet.ethswarm.org"]"

@acud acud added wontfix This will not be worked on and removed needs-triaging new issues that need triaging labels Oct 15, 2021
@acud
Copy link
Member

acud commented Oct 15, 2021

closing this as this appears to be related to user-error and there's no actionable item here

@acud acud closed this as completed Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants