tdakkota
Repos
120
Followers
74
Following
39

Telegram client, in Go. (MTProto API)

1099
70

WIP Telegram Bot API server in Go

17
1

Simple linter to check that your code does not contain non-ASCII identifiers

10
1

Golang module for working with VK API

145
28

A Rust API for parsing and working with PSD files.

217
29

win32metadata is Windows metadata parser

13
0

Events

pull request opened
feat(gen): implement text (un)marshaler for string enum
Created at 4 hours ago
create branch
tdakkota create branch feat/encoding-interface-enums
Created at 4 hours ago

go/types, types2: make the new comparable semantics the default

Ordinary interface types now satisfy comparable constraints.

This change makes the new comparable semantics the default behavior, depending on the Go -lang version.

It also renames the flag types2.Config.AltComparableSemantics to types2.Config.OldComparableSemantics and inverts its meaning (or types.Config.oldComparableSemantics respectively).

Adjust some existing tests by setting -oldComparableSemantics and add some new tests that verify version-dependent behavior.

The compiler flag -oldcomparable may be used to temporarily switch back to the Go 1.18/1.19 behavior should this change cause problems, or to identify that a problem is unrelated to this change. The flag will be removed for Go 1.21.

For #52509. For #56548.

Change-Id: Ic2b22db9433a8dd81dc1ed9d30835f0395fb7205 Reviewed-on: https://go-review.googlesource.com/c/go/+/453978 Reviewed-by: Robert Findley rfindley@google.com Reviewed-by: Robert Griesemer gri@google.com Run-TryBot: Robert Griesemer gri@google.com TryBot-Result: Gopher Robot gobot@golang.org Auto-Submit: Robert Griesemer gri@google.com

crypto/ecdsa: verify validity of signature parameters in Verify

CL 353849 removed validation of signature parameters being passed to Verify which led to two distinct problems. If passed a R or S == 0, encodeSignature would panic since it expects them to be non-zero. encodeSignature would also normalize (i.e. make non-negative) parameters by zero padding them, which would result in a signature being passed to VerifyASN1 which did not match the input signature, resulting in success in cases where it should've failed. This change re-adds the verification that 0 < r,s < N before calling ecnodeSignature.

This was caught because tink runs the wycheproof ECDSA vectors against Verify, where we only run the vectors against VerifyASN1. We should be doing both.

Change-Id: I1dcf41626b4df2b43296e8b878dc607ff316a892 Reviewed-on: https://go-review.googlesource.com/c/go/+/453675 Auto-Submit: Roland Shoemaker roland@golang.org Reviewed-by: Filippo Valsorda filippo@golang.org TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Than McIntosh thanm@google.com Run-TryBot: Roland Shoemaker roland@golang.org

archive/zip: tolerate compressed directories with zero uncompressed size

In CL 449955 we made reading of directories with associated file data an error, since it is a "must not" in the zip specification. It turns out that a number of implementations make the mistake of not setting the correct compression method on directories (in particular the Java jar tool does this when storing the META-INF directory). If the compression method used is not 0 (stored) then the compressed size of the directory can be > 0, despite the uncompressed size still being 0.

Since this mistake is not uncommon, we are forced to tolerate it. We still fail if the recorded uncompressed size is > 0, which should be a significantly harder mistake to make.

Change-Id: Ia732b10787f26ab937ac9cf9869ac3042efb8118 Reviewed-on: https://go-review.googlesource.com/c/go/+/454475 Reviewed-by: Ian Lance Taylor iant@google.com Auto-Submit: Roland Shoemaker roland@golang.org TryBot-Result: Gopher Robot gobot@golang.org Run-TryBot: Roland Shoemaker roland@golang.org

internal/coverage: use io.Seek* constants

Use the io.Seek* constants instead of their deprecated os.SEEK_* counterparts.

Change-Id: Ie899f9cf286de7304a394d9a2a091a55937542ad Reviewed-on: https://go-review.googlesource.com/c/go/+/453576 Reviewed-by: Than McIntosh thanm@google.com Auto-Submit: Tobias Klauser tobias.klauser@gmail.com TryBot-Result: Gopher Robot gobot@golang.org Run-TryBot: Tobias Klauser tobias.klauser@gmail.com Reviewed-by: Ian Lance Taylor iant@google.com Run-TryBot: Than McIntosh thanm@google.com

Revert "go/types, types2: make the new comparable semantics the default"

The CL below was accidentally submitted, while waiting for the freeze exception. Reverting.

This reverts commit 15e705ea963b5008112793507365e24b743606bc.

Change-Id: I4dbf92dcb01fa9245a6e6a2d1514d8aa898d0048 Reviewed-on: https://go-review.googlesource.com/c/go/+/454476 Reviewed-by: Robert Griesemer gri@google.com Run-TryBot: Robert Griesemer gri@google.com TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Robert Findley rfindley@google.com Reviewed-by: Cuong Manh Le cuong.manhle.vn@gmail.com Auto-Submit: Robert Griesemer gri@google.com

crypto/x509: include more hints for verification failure

Include hint from isValid, as well as CheckSignatureFrom.

Change-Id: I408f73fc5f12572f1937da50be7fa3e1109164b0 Reviewed-on: https://go-review.googlesource.com/c/go/+/454477 TryBot-Result: Gopher Robot gobot@golang.org Auto-Submit: Roland Shoemaker roland@golang.org Run-TryBot: Roland Shoemaker roland@golang.org Reviewed-by: Damien Neil dneil@google.com

runtime,hash/maphash: eliminate maphash torture test for -race

Disable the "torture" portion of the maphash tests if -race is in effect (these tests can cause timeouts on the longtest -race builder).

Fixes #57030.

Change-Id: I23d7561dac3e81d979cad9e0efa6f5b7154aadd2 Reviewed-on: https://go-review.googlesource.com/c/go/+/454455 Run-TryBot: Than McIntosh thanm@google.com Reviewed-by: Keith Randall khr@golang.org TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Keith Randall khr@google.com

cmd/go/internal/work: make formatOutput return an error that includes the import path

This refines the error output that was previously adjusted in CL 437298.

Longer term, we should consider unraveling the call chains involving formatOutput to avoid passing so many parameters through so many different formatting functions.

Updates #25842.

Change-Id: I3b9d03bf5968902d8ccc4841ab4dbe114a2239e4 Reviewed-on: https://go-review.googlesource.com/c/go/+/451218 Reviewed-by: Bryan Mills bcmills@google.com Auto-Submit: Russ Cox rsc@golang.org Reviewed-by: Russ Cox rsc@golang.org Run-TryBot: Russ Cox rsc@golang.org Reviewed-by: Than McIntosh thanm@google.com TryBot-Result: Gopher Robot gobot@golang.org

go/types, types2: make the new comparable semantics the default

Ordinary interface types now satisfy comparable constraints. This is a fully backward-compatible change: it simply permits additional code to be valid that wasn't valid before.

This change makes the new comparable semantics the default behavior, depending on the Go -lang version.

It also renames the flag types2.Config.AltComparableSemantics to types2.Config.OldComparableSemantics and inverts its meaning (or types.Config.oldComparableSemantics respectively).

Add new predicate Satisfies (matching the predicate Implements but for constraint satisfaction), per the proposal description.

Adjust some existing tests by setting -oldComparableSemantics and add some new tests that verify version-dependent behavior.

The compiler flag -oldcomparable may be used to temporarily switch back to the Go 1.18/1.19 behavior should this change cause problems, or to identify that a problem is unrelated to this change. The flag will be removed for Go 1.21.

For #52509. For #56548. For #57011.

Change-Id: I8b3b3d9d492fc24b0693567055f0053ccb5aeb42 Reviewed-on: https://go-review.googlesource.com/c/go/+/454575 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Robert Griesemer gri@google.com Run-TryBot: Robert Griesemer gri@google.com Reviewed-by: Robert Findley rfindley@google.com

go/internal/gcimporter: fix TestImportStdLib

The test attempted to find all stdlib packages by scanning pkg/$GOOS_$GOARCH for .a files and then tried to import all of them. Now that .a files are no longer being placed there, the test is a noop. Fix this by using go list std (and filtering out testonly packages) and trying to import all of those to recreate what the test intended to do.

This also removes a dependency on the pkg/$GOOS_$GOARCH directory which will stop being produced by dist in CL 453496.

For #47257

Change-Id: I7c1944a89db9da9269def3d64a11408a60d73d46 Reviewed-on: https://go-review.googlesource.com/c/go/+/453858 Reviewed-by: Michael Matloob matloob@golang.org Run-TryBot: Michael Matloob matloob@golang.org Reviewed-by: Bryan Mills bcmills@google.com TryBot-Result: Gopher Robot gobot@golang.org

doc/go1.20: document new semantics for comparable constraint

For #54202. For #56548.

Change-Id: If2b9e41813c3e1c8d373469a40e1bd0bd5ea2b16 Reviewed-on: https://go-review.googlesource.com/c/go/+/454595 Reviewed-by: Ian Lance Taylor iant@golang.org Reviewed-by: Ian Lance Taylor iant@google.com TryBot-Bypass: Robert Griesemer gri@google.com Reviewed-by: Robert Griesemer gri@google.com

doc: add release note for cgo.Incomplete

Updates #46731

Change-Id: Ie64e87d759c48642582342d221b24f77bb81d47a Reviewed-on: https://go-review.googlesource.com/c/go/+/453556 Auto-Submit: Cuong Manh Le cuong.manhle.vn@gmail.com TryBot-Result: Gopher Robot gobot@golang.org Run-TryBot: Cuong Manh Le cuong.manhle.vn@gmail.com Reviewed-by: Russ Cox rsc@golang.org Reviewed-by: Ian Lance Taylor iant@google.com

crypto/des: fix a typo in the comment for permuteFinalBlock

The comment copy pasted from the permuteInitialBlock and should be fixed.

Change-Id: I101f1deceadf9b0480e5b679e4e237bda601950b GitHub-Last-Rev: 7662df772e4ac171c79467678861ac4d7547da78 GitHub-Pull-Request: golang/go#56982 Reviewed-on: https://go-review.googlesource.com/c/go/+/453995 TryBot-Result: Gopher Robot gobot@golang.org Auto-Submit: Ian Lance Taylor iant@google.com Run-TryBot: Ian Lance Taylor iant@google.com Reviewed-by: Ian Lance Taylor iant@google.com Reviewed-by: Russ Cox rsc@golang.org

net: retry with bigger buffer in resSearch

Glibc returns size > bufSize, when the entire dns reply does not fit inside the provided buffer.

Change-Id: Ie1c1c6a3411880bd8bdb4371f1f1b7bcce837ea2 GitHub-Last-Rev: 488cd3ed0db2a86433aa921117b8f1e9192b1fa5 GitHub-Pull-Request: golang/go#57020 Reviewed-on: https://go-review.googlesource.com/c/go/+/454395 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Russ Cox rsc@golang.org Auto-Submit: Ian Lance Taylor iant@google.com Reviewed-by: Ian Lance Taylor iant@google.com Run-TryBot: Ian Lance Taylor iant@google.com

net: acquire thread in resSearch

Change-Id: I042906d8eee8defafbd98f671fd30c2a68281705 GitHub-Last-Rev: 0660c9a989600eeb8652d1228777488d28397731 GitHub-Pull-Request: golang/go#57021 Reviewed-on: https://go-review.googlesource.com/c/go/+/454396 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Russ Cox rsc@golang.org Run-TryBot: Ian Lance Taylor iant@google.com Reviewed-by: Ian Lance Taylor iant@google.com Auto-Submit: Ian Lance Taylor iant@google.com

doc/go1.20: add code tags in crypto/{rsa,subtle} and net/netip sections

Change-Id: I516195fb1c2434feb3bf130d37012a98d77beeb6 Reviewed-on: https://go-review.googlesource.com/c/go/+/454235 Run-TryBot: Tobias Klauser tobias.klauser@gmail.com Reviewed-by: Russ Cox rsc@golang.org Auto-Submit: Tobias Klauser tobias.klauser@gmail.com Reviewed-by: Ian Lance Taylor iant@google.com TryBot-Result: Gopher Robot gobot@golang.org

cmd/dist: remove pkg/$GOOS_$GOARCH and pkg/obj at exit

pkg/obj will be empty, so just remove it. pkg/$GOOS_$GOARCH will be empty unless the user has specified GODEBUG=installgoroot=all, so check if it's empty, and if so, delete it.

Also remove xreaddirfiles, which is unused.

Also remove the copy of pkg/$GOOS_$GOARCH in the cmd/go test TestNewReleaseRebuildsStalePackagesInGOPATH. The directory is empty so copying it has no effect.

For #47257

Change-Id: Ief90b882d157bd16078cd5d2b83a915bfc831f9a Reviewed-on: https://go-review.googlesource.com/c/go/+/453496 Run-TryBot: Michael Matloob matloob@golang.org TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Michael Matloob matloob@golang.org Reviewed-by: Bryan Mills bcmills@google.com

go/internal/gcimporter: load cached export data for packages individually

Previously, we were using internal/goroot.PkgfileMap to locate cached export data. However, PkgfileMap regenerates export data for the entire standard library, whereas gcimporter may only need a single package.

Under the new approach, we load the export data (still using 'go list -export') for each GOROOT package individually, avoiding work to rebuild export data for packages that are not needed. This is a tradeoff: if most packages in GOROOT are actually needed, we end up making many more calls to 'go list', and may build packages sequentially instead of in parallel (but with lower latency to start using the export data from the earlier packages).

On my workstation, starting from a clean cache for each run, this reduces the wall time of 'go test go/internal/gcimporter -run=TestImportedTypes' from 22s real time (2m10s user time) to 6s real (27s user), and only increases 'go test go/internal/gcimporter' from 28s real (2m16s user) to 30s real (2m19s user).

Updates #56967. Updates #47257.

Change-Id: I22556acdd9b1acc56533ed4c2728ea29b585c073 Reviewed-on: https://go-review.googlesource.com/c/go/+/454497 Reviewed-by: Michael Matloob matloob@golang.org Auto-Submit: Bryan Mills bcmills@google.com TryBot-Result: Gopher Robot gobot@golang.org Run-TryBot: Bryan Mills bcmills@google.com

Created at 6 hours ago

doc/website/blog: add new blog post (#3132)

doc/website/blog: rename zero downtime blog (#3135)

entc/gen: remove driver specific implementation for Exist (#3136)

Created at 6 hours ago

docs/website/blog: Picking a database migration tool for Go projects in 2023 (#1321)

Created at 6 hours ago

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #928 from gotd/dependabot/go_modules/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

chore(deps): bump go.uber.org/zap in /examples/bg-run

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

chore(deps): bump go.uber.org/zap in /examples/gif-download

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #930 from gotd/dependabot/go_modules/examples/gif-download/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples/gif-download

Merge pull request #929 from gotd/dependabot/go_modules/examples/bg-run/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples/bg-run

Created at 6 hours ago

refactor(ogenreflect): rename Operation to RuntimeOperation

chore: commit generated files

Created at 8 hours ago

refactor(ogenreflect): rename Operation to RuntimeOperation

chore: commit generated files

Created at 1 day ago

refactor(ogenreflect): rename Operation to RuntimeOperation

chore: commit generated files

Created at 1 day ago

refactor(ogenreflect): rename Operation to RuntimeOperation

chore: commit generated files

Created at 1 day ago

feat(gen): generate API metadata definition

chore: commit generated files

Created at 1 day ago

chore: bump dependencies

chore: commit generated files

Created at 1 day ago

feat(gen): generate runtime information

perf(gen): use map to store operation info

This is a workaround for goimports: it does not treat map lookup as missing package, unlike field lookup of global variable.

Otherwise, goimports will try to search for the non-existing package and will check every package in GOPATH. This leads to a very big slowdown.

Benchstat:

name                        old time/op    new time/op    delta
Generator/api.github.com-4     13.1s ± 1%      4.7s ± 2%  -64.10%  (p=0.000 n=13+15)
Generator/gotd_bot_api-4       8.63s ± 2%     0.91s ± 3%  -89.49%  (p=0.000 n=13+15)
Generator/tinkoff-4            7.96s ± 3%     0.17s ± 6%  -97.90%  (p=0.000 n=15+14)
Generator/manga-4              7.85s ± 2%     0.05s ± 6%  -99.38%  (p=0.000 n=14+14)

name                        old alloc/op   new alloc/op   delta
Generator/api.github.com-4    1.95GB ± 0%    1.48GB ± 0%  -24.32%  (p=0.000 n=15+14)
Generator/gotd_bot_api-4       710MB ± 1%     220MB ± 0%  -69.07%  (p=0.000 n=15+15)
Generator/tinkoff-4            537MB ± 2%      41MB ± 3%  -92.40%  (p=0.000 n=15+15)
Generator/manga-4              507MB ± 2%      11MB ± 2%  -97.87%  (p=0.000 n=15+15)

name                        old allocs/op  new allocs/op  delta
Generator/api.github.com-4     41.1M ± 0%     37.5M ± 0%   -8.63%  (p=0.000 n=15+13)
Generator/gotd_bot_api-4       9.86M ± 0%     6.02M ± 0%  -39.01%  (p=0.000 n=15+14)
Generator/tinkoff-4            5.01M ± 1%     1.15M ± 0%  -77.11%  (p=0.000 n=15+14)
Generator/manga-4              4.17M ± 1%     0.30M ± 0%  -92.82%  (p=0.000 n=15+15)

chore: commit generated files

Created at 1 day ago

feat(ogenreflect): add new runtime information package

refactor(middleware): use new ogenreflect package

refactor(ogenerrors): use new ogenreflect package

feat(gen): generate runtime information

perf(gen): use map to store operation info

This is a workaround for goimports: it does not treat map lookup as missing package, unlike field lookup of global variable.

Otherwise, goimports will try to search for the non-existing package and will check every package in GOPATH. This leads to a very big slowdown.

Benchstat:

name                        old time/op    new time/op    delta
Generator/api.github.com-4     13.1s ± 1%      4.7s ± 2%  -64.10%  (p=0.000 n=13+15)
Generator/gotd_bot_api-4       8.63s ± 2%     0.91s ± 3%  -89.49%  (p=0.000 n=13+15)
Generator/tinkoff-4            7.96s ± 3%     0.17s ± 6%  -97.90%  (p=0.000 n=15+14)
Generator/manga-4              7.85s ± 2%     0.05s ± 6%  -99.38%  (p=0.000 n=14+14)

name                        old alloc/op   new alloc/op   delta
Generator/api.github.com-4    1.95GB ± 0%    1.48GB ± 0%  -24.32%  (p=0.000 n=15+14)
Generator/gotd_bot_api-4       710MB ± 1%     220MB ± 0%  -69.07%  (p=0.000 n=15+15)
Generator/tinkoff-4            537MB ± 2%      41MB ± 3%  -92.40%  (p=0.000 n=15+15)
Generator/manga-4              507MB ± 2%      11MB ± 2%  -97.87%  (p=0.000 n=15+15)

name                        old allocs/op  new allocs/op  delta
Generator/api.github.com-4     41.1M ± 0%     37.5M ± 0%   -8.63%  (p=0.000 n=15+13)
Generator/gotd_bot_api-4       9.86M ± 0%     6.02M ± 0%  -39.01%  (p=0.000 n=15+14)
Generator/tinkoff-4            5.01M ± 1%     1.15M ± 0%  -77.11%  (p=0.000 n=15+14)
Generator/manga-4              4.17M ± 1%     0.30M ± 0%  -92.82%  (p=0.000 n=15+15)

chore: commit generated files

Created at 1 day ago
create tag
tdakkota create tag v0.56.0-alpha.1
Created at 1 day ago

chore(gen): make error reporting consistent

feat(gen): use structured param decoding error

chore: commit generated files

Merge pull request #699 from tdakkota/feat/param-structured-error

feat(gen): use structured param decoding error

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #700 from ogen-go/dependabot/go_modules/examples/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #701 from ogen-go/dependabot/go_modules/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

feat(ogenreflect): add new runtime information package

refactor(middleware): use new ogenreflect package

refactor(ogenerrors): use new ogenreflect package

feat(gen): generate runtime information

refactor(gen): move ogenreflect support to separate template

perf(gen): use map to store operation info

This is a workaround for goimports: it does not treat map lookup as missing package, unlike field lookup of global variable.

Otherwise, goimports will try to search for the non-existing package and will check every package in GOPATH. This leads to a very big slowdown.

Benchstat:

name                        old time/op    new time/op    delta
Generator/api.github.com-4     13.1s ± 1%      4.7s ± 2%  -64.10%  (p=0.000 n=13+15)
Generator/gotd_bot_api-4       8.63s ± 2%     0.91s ± 3%  -89.49%  (p=0.000 n=13+15)
Generator/tinkoff-4            7.96s ± 3%     0.17s ± 6%  -97.90%  (p=0.000 n=15+14)
Generator/manga-4              7.85s ± 2%     0.05s ± 6%  -99.38%  (p=0.000 n=14+14)

name                        old alloc/op   new alloc/op   delta
Generator/api.github.com-4    1.95GB ± 0%    1.48GB ± 0%  -24.32%  (p=0.000 n=15+14)
Generator/gotd_bot_api-4       710MB ± 1%     220MB ± 0%  -69.07%  (p=0.000 n=15+15)
Generator/tinkoff-4            537MB ± 2%      41MB ± 3%  -92.40%  (p=0.000 n=15+15)
Generator/manga-4              507MB ± 2%      11MB ± 2%  -97.87%  (p=0.000 n=15+15)

name                        old allocs/op  new allocs/op  delta
Generator/api.github.com-4     41.1M ± 0%     37.5M ± 0%   -8.63%  (p=0.000 n=15+13)
Generator/gotd_bot_api-4       9.86M ± 0%     6.02M ± 0%  -39.01%  (p=0.000 n=15+14)
Generator/tinkoff-4            5.01M ± 1%     1.15M ± 0%  -77.11%  (p=0.000 n=15+14)
Generator/manga-4              4.17M ± 1%     0.30M ± 0%  -92.82%  (p=0.000 n=15+15)

chore: commit generated files

Created at 1 day ago

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #700 from ogen-go/dependabot/go_modules/examples/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #701 from ogen-go/dependabot/go_modules/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Created at 1 day ago
delete branch
tdakkota delete branch dependabot/go_modules/go.uber.org/zap-1.24.0
Created at 1 day ago

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #701 from ogen-go/dependabot/go_modules/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Created at 1 day ago
pull request closed
chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 1 day ago
delete branch
tdakkota delete branch dependabot/go_modules/examples/go.uber.org/zap-1.24.0
Created at 1 day ago

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #700 from ogen-go/dependabot/go_modules/examples/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

Created at 1 day ago
pull request closed
chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0 in /examples

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 1 day ago
delete branch
tdakkota delete branch feat/param-structured-error
Created at 1 day ago

doc/go1.20: quote code in Bootstrap and Cgo sections

Add code tag for environment variables and file paths in the Bootstrap section.

Add code tag for packages in the Cgo section.

Change-Id: Ib0fad1c09fbc497a097ef43cbf5850a27b9a6532 Reviewed-on: https://go-review.googlesource.com/c/go/+/453621 Run-TryBot: Cherry Mui cherryyz@google.com TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Ian Lance Taylor iant@google.com

doc/go1.20: revise linker release note

I misunderstood CL 420774. We didn't remove GO_LDSO, just that make.bash no longer tries to set it automatically. If GO_LDSO is explicitly set at make.bash, it is still used as the default dynamic interpreter.

For #54202.

Change-Id: Ided775438b8e4b87a6acd9bc87657657dbd3d91c Reviewed-on: https://go-review.googlesource.com/c/go/+/453601 TryBot-Result: Gopher Robot gobot@golang.org Run-TryBot: Cherry Mui cherryyz@google.com Reviewed-by: Than McIntosh thanm@google.com

Revert "encoding/xml: disallow empty namespace when prefix is set"

This reverts CL 105636.

Reason for revert: Fails with existing XML data. At this stage in the release cycle we should revert, and try again next time with some way to support existing XML.

For #8068

Change-Id: Ia84cbf3a84878ac7190f72998545dee22c36c45e Reviewed-on: https://go-review.googlesource.com/c/go/+/453996 Auto-Submit: Ian Lance Taylor iant@golang.org TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Heschi Kreinick heschi@google.com Reviewed-by: Emmanuel Odeke emmanuel@orijtech.com Reviewed-by: Ian Lance Taylor iant@google.com Run-TryBot: Ian Lance Taylor iant@golang.org

reflect: correct tpoy in comment

Change-Id: I75aef6bf957939fae8b680214f5923dc2713a5cc Reviewed-on: https://go-review.googlesource.com/c/go/+/452935 Reviewed-by: Russ Cox rsc@golang.org Run-TryBot: Ian Lance Taylor iant@google.com TryBot-Result: Gopher Robot gobot@golang.org Auto-Submit: Russ Cox rsc@golang.org Auto-Submit: Ian Lance Taylor iant@google.com Reviewed-by: Ian Lance Taylor iant@google.com Run-TryBot: Ian Lance Taylor iant@golang.org

spec: document conversion from slice to array

Document that a slice can be converted to either an array or a pointer to an array of a matching underlying array type. This was documented in the "Conversions from slice to array or array pointer" subsection, but not in the list of conversion rules.

Updates #46505.

Change-Id: I16a89a63ef23c33580129952415e977a8f334009 Reviewed-on: https://go-review.googlesource.com/c/go/+/452936 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Robert Griesemer gri@google.com Run-TryBot: Tim King taking@google.com

cmd/compile: disallow CMOV optimization with ptr arithmetic as an arg

if q != nil {
    p = &q.f
}

Which gets rewritten to a conditional move:

tmp := &q.f
p = Select q!=nil, tmp, p

Unfortunately, we can't compute &q.f before we've checked if q is nil, because if it is nil, &q.f is an invalid pointer (if f's offset is nonzero but small).

Normally this is not a problem because the tmp variable above immediately dies, and is thus not live across any safepoint. However, if later there is another &q.f computation, those two computations are CSEd, causing tmp to be used at both use points. That will extend tmp's lifetime, possibly across a call.

Fixes #56990

Change-Id: I3ea31be93feae04fbe3304cb11323194c5df3879 Reviewed-on: https://go-review.googlesource.com/c/go/+/454155 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Cherry Mui cherryyz@google.com Run-TryBot: Keith Randall khr@golang.org Reviewed-by: Keith Randall khr@google.com

net/http: deflake and fix TestWrappedResponseController

Read the full (empty) response body before closing it, to avoid cancelling the request while the server handler is still running.

Wrap the ResponseWriter before calling NewResponseController: This test is intended to verify that wrapping the controller works properly, but neglected to actually wrap the controller.

Fixes #56961.

Change-Id: I00269f897448ab34676338707b7a04d19ff17963 Reviewed-on: https://go-review.googlesource.com/c/go/+/453860 Run-TryBot: Damien Neil dneil@google.com Reviewed-by: Bryan Mills bcmills@google.com TryBot-Result: Gopher Robot gobot@golang.org

doc/go1.20: add runtime, runtime/trace, and runtime/pprof release notes

This change adds release notes for the "Runtime" section and updated some of the release notes for runtime/trace and runtime/pprof after I looked at the full list of runtime-related changes.

For #54202.

Change-Id: Id1395f4e6e02d3fcc248855ca98ad0ee26cae574 Reviewed-on: https://go-review.googlesource.com/c/go/+/454075 TryBot-Result: Gopher Robot gobot@golang.org Reviewed-by: Cherry Mui cherryyz@google.com Reviewed-by: Robert Griesemer gri@google.com Reviewed-by: Austin Clements austin@google.com Run-TryBot: Michael Knyszek mknyszek@google.com Auto-Submit: Michael Knyszek mknyszek@google.com

net: reenable SRV tests with _ldap._tcp.google.com

TestLookupDotsWithRemoteSource and TestLookupGoogleSRV were disabled because they look up the no-longer-present SRV record for _xmpp-server._tcp.google.com.

Change the tests to look for _ldap._tcp.google.com and reenable them.

Fixes #56708.

Change-Id: I26475fa3ff6fc008048a4e5f24f0e96ee12f655c Reviewed-on: https://go-review.googlesource.com/c/go/+/453861 Run-TryBot: Damien Neil dneil@google.com Reviewed-by: Michael Knyszek mknyszek@google.com TryBot-Result: Gopher Robot gobot@golang.org

cmd/cgo: walk {FuncType,TypeSpec}.TypeParams fields

This CL updates the cgo tool to walk the TypeParams fields for function types and type declarations, so that C.xxx identifiers can appear within type parameter lists.

Fixes #52542.

Change-Id: Id02a88d529d50fe59b0a834f415c2575204ffd1f Reviewed-on: https://go-review.googlesource.com/c/go/+/453977 Run-TryBot: Matthew Dempsky mdempsky@google.com Reviewed-by: Ian Lance Taylor iant@google.com Auto-Submit: Matthew Dempsky mdempsky@google.com TryBot-Result: Gopher Robot gobot@golang.org

cmd/compile: fix inline static init with derived types

CL 450136 added handling for simple calls in staticinit. If there's any derived types conversion in the body of generic function called, that conversion will require runtime dictionary, thus the optimization could not happen.

Fixes #56923

Change-Id: I498cee9f8ab4397812ef79a6c2ab6c55e0ee4aef Reviewed-on: https://go-review.googlesource.com/c/go/+/453315 TryBot-Result: Gopher Robot gobot@golang.org Auto-Submit: Cuong Manh Le cuong.manhle.vn@gmail.com Reviewed-by: Keith Randall khr@google.com Reviewed-by: Matthew Dempsky mdempsky@google.com Run-TryBot: Cuong Manh Le cuong.manhle.vn@gmail.com Reviewed-by: Gabriel Morency (Amgc63spaming) morencyvincent8@gmail.com

Created at 1 day ago

chore(gen): make error reporting consistent

feat(gen): use structured param decoding error

chore: commit generated files

Merge pull request #699 from tdakkota/feat/param-structured-error

feat(gen): use structured param decoding error

Created at 1 day ago

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Bumps go.uber.org/zap from 1.23.0 to 1.24.0.


updated-dependencies:

  • dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #215 from ClickHouse/dependabot/go_modules/go.uber.org/zap-1.24.0

chore(deps): bump go.uber.org/zap from 1.23.0 to 1.24.0

Created at 1 day ago