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

Upgrade firebase-admin, avoid gRPC version conflict #4088

Closed

Conversation

ghubstan
Copy link
Contributor

@ghubstan ghubstan commented Mar 21, 2020

The latest firebase-admin (v6.12.2) depends on grpc-api v1.24.1,
which is as close as we can get to the grpc v1.25.0 specified in
build.gradle. Currently, the firebase-admin version requirement
is causing the build to download grpc v1.10.1 jars, but at runtime,
grpc 1.25.0 jars are loaded.

Transitive dependencies grpc-netty-shaded, grpc-protobuf, grpc-protobuf-lite,
grpc-protobuf-stub, grpc-protobuf-grpclb (v1.23.0) are also excluded.

Eliminating version conflicts saves a dev time thinking about which
source jar to choose from while examining exception stack traces.

Partial fix for #4086

@ghubstan ghubstan requested a review from cbeams as a code owner March 21, 2020 23:14
@ghubstan ghubstan force-pushed the upgrade-firebase-admin branch 2 times, most recently from 06596cf to 1a4451e Compare March 22, 2020 00:01
The latest firebase-admin (v6.12.2) depends on grpc-api v1.24.1,
which is as close as we can get to the grpc v1.25.0 specified in
build.gradle.  Currently, the firebase-admin version requirement
is causing the build to download grpc v1.10.1 jars, but at runtime,
grpc 1.25.0 jars are loaded.

Transitive dependencies grpc-netty-shaded, grpc-protobuf,
grpc-protobuf-lite, grpc-protobuf-stub, grpc-protobuf-grpclb
(v1.23.0) are also excluded.

Upgraded slf4j version requirement to match that of other
dependencies.  This removes a duplicated dependency, and corrects
the gradle-witness.gradle slf4-api entry, which verified slf4-api
v1.7.22, not the transitive dependency loaded at runtime.

Beyond avoiding general version hell issues, eliminating version
conflicts saves a dev time thinking about which source jar to
choose from while examining exception stack traces.

Partial fix for bisq-network#4086
@ghubstan ghubstan force-pushed the upgrade-firebase-admin branch from baa92ae to ca18bc3 Compare March 22, 2020 13:35
@ghubstan
Copy link
Contributor Author

This has to be cancelled because it breaks bisq-daemon.

PR #4089 is OK (it just upgrades apache and logback deps.)

@ghubstan ghubstan closed this Mar 23, 2020
@ghubstan ghubstan deleted the upgrade-firebase-admin branch April 23, 2020 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant