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

Adding JVM support for Firebase Storage #704

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pontomedon
Copy link

@pontomedon pontomedon commented Feb 18, 2025

This is based on previous work by Cesar Ramirez and it is obviously depending on work done on the java sdk: GitLiveApp/firebase-java-sdk#44

I know the checks are failing, I'll need some input on how I can publish a snapshot of firebase-java-sdk to make them succeed. It all works locally when deploying to maven local.

I needed to make sure we never call the firebase storage API with null metadata - when doing this on a desktop target, the CONTENT_LENGTH = 0 header gets removed somehow, causing the firebase backend to respond with a HTTP 411. Passing defaults causes a few bytes to be serialized in the first requrest before the upload, which circumvents the problem.

The root cause for this must be in the http driver somewhere, as it doesn't happen when running the same code on Android.

@pontomedon
Copy link
Author

fixes #689 and #533

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