chore: merge main to develop #14
Merged
Conversation
This file contains 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
* feat: add coingeckoProxy * chore: add fastify/caching dependency * feat: add request caching * fix: only cache successfull responses from proxy origin * chore: improve logging * feat: set our own cache-control header * chore: delete set-cookie header * chore: rename local header from cache-hit to x-proxy-cache and leave it in the reply * Revert "chore: rename local header from cache-hit to x-proxy-cache and leave it in the reply" This reverts commit 71ecd72. * fix: prevent content-length header mismatch errors * fix: do not fwd accept-encoding header, as I'm not able to deal with encoded response * chore: attempt at caching only after response is sent * chore: try a different way to decorate * Revert "chore: try a different way to decorate" This reverts commit 31672a6. * Revert "chore: attempt at caching only after response is sent" This reverts commit b2dd0eb. * feat: coingecko proxy without server side caching (#11) * feat: remove server side caching * chore: remove fastify-caching * chore: undici config still required * fix: type on cdn-cache-control value * feat: add coingecko api key (#12) * feat: add coingecko api key to proxy requests * fix: remove `simple` prefix as it's part of coingecko's path * chore: temporarily change to pro-api since the key is set to "pro" * chore: add COINGECKO_PROXY_UPSTREAM as optional env var
* refactor: move coingeckoProxy to proxies/coingecko * refactor: move proxy to proxies/tokens and keep proxy for awhile * chore: add permanent redirect rather than re-exporting same path * Revert "chore: add permanent redirect rather than re-exporting same path" This reverts commit 246c27a. graphql-request doesn't seem able to handle redirects natively: https://stackoverflow.com/questions/78015439/follow-redirects-setting-for-aws-appsync#comment137542218_78015439 Thus, reverting the redirect change
chore: update yarn.lock
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@cowprotocol/cow-sdk@2.1.0, npm/@types/node@18.7.23, npm/vite-plugin-dts@3.1.0, npm/vite-tsconfig-paths@4.2.0, npm/vite@4.5.3, npm/walker@1.0.8, npm/yargs-parser@21.1.1, npm/yargs@17.7.2 |
anxolin
approved these changes
May 7, 2024
# for free
to join this conversation on GitHub.
Already have an account?
# 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.
Summary
Merge latest main to develop