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

build(deps): bump google.golang.org/protobuf from 1.27.1 to 1.28.0 #474

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 23, 2022

Bumps google.golang.org/protobuf from 1.27.1 to 1.28.0.

Release notes

Sourced from google.golang.org/protobuf's releases.

v1.28.0

Overview

The release provides a new unmarshal option for limiting the recursion depth when unmarshalling nested messages to prevent stack overflows. (UnmarshalOptions.RecursionLimit).

Notable changes

New features:

  • CL/340489: testing/protocmp: add Message.Unwrap

Documentation improvements:

  • CL/339569: reflect/protoreflect: add more docs on Value aliasing

Updated supported versions:

UnmarshalOption RecursionLimit

  • CL/385854: all: implement depth limit for unmarshalling

The new UnmarshalOptions.RecursionLimit limits the maximum recursion depth when unmarshalling messages. The limit is applied for nested messages. When messages are nested deeper than the specified limit the unmarshalling will fail. If unspecified, a default limit of 10,000 is applied.

In addition to the configurable limit for message nesting a non-configurable recursion limit for group nesting of 10,000 was introduced.

Upcoming breakage changes

The default recursion limit of 10,000 introduced in the release is subject to change. We want to align this limit with implementations for other languages in the long term. C++ and Java use a limit of 100 which is also the target for the Go implementation.

Commits
  • 32051b4 all: release v1.28.0
  • 3992ea8 all: implement depth limit for unmarshaling
  • e5db296 all: update supported versions
  • 3a9e1dc all: gofmt all
  • 26e8bcb all: remove unnecessary string([]byte) conversion in fmt.Sprintf with %s
  • 5aec41b testing/protocmp: add Message.Unwrap
  • 05be61f reflect/protoreflect: add more docs on Value aliasing
  • b03064a all: start v1.27.1-devel
  • See full diff in compare view

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.


Dependabot commands and options

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)

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Mar 23, 2022
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 23, 2022

Dependabot tried to add @alexanderbez and @fedekunze as reviewers to this PR, but received the following error from GitHub:

POST https://api.github.com/repos/line/lbm-sdk/pulls/474/requested_reviewers: 422 - Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the line/lbm-sdk repository. // See: https://docs.github.com/rest/reference/pulls#request-reviewers-for-a-pull-request

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Mar 23, 2022

The following labels could not be found: automerge.

@codecov
Copy link

codecov bot commented Mar 23, 2022

Codecov Report

Merging #474 (c78a5ad) into main (396a1fe) will increase coverage by 0.03%.
The diff coverage is 74.19%.

❗ Current head c78a5ad differs from pull request most recent head e16561d. Consider uploading reports for the commit e16561d to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #474      +/-   ##
==========================================
+ Coverage   56.75%   56.79%   +0.03%     
==========================================
  Files         777      777              
  Lines       83389    83369      -20     
==========================================
+ Hits        47331    47352      +21     
+ Misses      33004    32961      -43     
- Partials     3054     3056       +2     
Impacted Files Coverage Δ
x/wasm/client/cli/tx.go 14.54% <0.00%> (ø)
x/wasm/keeper/msg_server.go 0.00% <0.00%> (ø)
x/consortium/keeper/genesis.go 19.23% <19.23%> (ø)
x/wasm/keeper/query_plugins.go 73.26% <40.00%> (-0.51%) ⬇️
x/consortium/keeper/keeper.go 90.00% <66.66%> (ø)
x/wasm/types/tx.go 40.72% <69.23%> (+1.31%) ⬆️
x/wasm/keeper/legacy_querier.go 65.51% <80.00%> (ø)
x/consortium/keeper/proposal.go 83.33% <83.33%> (ø)
simapp/app.go 86.94% <100.00%> (ø)
x/consortium/client/testutil/grpc.go 94.64% <100.00%> (ø)
... and 11 more

Bumps [google.golang.org/protobuf](https://github.com/protocolbuffers/protobuf-go) from 1.27.1 to 1.28.0.
- [Release notes](https://github.com/protocolbuffers/protobuf-go/releases)
- [Changelog](https://github.com/protocolbuffers/protobuf-go/blob/master/release.bash)
- [Commits](protocolbuffers/protobuf-go@v1.27.1...v1.28.0)

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/google.golang.org/protobuf-1.28.0 branch from 2207ea7 to 0e0b839 Compare April 4, 2022 04:17
…ot/go_modules/google.golang.org/protobuf-1.28.0
@zemyblue zemyblue requested review from 0Tech and dudong2 May 2, 2022 10:17
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ zemyblue
❌ mergify[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

@mergify mergify bot merged commit 0b28990 into main May 9, 2022
@mergify mergify bot deleted the dependabot/go_modules/google.golang.org/protobuf-1.28.0 branch May 9, 2022 08:08
brew0722 pushed a commit to brew0722/lbm-sdk that referenced this pull request May 11, 2022
* main:
  fix: genesis auth account format (Finschia#517)
  Remove reviewers on dependabot.yml
  updated ostracon to v1.0.5; `unsafe-reset-all` command has been moved to the `ostracon` sub-command. (Finschia#536)
  build(deps): bump docker/build-push-action from 2 to 3 (Finschia#532)
  build(deps): bump docker/metadata-action from 3 to 4 (Finschia#530)
  build(deps): bump docker/login-action from 1 to 2 (Finschia#531)
  build(deps): bump docker/setup-buildx-action from 1 to 2 (Finschia#529)
  chore(deps): bump github.com/VictoriaMetrics/fastcache from 1.9.0 to 1.10.0 (Finschia#499)
  build(deps): bump google.golang.org/protobuf from 1.27.1 to 1.28.0 (Finschia#474)
  chore(deps): bump github.com/prometheus/common from 0.32.1 to 0.34.0 (Finschia#510)
  chore(deps): bump github.com/coinbase/rosetta-sdk-go from 0.7.0 to 0.7.9 (Finschia#520)
  build(deps): bump github.com/spf13/cobra from 1.3.0 to 1.4.0 (Finschia#458)
  build(deps): bump github.com/confio/ics23/go from 0.6.6 to 0.7.0 (Finschia#448)
  build(deps): bump github.com/stretchr/testify from 1.7.0 to 1.7.1 (Finschia#465)
  build(deps): bump github.com/armon/go-metrics from 0.3.10 to 0.3.11 (Finschia#522)
  build(deps): bump github.com/magiconair/properties from 1.8.5 to 1.8.6 (Finschia#445)
  ci: change automerge conditions of Mergify (Finschia#523)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants