eivindveg / HotSUploader

JavaFX-based Replay Uploader for Heroes of the Storm
Apache License 2.0
185 stars 36 forks source link

Connection Error #157

Closed matt-stuart closed 7 years ago

matt-stuart commented 7 years ago

Since 2.0 launched my client has been showing a "Connection Error" message in the top right. Is there a log I can look for for more details?

screen shot 2017-04-26 at 8 55 57 pm

kvting commented 7 years ago

Same here

eivindveg commented 7 years ago

Will review this after work today.

matt-stuart commented 7 years ago

If it means anything, the URL in the source to hit for uploads https://www.hotslogs.com/UploadFile?Source= is responding with not found.

eivindveg commented 7 years ago

That's normal - that URL requires some information about the stashed replay file.

ulrichsg commented 7 years ago

Same here. It uploaded a whole bunch of pre-2.0 replays without problems, so it's specifically an issue with 2.0 replays.

gregahsue commented 7 years ago

I'm seeing this error as well ever since the 2.0 patch.

eivindveg commented 7 years ago

If you wish to help me debug this issue a lot faster, please upload one or two problematic replays to Google Drive or Dropbox and link them here. That way I won't have to play entire matches to produce enough data to verify the bug and fixes.

ulrichsg commented 7 years ago

Here's one: https://www.dropbox.com/s/xbujcnocu3ciojm/Braxis%20Holdout%20%2839%29.StormReplay?dl=0

matt-stuart commented 7 years ago

If you need 3 more: https://www.dropbox.com/sh/i0hheeaeunj14c6/AAC5QjC_7V7SAt-5j77E-GLCa?dl=0

eivindveg commented 7 years ago

Thanks alot. Getting right to work!

eivindveg commented 7 years ago

Verified fixed in https://github.com/eivindveg/HotSReplayParser/commit/1976764f208abcb830f02763ef36c495f39c88ed

Adding a few finishing touches to the parser, then aiming for a release tonight!

eivindveg commented 7 years ago

This issue is resolved in version 2.1.5 which has now been released. Thanks for the extra sample data, it all helped!

ulrichsg commented 7 years ago

Works like a charm. Thanks!

gregahsue commented 7 years ago

Wow, you fixed it faster than I could upload sample errors.

Thank you and great job!

On Apr 28, 2017, at 1:27 AM, Ulrich Schmidt-Goertz notifications@github.com wrote:

Works like a charm. Thanks!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.