Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

update arweave v1.10 -> v1.12 + bug fix #15

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cromatikap
Copy link

arweave now append the content-type header to the config option so passing it to arweave.api.post() was breaking

@rosmcmahon
Copy link

might be an arweave-js issue actually?

@cromatikap
Copy link
Author

Hi Ross,

It is an issue from arweave npm package related to the repository arweave-js if I'm not mistaken somewhere.

I have tested the fix here

@rosmcmahon
Copy link

rosmcmahon commented Jan 26, 2023

please test against latest arweave-js arweave v1.12.4 this should mitigate the issue.
if not, could you open an Issue or PR in the arweave-js repo?

@sonarcloud
Copy link

sonarcloud bot commented Feb 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@cromatikap
Copy link
Author

It works for me, well done 🥇

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants