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

Updated expectedSize and expectedChecksum #5737

Closed
wants to merge 3 commits into from

Conversation

aalej
Copy link
Contributor

@aalej aalej commented Apr 26, 2023

Description

Proposed fix for #5734.

Looks like the cause is the incorrect expectedSize and expectedChecksum here. expectedSize is 64723779 and expectedChecksum is 963b46bce1a2f8ca5dfc504ab7b2c901.

Scenarios Tested

  1. Deleting the .cache/firebase/emulators/cloud-firestore-emulator-v1.17.2.jar.
  2. Run firebase emulators:start --project demo-proj --only firestore to re-install the emulator.
  3. Firestore emulator is downloaded as intended.

@trakhimenok
Copy link

I wonder if you can put some check in place to prevent this in future?

@joehan
Copy link
Contributor

joehan commented Apr 26, 2023

This was an odd one - we have tests in place to catch this, but we changed the JAR after running them. Ended up fixing this issue by putting the correct JAR in storage.

@joehan joehan closed this Apr 26, 2023
@aalej aalej deleted the fstr-em-bytsz branch May 19, 2023 22:08
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.

None yet

3 participants