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

feat: support multiple gateway URLs #2126

Merged
merged 4 commits into from
Oct 18, 2023
Merged

Conversation

alanshaw
Copy link
Member

This PR adds support for multiple gateway URLs in the API proxy. It allows us to ask freeway first, then fallback to ipfs.io.

Copy link
Contributor

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

Please set the Secret before merging and add it to README.md 🙏🏼

Copy link
Contributor

@olizilla olizilla left a comment

Choose a reason for hiding this comment

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

GATEWAY_URL is set in wrangler.toml so consider setting the new CSV var there too.

@olizilla olizilla requested review from a team as code owners October 18, 2023 12:36
@github-actions
Copy link
Contributor

License: MIT
Signed-off-by: Oli Evans <[email protected]>
License: MIT
Signed-off-by: Oli Evans <[email protected]>
@olizilla
Copy link
Contributor

@alanshaw good to go?

Copy link
Member Author

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

🚢

@olizilla olizilla merged commit 415dcf6 into main Oct 18, 2023
6 checks passed
@olizilla olizilla deleted the feat/support-multiple-gateway-urls branch October 18, 2023 14:15
olizilla pushed a commit that referenced this pull request Oct 19, 2023
🤖 I have created a release *beep* *boop*
---


##
[7.20.0](api-v7.19.0...api-v7.20.0)
(2023-10-18)


### Features

* support multiple gateway URLs
([#2126](#2126))
([415dcf6](415dcf6))


### Bug Fixes

* use dag-staging gateway from staging api
([#2318](#2318))
([943b4cd](943b4cd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants