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

tbc: allow seeds to be overwritten #158

Merged
merged 14 commits into from
Jun 26, 2024
Merged

Conversation

ClaytonNorthey92
Copy link
Contributor

Summary
allow tbc seeds to be overwritten

Changes
if cfg seeds are set, use those, otherwise use the defaults for the network chosen

fixes #157

if cfg seeds are set, use those, otherwise use the defaults for the network chosen
@ClaytonNorthey92 ClaytonNorthey92 marked this pull request as ready for review June 24, 2024 20:29
service/tbc/tbc.go Outdated Show resolved Hide resolved
service/tbc/tbc.go Outdated Show resolved Hide resolved
service/tbc/tbc.go Outdated Show resolved Hide resolved
Copy link
Contributor

@joshuasing joshuasing left a comment

Choose a reason for hiding this comment

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

Will cfg.Seeds be exposed via an environment variable?
If so, I think you need to add an item to the slice in cmd/tbcd/

service/tbc/tbc.go Outdated Show resolved Hide resolved
@ClaytonNorthey92
Copy link
Contributor Author

Will cfg.Seeds be exposed via an environment variable? If so, I think you need to add an item to the slice in cmd/tbcd/

ah yes I had forgottent we have the command (I was thinking of only op-geth) I'll update 👍

service/tbc/tbc.go Outdated Show resolved Hide resolved
cmd/tbcd/tbcd.go Outdated Show resolved Hide resolved
config/config.go Outdated Show resolved Hide resolved
config/config.go Outdated Show resolved Hide resolved
config/config.go Outdated Show resolved Hide resolved
service/tbc/tbc.go Outdated Show resolved Hide resolved
Co-authored-by: Joshua Sing <joshua@bloq.com>
@ClaytonNorthey92 ClaytonNorthey92 changed the title allow TBC seeds to be overwritten tbc: allow seeds to be overwritten Jun 26, 2024
@ClaytonNorthey92 ClaytonNorthey92 merged commit 27a5081 into main Jun 26, 2024
4 checks passed
@ClaytonNorthey92 ClaytonNorthey92 deleted the clayton/config-seed branch June 26, 2024 14:03
@joshuasing joshuasing added type: refactor This refactors existing functionality type: enhancement This improves existing functionality area: tbc This is a change to TBC (Tiny Bitcoin) labels Jul 1, 2024
web3cryptoguy pushed a commit to web3cryptoguy/heminetwork that referenced this pull request Nov 1, 2024
* allow TBC seeds to be overwritten

if cfg seeds are set, use those, otherwise use the defaults for the network chosen

* fixed missing seeding

* remove tbc Server.port

* using string for seeds

* use host for lookupip

* added TBC_SEEDS env config

* Update service/tbc/tbc.go

Co-authored-by: Joshua Sing <joshua@bloq.com>

* simplify NewPeer in localPeerManager

* Update cmd/tbcd/tbcd.go

Co-authored-by: Joshua Sing <joshua@bloq.com>

* use AppendSlice

* removed ports from variables

* fix reflect handling of seeds

* Update config/config.go

Co-authored-by: Joshua Sing <joshua@bloq.com>

* using 127.0.0.1 as default IP/host for localnet

---------

Co-authored-by: Joshua Sing <joshua@bloq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: tbc This is a change to TBC (Tiny Bitcoin) type: enhancement This improves existing functionality type: refactor This refactors existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow TBC seeds to be overridden
3 participants