Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feature: delegated-routing: Add HTTP delegated routing. #9453

Merged
merged 5 commits into from
Dec 7, 2022

Conversation

ajnavarro
Copy link
Member

@ajnavarro ajnavarro commented Dec 5, 2022

Added the new HTTP delegated routing as a possible routing system that can be added to provide and resolve content locations.

Signed-off-by: Antonio Navarro Perez antnavper@gmail.com

@ajnavarro ajnavarro self-assigned this Dec 5, 2022
@ajnavarro ajnavarro requested a review from guseggert December 5, 2022 18:02
@ajnavarro ajnavarro force-pushed the ajnavarro/feature/add-http-delegated-routing branch 3 times, most recently from e4950bb to 6650bee Compare December 6, 2022 11:03
Signed-off-by: Antonio Navarro Perez <antnavper@gmail.com>
Signed-off-by: Antonio Navarro Perez <antnavper@gmail.com>
@ajnavarro ajnavarro force-pushed the ajnavarro/feature/add-http-delegated-routing branch 2 times, most recently from 73c2afe to 6d61774 Compare December 6, 2022 11:15
Signed-off-by: Antonio Navarro Perez <antnavper@gmail.com>
@ajnavarro ajnavarro force-pushed the ajnavarro/feature/add-http-delegated-routing branch from 6d61774 to 9b5f811 Compare December 6, 2022 11:21
@ajnavarro ajnavarro marked this pull request as ready for review December 6, 2022 12:05
transport.MaxIdleConns = 500
transport.MaxIdleConnsPerHost = 100

delegateHTTPClient := &http.Client{
Copy link
Contributor

Choose a reason for hiding this comment

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

will need to wrap this in https://github.com/ipfs/go-libipfs/blob/main/routing/http/client/transport.go#L9 but I can take care of that in a followup PR

@ajnavarro ajnavarro merged commit 3185dee into master Dec 7, 2022
@ajnavarro ajnavarro deleted the ajnavarro/feature/add-http-delegated-routing branch December 7, 2022 15:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants