Remove duplicate check for request prefix

[itschasa]

As mentioned in this comment on the commit:
6ddb73e#r140981857

This check is already performed in the fetch method:

Ultraviolet/src/uv.sw.js

Lines 50 to 58 in c34745f

	async fetch({ request }) {
	/**
	* @type {string|void}
	*/
	let fetchedURL;
	try {
	if (!request.url.startsWith(location.origin + this.config.prefix))
	return await fetch(request);

[incognitotgt]

This is actually necessary.
It's weird

[itschasa]

Does someone mind explaining?

[proudparrot2]

In case someone who id using their own service worker or and didn’t include this check inside of it

[itschasa]

In case someone who id using their own service worker or and didn’t include this check inside of it

Hasn't the check for it always been inside the service worker fetch function? I don't get how doing the check again before it enters the function's scope helps?

[Percslol]

In case someone who id using their own service worker or and didn’t include this check inside of it

Hasn't the check for it always been inside the service worker fetch function? I don't get how doing the check again before it enters the function's scope helps?

no it was added recently because of some weird race conditions, its also good idiot proofing since anyone can modify the service worker and easily mess up this check somehow

[incognitotgt]

race conditions my beloved

[itschasa]

In case someone who id using their own service worker or and didn’t include this check inside of it

Hasn't the check for it always been inside the service worker fetch function? I don't get how doing the check again before it enters the function's scope helps?

no it was added recently because of some weird race conditions, its also good idiot proofing since anyone can modify the service worker and easily mess up this check somehow

Understood, I'll close the PR.