You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am trying to update one of my app from TinyGo 1.29 to 1.31.x, and running into a problem because package net/http/httptrace no longer exists. an issue in TinyGo repo has been reported for this here: tinygo-org/tinygo#4186
is there any objection (or obvious known blockers) to adding the package in this repo? I am happy to try to submit a PR for that if this is ok to add.
many thanks for considering
Rajat Jindal
The text was updated successfully, but these errors were encountered:
to implement this, I was thinking of doing following:
add /src/internal/nettrace/nettrace.go - this just has some type definitions, so should be straightforward to add
add /http/httptrace/trace.go - bring the code over from official golang net package and verify the tests.
There is a possibility that this might not work, in which case, I was thinking we can just stub out the exported functions. This will atleast allow the compilation to succeed.
Also happy to hear if there are other approaches we can take for making progress on this issue.
There is a possibility that this might not work, in which case, I was thinking we can just stub out the exported functions. This will atleast allow the compilation to succeed.
This does introduce the problem that code that works with Big Go will compile but fail to actually work with TinyGo, which seems more dangerous than having to reimplement bits of httputil/httptrace in your own code though 😅
hello folks,
Thank you for your wonderful work on TinyGo.
I am trying to update one of my app from TinyGo 1.29 to 1.31.x, and running into a problem because package
net/http/httptrace
no longer exists. an issue in TinyGo repo has been reported for this here: tinygo-org/tinygo#4186is there any objection (or obvious known blockers) to adding the package in this repo? I am happy to try to submit a PR for that if this is ok to add.
many thanks for considering
Rajat Jindal
The text was updated successfully, but these errors were encountered: