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

large upload to wago fails #142

Open
arkayenro opened this issue Oct 31, 2022 · 5 comments
Open

large upload to wago fails #142

arkayenro opened this issue Oct 31, 2022 · 5 comments

Comments

@arkayenro
Copy link

arkayenro commented Oct 31, 2022

it was failing with this error - curl: (92) HTTP/2 stream 0 was not closed cleanly: PROTOCOL_ERROR (err 1)

my addon is large(ish), 1.43mb, and while it will upload to curse and wowi without a problem it will fail with that error to wago.

a small 43k test addon to wago works without any issues so the underlying code is valid and it was something else, probably size related..

googling the error seemed to indicate either a http version issue (2.0 vs 1.1 type thing) or a buffer size limitation.

i tried adding --http1.1 to the curl command in the wago section but it still failed.

when i added the --no-buffer option it worked.

if theres no downside to adding that option could the code be updated to include it in the wago section please?

edit: while it worked for my test addon it did not work for my main addon and thats still failing with the error so maybe its something else

@nebularg
Copy link
Member

nebularg commented Nov 4, 2022

It's an issue with wago servers that I haven't found a way to mitigate.

@arkayenro
Copy link
Author

arkayenro commented Nov 5, 2022

figured it out (at least i think i have, and its working so far after multiple packages).

the wago secret appears to have a newline at the end (or theres something funky with it). i added a line to echo out what it was checking and the closing bracket (around the secret) would be on the next line for the project that was failing, but would be on the same line for the test project that always worked. it should have been the same secret value for both projects so they should have been the same.

so id see[***] vs [***
]

i created a new api token in wago and updated the wago secret in both projects and both projects are now uploading to wago without any issues.

im not sure how a "broken" secret would ever work in the first place. i would have expected an auth failure instead.

i doubt its anything to do with the code but if you have a broken project that you can test with to be able to confirm a new secret works? otherwise i guess just close this.

@nebularg
Copy link
Member

nebularg commented Nov 5, 2022

interesting, the last person to report this also "fixed" it by regenerating their wago token

@janpantel
Copy link
Contributor

janpantel commented Dec 16, 2022

@arkayenro this is an interesting find, did you use the copy to clipboard button to copy the secret? I'm wondering how a newline made its way in there. I agree tho, the response is far from perfect.

PS: i'm one of the devs of wago addons

@arkayenro
Copy link
Author

i think i did but this would have been my first time with secrets so i most likely pasted it into notepad first to check it and when ive copied it from there ive grabbed the line, not just the secret, and it got pasted with the cr/nl on the end.

it should be fairly simple to test though, and hopefully work out why its not generating an auth failure.

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

No branches or pull requests

3 participants