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

Firebase 6.0.4 and web worker #1817

Closed
universse opened this issue May 24, 2019 · 7 comments · Fixed by #1819
Closed

Firebase 6.0.4 and web worker #1817

universse opened this issue May 24, 2019 · 7 comments · Fixed by #1819
Assignees

Comments

@universse
Copy link

Hi. I'm using firebase authentication, firestore and storage in web worker. After upgrading from version 6.02 to 6.04 i'm starting to get ReferenceError: window is not defined. It seems to come from firestore.

@google-oss-bot
Copy link
Contributor

I found a few problems with this issue:

  • I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
  • This issue does not seem to follow the issue template. Make sure you provide all the required information.

@Feiyang1
Copy link
Member

Thanks for reporting it! We have made a fix #1819

@schmidt-sebastian
Copy link
Contributor

@universse Are you able to pin your Firestore release to 6.0.2?

@Feiyang1
Copy link
Member

@universse Can you also post a detailed log, so we can verify we actually fixed what is causing the issue for you?

@universse
Copy link
Author

universse commented May 25, 2019

Hi. I reverted to 6.02 when I reported this issue. After upgrading to 6.04 again just now, there's no more issue. Thank you very much!

@schmidt-sebastian
Copy link
Contributor

I'm glad to hear that you are no longer blocked! Please note that the fix for your issue is, however, not yet released and will likely be part of 6.0.5. If you do see this issue again, please upgrade to 6.0.5 when it gets released.

@universse
Copy link
Author

universse commented May 25, 2019

The issue cropped up again. Somehow it did not show up earlier. Sorry for any confusion caused.

From firefox
image

From chrome
image

@Feiyang1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants