-
Notifications
You must be signed in to change notification settings - Fork 19
Add support for external testnets #3
Comments
no response (yet)? |
I have encountered the same problem. node is running on GKE. I would like to support this change. |
Just commenting/removing those line should work! @jim-toth |
definitely! I haven't looked at the source since I opened this issue but iirc there are other parts in the code that would prevent this from working outside of localhost |
Hi @jim-toth, I have setup this on EC2. It works for me till now apart from balance transfer, it keeps on waiting state! Created fork and made some smaller changes
|
Hello, I have a problem with the baseURL=localhost. Thank you very much |
When will this branch be merged? |
Hi,
I've recently started a project using Arweave and long story short I'm using a cloud container to host my testnet rather than run it locally. I've got the testnet running and responding to external requests, and when I don't initialize TestWeave everything works fine.
I noticed an uncaught error where the http response was undefined. I was able to trace this down to
TestWeaveTransactionsManager
where it's explicitly overriding thebaseURL
of the request tolocalhost
https://github.com/ArweaveTeam/testweave-sdk/blob/main/src/classes/class.testweave-transactions-manager.ts#L69
I imagine the same issue would arise later on in the
getPost()
method on this line:https://github.com/ArweaveTeam/testweave-sdk/blob/main/src/classes/class.testweave-transactions-manager.ts#L93
Would it be possible to use the same baseURL building logic from ArweaveJS to allow for external testnets based on the config passed to the Arweave instance? e.g.
I was going to fork this repo and make the changes for my own use, but I wanted to see if this is something I could contribute to the project if that's something that is desired. I can open a PR if that's something the team wants 😄
The text was updated successfully, but these errors were encountered: