cloudflared icon indicating copy to clipboard operation
cloudflared copied to clipboard

FTBFS due to quic-go not supporting go 1.20

Open grawlinson opened this issue 2 years ago • 5 comments
trafficstars

Describe the bug cloudflared does not compile on go 1.20 due to the quic-go dependency specified in go.mod not supporting 1.20.

As of ~13 hours ago, 1.20 is supported upstream.

To Reproduce Compile with go 1.20 and it will fail with the following error:

# github.com/lucas-clemente/quic-go/internal/qtls
../pkg/mod/github.com/chungthuang/[email protected]/internal/qtls/go120.go:6:13: cannot use "The version of quic-go you're using can't be built on Go 1.20 yet. For more details, please see https://github.com/lucas-clemente/quic-go/wiki/quic-go-and-Go-versions." (untyped string constant "The version of quic-go you're using can't be built on Go 1.20 yet. F...) as int value in variable declarat

The dependency on quic-go should probably be updated sooner rather than later.

grawlinson avatar Feb 04 '23 08:02 grawlinson

Thanks for flagging this @grawlinson

sudarshan-reddy avatar Feb 04 '23 08:02 sudarshan-reddy

this is going to be extra tricky, since cloudflared does not use upstream quic-go, but a fork. would you accept a PR that changed to using the latest upstream quic-go release?

mraerino avatar Feb 16 '23 12:02 mraerino

@mraerino : There's work being done on this. I'll tag in @bwesterb here.

sudarshan-reddy avatar Feb 16 '23 12:02 sudarshan-reddy

We actually do use upstream quic-go, but replace the underlying qtls library. I prepared one for 1.20 yesterday, so it's ready to go.

bwesterb avatar Feb 16 '23 12:02 bwesterb

cloudflared-2023.3.1 go version go1.20.2 darwin/arm64

github.com/lucas-clemente/quic-go/internal/qtls vendor/github.com/lucas-clemente/quic-go/internal/qtls/go120.go:6:13: cannot use "The version of quic-go you're using can't be built on Go 1.20 yet. For more details, please see https://github.com/lucas-clemente/quic-go/wiki/quic-go-and-Go-versions." (untyped string constant "The version of quic-go you're using can't be built on Go 1.20 yet. F...) as int value in variable declaration

b0gdanw avatar Mar 14 '23 10:03 b0gdanw

any update?

1715173329 avatar Mar 24 '23 09:03 1715173329

FYI, go 1.21 will be released soon (see rc2 announcement), which will make go 1.19 not maintained anymore.

stefanb avatar Jun 22 '23 07:06 stefanb

What’s the latest on this?

AndrewBucklin avatar Jul 09 '23 12:07 AndrewBucklin

Successfully built cloudflared 2023.7.1 from source with go version go1.20.6 darwin/arm64

b0gdanw avatar Jul 17 '23 08:07 b0gdanw

Same as above on x86_64

i0ntempest avatar Jul 17 '23 18:07 i0ntempest

Seems that it was already fixed in 2023.5.1 via TUN-7227: Migrate to devincarr/quic-go.

stefanb avatar Jul 18 '23 03:07 stefanb