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

TLS handshake error in AssetLib #97762

Closed
EAinsley opened this issue Oct 3, 2024 · 7 comments
Closed

TLS handshake error in AssetLib #97762

EAinsley opened this issue Oct 3, 2024 · 7 comments

Comments

@EAinsley
Copy link
Contributor

EAinsley commented Oct 3, 2024

Tested versions

  • Reproducible in v4.4.dev2.official [97ef3c8], v4.4.dev3.official [f4af820] and later. (error -110 and -28928)
  • In v4.3.stable.official [77dcf97], v4.4.dev1.official [28a72fa]. 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

@JekSun97
Copy link
Contributor

JekSun97 commented Oct 3, 2024

I tested this on 4.4 dev3 and it does happen
err

@HolonProduction
Copy link
Member

I've only been able to get the -110 handshake error which appears since 8ffb769 (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.

@timothyqiu
Copy link
Member

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

@EAinsley
Copy link
Contributor Author

EAinsley commented Oct 4, 2024

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 EAinsley changed the title Editor crashes when change page in AssetLib TLS handshake error in AssetLib Oct 4, 2024
@Seubmarine
Copy link

Seubmarine commented Feb 7, 2025

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

@dugramen
Copy link
Contributor

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.

@akien-mga akien-mga added this to the 4.4 milestone Feb 17, 2025
@akien-mga
Copy link
Member

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.

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

No branches or pull requests

9 participants