feat: enable using an HTTP block provider as a routing backend #110
Conversation
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
59bb83a to
a7f4096
Compare
a7f4096 to
bf995c9
Compare
lidel
reviewed
May 7, 2025
Co-authored-by: Marcin Rataj <lidel@lidel.org>
Co-authored-by: Marcin Rataj <lidel@lidel.org>
e321a2a to
e820148
Compare
…w disabling the dht entirely
e820148 to
992a0c2
Compare
- finish sooner rather than waiting 30s (DefaultRoutingTimeout) - dont return unactionable errors to end user, everything other than HTTP 200 should produce empty /routing/v1/providers result set
9cf4b6e to
5fb5c32
Compare
lidel
approved these changes
May 8, 2025
Member
lidel
left a comment
There was a problem hiding this comment.
@aschmahmann pushed tests, docs, and small fixes (hard 5s timeout).
I also updated description of this PR with example how to use it.
This should be good enough to merge & release and ask for feedback – lmk if I should make a release with this.
Contributor
Author
|
Yep @lidel, LGTM. Merge + release whenever you're ready. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
fixes #109
docs/environment-variables.mdDemo
Start someguy as
/routing/v1proxy for gateway atdag.w3s.link:$ SOMEGUY_HTTP_BLOCK_PROVIDER_ENDPOINTS="https://dag.w3s.link" SOMEGUY_HTTP_BLOCK_PROVIDER_PEERIDS="QmaSDHYKwuUKTvnsGwqutEZWg5jrpzLJhDHaYRQ9VvrDyZ" SOMEGUY_DHT=disabled SOMEGUY_PROVIDER_ENDPOINTS="" ./someguy startCID that is at
dag.w3s.link:If CID is not at gateway, there will be no providers returned.
Important
Set explicit
SOMEGUY_PROVIDER_ENDPOINTS=""to disable implicit default of queryingcid.contact, andSOMEGUY_DHT=disabledto also disable DHT backend, and only query definedSOMEGUY_HTTP_BLOCK_PROVIDER_ENDPOINTS.