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

[Workbox-background-sync] Sync event fails first time when connectivity is restored #2933

Closed
RoLfBOT opened this issue Sep 1, 2021 · 4 comments
Assignees
Labels
Bug An issue with our existing, production codebase. Needs More Info Waiting on additional information from the community. workbox-background-sync

Comments

@RoLfBOT
Copy link

RoLfBOT commented Sep 1, 2021

Library Affected: workbox-background-sync
workbox-sw, workbox-build, etc.

Browser & Platform: Edge 92 (Win), Chrome 92 (Android)
E.g. Google Chrome v51.0.1 for Android, or "all browsers".

Issue or Feature Request Description:
Background sync is failing for first time when network is restored after being offine. This is specific to v6.2.4 (as per my usage). Was using v6.1.5 which was working fine up until today when I upgraded the packages. Attaching screenshots from bg sync event recording for both versions

When reporting bugs, please include relevant JavaScript Console logs and links to public URLs at which the issue could be reproduced.
v6.2.4
wb-bg-sync-6 2 4

v6.1.5
wb-bg-sync-6 1 5

@jeffposnick jeffposnick added Bug An issue with our existing, production codebase. workbox-background-sync labels Sep 2, 2021
@jeffposnick
Copy link
Contributor

Thanks for letting us know!

@tropicadri, can you investigate and see if this is related to the IndexedDB changes introduced in v6.2.0?

@tropicadri
Copy link
Collaborator

@RoLfBOT I can't reproduce this issue on my side. I'm also using Chrome 92 and I used both versions of Workbox 6.1.5 and 6.2.4. I tried on my Mac and I also tried on Android. I tried a simple demo, with one request from the same origin, is there anything else you can share about how you are reproducing the issue? Can you also share the logs from the console? Here are mine as an example:
Screen Shot 2021-09-07 at 4 29 06 PM

Thank you!

@tropicadri tropicadri added the Needs More Info Waiting on additional information from the community. label Sep 8, 2021
@wilson382
Copy link

This issue is happening to me. To my understanding it usually the first queue which nrnever make it to the server. I will try to do some debugging and add more information

@tomayac
Copy link
Member

tomayac commented Apr 25, 2024

Hi there,

Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

Thanks, and we hope for your understanding!
The Workbox team

@tomayac tomayac closed this as completed Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug An issue with our existing, production codebase. Needs More Info Waiting on additional information from the community. workbox-background-sync
Projects
None yet
Development

No branches or pull requests

5 participants