improvement(network): remove static IPs from seed lists#2407
Merged
Conversation
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
|
fr1.cipig.net is up and running, just the DNS entry was removed. 46.4.78.11 is fine, 46.4.87.18 too... and many more, eg the ones FIRO is running. How will kdf that was started without specifying |
Author
Why DNS entry is removed? That's an obvious requirement from KDF (we enter domain names on each node).
As same as before, KDF will connect to nodes from this list. |
onur-ozkan
force-pushed
the
ip-resolution
branch
from
4b0f8c0 to
c2edb1f
Compare
onur-ozkan
changed the title
SeedNodeInfo
onur-ozkan
force-pushed
the
ip-resolution
branch
from
33bb5aa to
4b0f8c0
Compare
onur-ozkan
changed the title
SeedNodeInfoSigned-off-by: onur-ozkan <work@onurozkan.dev>
onur-ozkan
force-pushed
the
ip-resolution
branch
from
7310436 to
8dd3241
Compare
smk762
approved these changes
Apr 11, 2025
smk762
left a comment
smk762
left a comment
There was a problem hiding this comment.
builds, launches and connects to seeds as expected.
shamardy
approved these changes
Apr 11, 2025
dimxy
pushed a commit
to dimxy/komodo-defi-framework
that referenced
this pull request
May 13, 2025
* dev: (26 commits) chore(deps): remove base58 and replace it completely with bs58 (GLEECBTC#2427) feat(tron): initial groundwork for full TRON integration (GLEECBTC#2425) fix(UTXO): improve tx fee calculation and min relay fee handling (GLEECBTC#2316) deps(timed-map): bump to 1.3.1 (GLEECBTC#2413) improvement(tendermint): safer IBC channel handler (GLEECBTC#2298) chore(release): complete v2.4.0-beta changelogs (GLEECBTC#2436) fix(event-streaming): initial addresses registration in utxo balance streaming (GLEECBTC#2431) improvement(watchers): re-write use-watchers handling (GLEECBTC#2430) fix(evm): make withdraw_nft work in HD mode (GLEECBTC#2424) feat(taproot): support parsing taproot output address types chore(RPC): use consistent param name for QTUM delegation (GLEECBTC#2419) fix(makerbot): add LiveCoinWatch price provider (GLEECBTC#2416) chore(release): add changelog entries for v2.4.0-beta (GLEECBTC#2415) fix(wallets): prevent path traversal in `wallet_file_path` and update file extension (GLEECBTC#2400) fix(nft): make `update_nft` work with hd wallets using the enabled address (GLEECBTC#2386) fix(wasm): unify error handling for mm2_main (GLEECBTC#2389) fix(tx-history): token information and query (GLEECBTC#2404) test(electrums): fix failing test_one_unavailable_electrum_proto_version (GLEECBTC#2412) improvement(network): remove static IPs from seed lists (GLEECBTC#2407) improvement(best-orders): return an rpc error when we can't find best orders (GLEECBTC#2318) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It's dangerous to hard-code IPs statically because they can become outdated. In fact, with this change, I was able to discover that 'falkor.dragon-seed.com' corresponds to the IP '65.108.252.86', not '168.119.237.8' and 'fr1.cipig.net' is an unavailable node.