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

Unable to confirm transaction in Deno #2461

Closed
livthomas opened this issue Apr 8, 2024 · 6 comments · Fixed by #2500
Closed

Unable to confirm transaction in Deno #2461

livthomas opened this issue Apr 8, 2024 · 6 comments · Fixed by #2500
Labels
bug Something isn't working released

Comments

@livthomas
Copy link

livthomas commented Apr 8, 2024

Overview

Since #1120 was closed, I assume this library now supports Deno runtime. Some use cases work but not the Connection.confirmTransaction method.

Steps to reproduce

https://github.com/rockawayx-labs/bug-reproducers/tree/dev/solana-labs/solana-web3.js/deno

Description of bug

The following warning and error is being printed to the console:

Warning: Not implemented: ClientRequest.options.createConnection
ws error: Invalid WebSocket frame: invalid UTF-8 sequence

It looks like it is a missing feature in Deno: denoland/deno#19507 However, other libraries were able to work around it as you can see in the discussion (e.g. discord.js).

Repro for 2.x line: https://codesandbox.io/p/devbox/flamboyant-villani-f4kgd7

@livthomas livthomas added the bug Something isn't working label Apr 8, 2024
@benjiqq
Copy link

benjiqq commented Apr 8, 2024

thanks for reporting, getting the same. so would need to patch web3.js?

@steveluscher
Copy link
Collaborator

Awesome report. Thank you.

We won't be fixing this for the 1.x library (accepting PRs though) but I will use this issue to track the bug for the 2.x line.

Copy link
Contributor

🎉 This issue has been resolved in version 1.91.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jcardama
Copy link

Just tested with v1.91.6 on Deno, and the issue still persists.

Warning: Not implemented: ClientRequest.options.createConnection
ws error: Invalid WebSocket frame: invalid UTF-8 sequence

The issue was fixed for v2.x so until v2.x is released or it gets fixed for 1.x the issue will remain.

@steveluscher
Copy link
Collaborator

That's right. We're generally not working on the 1.x line any longer, but I'll happily review a PR if a Deno user would like to propose one.

It might be easier to send a PR to the rpc-websockets project, but again I haven't looked into it.

Copy link
Contributor

Because there has been no activity on this issue for 7 days since it was closed, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants