java-control-plane icon indicating copy to clipboard operation
java-control-plane copied to clipboard

Use the public maven central relase of com.google.api.grpc:proto-goog…

Open sfc-gh-pbennes opened this issue 3 years ago • 3 comments
trafficstars

…le-common-protos

This fixes an issue of duplicate and possibly incompatible classes for com/google/api and com/google/rpc proto generated classes. If a project includes both api and proto-google-common-protos dependencies, there will be duplicate classes on the class path. This is because:

  • these classes are vendored in this repo but not declared in the pom
  • the version of proto-google-common-protos used is not a public release in maven central

Similar to the issue in #171 that #237 will fix.

Signed-off-by: Preston Bennes [email protected]

sfc-gh-pbennes avatar May 13 '22 23:05 sfc-gh-pbennes

Codecov Report

Merging #238 (b0d8ad6) into main (e5b9700) will decrease coverage by 0.40%. The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #238      +/-   ##
============================================
- Coverage     82.48%   82.07%   -0.41%     
+ Complexity      275      274       -1     
============================================
  Files            31       31              
  Lines           982      982              
  Branches         78       78              
============================================
- Hits            810      806       -4     
- Misses          137      140       +3     
- Partials         35       36       +1     
Impacted Files Coverage Δ
...o/envoyproxy/controlplane/cache/TestResources.java 85.78% <0.00%> (-2.11%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e5b9700...b0d8ad6. Read the comment docs.

codecov-commenter avatar May 13 '22 23:05 codecov-commenter

FWIW I also wanted to remove the other google protos but I couldn't find either the protos or classes in any published jar that looked like it was intentionally publishing them.

sfc-gh-pbennes avatar May 13 '22 23:05 sfc-gh-pbennes

@sfc-gh-pbennes Thanks for the PR! I recently got into the same issue and found this fix. Can any reviewers help review this PR? Thanks!

slowsoul avatar Nov 06 '23 19:11 slowsoul