Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0050846 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [POS2] Core | major | have not tried | 2022-11-10 13:21 | 2022-11-14 16:51 | |||
| Reporter | cberner | View Status | public | |||||
| Assigned To | cberner | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 22Q4.1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | 22Q4.1 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0050846: On occasions service worker doesn't properly use the updated files | |||||||
| Description | It seems on occasions, there is a reload of the application when a new version of the builds is present and the files being used after the refresh are the same as the old version, another refresh is required for this to be fixed afterwards. The main problem seems to be that the reload of the application could generate a race condition with the activation of the new service worker, we should refresh only afterwards. | |||||||
| Steps To Reproduce | 1. Apply this diff serviceWorker.diff attached, it adds a setTimeout on the processing of the skipWaiting message of the service worker 2. Run ant build in core2 3. Start Tomcat server, and use the production build: localhost:8080/openbravo/web/pos 4. Modify the service-worker.js file in WebContent/web/pos/static to generate a new update, you can modify one of the .chunk.js files to see which one is the web page currently using After modification, refresh the webpage, check that we're still using the older version, instead of the new one. You can check this by looking in dev-tools at the Sources tab, check the files that you modified and updated the version. | |||||||
| Proposed Solution | Refresh the application on service worker activation, after the messages has been properly created. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0143581) hgbot (developer) 2022-11-14 16:03 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/914 [^] |
|
(0143585) hgbot (developer) 2022-11-14 16:51 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^] Changeset: aa28c314febdabc03b312484b0994b71332e9de8 Author: Cristian Berner <cristian.berner@openbravo.com> Date: 14-11-2022 16:03:03 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/aa28c314febdabc03b312484b0994b71332e9de8 [^] Fixes ISSUE-50846: On occasions service worker doesn't properly use the updated files There is a race-condition when changes are detected in the service worker, and a reload is required. The reload was happening just after sending a Skip Waiting message to the service-worker thread, which sometimes does not allow the service worker to finish its process of fetching + activation, as such it will keep old version .js files until another refresh happens. To avoid this, we refresh the application after we sent the message, but once the service worker is not waiting anymore, and is in an "activated" state. --- M web-jspack/org.openbravo.core2/src/registerServiceWorker.js --- |
|
(0143588) hgbot (developer) 2022-11-14 16:51 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/914 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-11-10 16:45 | guilleaer | Type | defect => backport |
| 2022-11-10 16:45 | guilleaer | Target Version | => 22Q4.1 |
| 2022-11-14 16:03 | hgbot | Note Added: 0143581 | |
| 2022-11-14 16:51 | hgbot | Resolution | open => fixed |
| 2022-11-14 16:51 | hgbot | Status | scheduled => closed |
| 2022-11-14 16:51 | hgbot | Fixed in Version | => 22Q4.1 |
| 2022-11-14 16:51 | hgbot | Note Added: 0143585 | |
| 2022-11-14 16:51 | hgbot | Note Added: 0143588 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |