Skip to content
Snippets Groups Projects

Fix with a patch

Merged Eirik Morland requested to merge issue/drupal_cms-3477135:3477135-double into 0.x
2 unresolved threads

Closes #3477135

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
101 101 action: 'service_worker_activated'
102 102 })
103 103 });
104 worker.addEventListener('fetch', event => php.handleFetchEvent(event));
104 worker.addEventListener('fetch', event => {
105 const url = new URL(event.request.url);
106 if (url.pathname.endsWith('.so') || url.pathname.endsWith('.zip') || url.pathname.endsWith('.phpcode') || url.pathname.endsWith('.wasm')) {
107 // Don't download it. These are basically fetch from another service worker,
108 // and never something we want to pass to the CGI worker.
109 return;
110 }
111 return php.handleFetchEvent(event);
  • Matt Glaman added 1 commit

    added 1 commit

    • e860bbae - patch double fetch from php-cgi-wasm

    Compare with previous version

  • Matt Glaman added 1 commit

    added 1 commit

    • 8b12c405 - stop double booting php in worker

    Compare with previous version

  • 25 })
    26 php.addEventListener('output', event => {
    27 event.detail.forEach(detail => {
    28 try {
    29 const data = JSON.parse(detail.trim());
    30 postMessage({
    31 action: `status`,
    32 params,
    33 ...data
    34 })
    35 } catch (e) {
    36 console.log(detail)
    37 }
    38 })
    39 });
    40 php.addEventListener('error', event => console.log(event.detail));
  • Eirik Morland added 1 commit

    added 1 commit

    • ddcb3037 - Remove an error (params is not defined)

    Compare with previous version

  • Matt Glaman added 2 commits

    added 2 commits

    • 76459578 - Revert "Remove an error (params is not defined)"
    • a86f6665 - Revert "stop double booting php in worker"

    Compare with previous version

  • Adam G-H approved this merge request

    approved this merge request

  • Adam G-H added 10 commits

    added 10 commits

    • a765d7cb - 1 commit from branch project:0.x
    • eaef19a8 - Fix with a patch
    • 31a922e3 - Wrap the upstream method instead
    • da401190 - Revert
    • 41555634 - Forgot the url
    • 4e6241d0 - patch double fetch from php-cgi-wasm
    • be1ecbba - stop double booting php in worker
    • 117a7cb5 - Remove an error (params is not defined)
    • 31a686a1 - Revert "Remove an error (params is not defined)"
    • 94c9b7f1 - Revert "stop double booting php in worker"

    Compare with previous version

  • merged

  • Please register or sign in to reply
    Loading