update graph to thegraph,add graphApiKey #40

nanli777torn · 2025-01-18T04:29:43+03:00

nanli777torn commented

2025-01-18 04:29:43 +03:00

First-time contributor

Explanation

update graph to thegraph,add graphApiKey

Users can set graphApiKey by themselves

https://thegraph.com provides 100,000 requests per month to each user.

More information

The default setting is my thegraph apikey for the time being.

Screenshots/Screencaps

## Explanation update graph to thegraph,add graphApiKey Users can set graphApiKey by themselves https://thegraph.com provides 100,000 requests per month to each user. ## More information The default setting is my thegraph apikey for the time being. ## Screenshots/Screencaps ![image.png](/attachments/f0284e49-9d10-4cf4-a4a9-e9e230586935)

image.png

67 KiB

nanli777torn added 2 commits 2025-01-18 04:29:44 +03:00

test 2371ec0ebb

update graph to thegraph,add graphApiKey 8791b5aff8

tornadocontrib requested changes 2025-01-18 05:39:42 +03:00

services/graph.js

						
				@ -28,3 +36,1 @@

				  42161: 'https://tornadocash-rpc.com/subgraphs/name/tornadocash/arbitrum-tornado-subgraph',

				  43114: 'https://api.thegraph.com/subgraphs/name/tornadocash/avalanche-tornado-subgraph',

				  11155111: 'https://tornadocash-rpc.com/subgraphs/name/tornadocash/sepolia-tornado-subgraph'

				  56:

tornadocontrib commented

2025-01-18 05:39:33 +03:00

Some comments,

TheGraph is an american company, if the US Gov requests to remove any Tornado Cash subgraphs, they must comply.
Therefore, not only the graphApiKey but also the entire endpoint URL ( https://gateway.thegraph.com/api ) should be in replaceable format ( you can refer my previous work https://codeberg.org/tornadocash/classic-ui/pulls/2/files about the changes ).
Also, I would recommend leaving the default URL as is, it is under the control of Theo and will be fixed during this week.

Some comments, 1. TheGraph is an american company, if the US Gov requests to remove any Tornado Cash subgraphs, they must comply. 2. Therefore, not only the graphApiKey but also the entire endpoint URL ( https://gateway.thegraph.com/api ) should be in replaceable format ( you can refer my previous work https://codeberg.org/tornadocash/classic-ui/pulls/2/files about the changes ). 3. Also, I would recommend leaving the default URL as is, it is under the control of Theo and will be fixed during this week.

nanli777torn added 1 commit 2025-01-18 06:04:37 +03:00

update graph to thegraph,add graphApiKey 1a5e6711b0

tornadocontrib requested changes 2025-01-18 06:23:14 +03:00

package.json

						
				@ -10,3 +10,3 @@

				    "fix:vuex": "node ./scripts/vuex.js",

				    "dev": "cross-env NODE_OPTIONS=\"--max_old_space_size=8192\" nuxt",

				    "dev:lts": "yarn fix:vuex && cross-env NODE_OPTIONS=\"--max_old_space_size=8192 --openssl-legacy-provider\" nuxt",

				    "dev:lts": "yarn fix:vuex && cross-env NODE_OPTIONS=\"--max_old_space_size=8192 \" nuxt",

tornadocontrib commented

2025-01-18 06:22:58 +03:00

Also please do not remove necessary flags for Node.js LTS version, it is required after the supported Node.js version 18.x https://nodejs.org/en/about/previous-releases#release-schedule. Upgrade your node.js if you are using node.js older than that.

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u development:nanli777torn-development

git checkout nanli777torn-development

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tornadocash/classic-ui#40