godot icon indicating copy to clipboard operation
godot copied to clipboard

TLS handshake error in AssetLib

Open EAinsley opened this issue 1 year ago • 4 comments

Tested versions

  • Reproducible in v4.4.dev2.official [97ef3c837], v4.4.dev3.official [f4af8201b] and later. (error -110 and -28928)
  • In v4.3.stable.official [77dcf97d8], v4.4.dev1.official [28a72fa43]. It gives different error. (error -9984)

System information

Godot v4.4.dev2 - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; 31.0.15.3623) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)

Issue description

When change pages in Asset Store, it will sometimes give TLS handshake error.

TLS error

~And sometimes the editor will crash.~ (This is unrelated and have been fixed)

Computer network connection is fine and no proxy used.

Probably related to #79162 .

Steps to reproduce

  • Create an empty project.
  • Click AssetLib
  • (Change pages)
  • wait some time

Minimal reproduction project (MRP)

N/A

EAinsley avatar Oct 03 '24 10:10 EAinsley

I tested this on 4.4 dev3 and it does happen err

JekSun97 avatar Oct 03 '24 18:10 JekSun97

I've only been able to get the -110 handshake error which appears since 8ffb7699af98790b6a39c0d425945145e199e67a (even without changing pages rapidly) (this also was introduced in dev3 and the issue was reported against dev2).

ImageLoaderSVG: Failed to create SVG from buffer, error code 30. shouldn't be related, this happens when assets use unsupported image formats (like gif) because the editor will try to parse the data as svg if no other supported format matches.

HolonProduction avatar Oct 03 '24 21:10 HolonProduction

Sounds like #97078. Please try the latest dev3 and see if it still crashes.

timothyqiu avatar Oct 04 '24 04:10 timothyqiu

Sounds like #97078. Please try the latest dev3 and see if it still crashes.

You're right. The crash has been fixed, but TLS handshake error still exists. I will update the issue.

EAinsley avatar Oct 04 '24 05:10 EAinsley

Issue is still present in godot 4.4.beta2, the assets lib keeps throwing error, seem like an issue with image fetching ?

multiple ERROR: modules/mbedtls/stream_peer_mbedtls.cpp:88 - TLS handshake error: -110

and a single ERROR: modules/mbedtls/stream_peer_mbedtls.cpp:88 - TLS handshake error: -28928

Seubmarine avatar Feb 07 '25 16:02 Seubmarine

I noticed it only happens when the editor setting asset_library/use_threads is true. The number of errors also matches the number of icons that didn't load.

dugramen avatar Feb 16 '25 23:02 dugramen

I ran into some of these errors in 4.4.beta3, but they seemed fixed by #102774.

Now in the latest master branch I only seem to reproduce this one rarely:

ERROR: TLS handshake error: -9984
   at: _do_handshake (modules/mbedtls/stream_peer_mbedtls.cpp:88)
mbedtls error: returned -0x2700

In my test it happens on this asset: https://godotengine.org/asset-library/asset/2995 which uses an icon from a HTTPS URL with an invalid certificate.

I suggest we can close this issue as fixed as the handshake errors -110 and -28928 were fixed by #102774. I'll open a new issue for handshake error -9984.

akien-mga avatar Feb 17 '25 14:02 akien-mga