w3c / w3c/ServiceWorker

Batch Cache Operations can't return an empty list

Open
#1,677 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Bikeshed
Stars
3.6k
Forks
324
Avg merge
14d 22h
Merged PRs (30d)
1

Description

For batch cache operations, a request/response is appended to the list for every operation, so the list always has a size of number of operations.

Then the cache.delete steps are

  • Let requestResponses be the result of running Batch Cache Operations with operations. If this throws an exception, set errorData to the exception.
  • If requestResponses is not empty, resolve cacheJobPromise with true.

But this also means that when no entries are matched, it also returns a length of 1, meaning it always should be returning true?

I'm sorry if I'm misunderstanding anything but I'm stuck here.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the linked Batch Cache Operations algorithm and the cache.delete steps in the Service Workers specification. Trace how an operation with no matching cache entry contributes to requestResponses. Done means the specification clearly defines the empty-match case and the resulting cache.delete value, with any needed wording or algorithm correction reflected in both steps.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.