Let navigator follow redirect instead that handling redirect in fetch (#7500)
* Let navigator follow redirect instead that handling redirect in fetch * Do not use cache when fetched resource is to redirect
This commit is contained in:
		
				
					committed by
					
						
						Eugen Rochko
					
				
			
			
				
	
			
			
			
						parent
						
							17c1a62ec8
						
					
				
				
					commit
					65d6b253fb
				
			@@ -10,7 +10,7 @@ function openWebCache() {
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
function fetchRoot() {
 | 
			
		||||
  return fetch('/', { credentials: 'include' });
 | 
			
		||||
  return fetch('/', { credentials: 'include', redirect: 'manual' });
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
const firefox = navigator.userAgent.match(/Firefox\/(\d+)/);
 | 
			
		||||
@@ -31,14 +31,10 @@ self.addEventListener('fetch', function(event) {
 | 
			
		||||
    const asyncResponse = fetchRoot();
 | 
			
		||||
    const asyncCache = openWebCache();
 | 
			
		||||
 | 
			
		||||
    event.respondWith(asyncResponse.then(response => {
 | 
			
		||||
      if (response.ok) {
 | 
			
		||||
        return asyncCache.then(cache => cache.put('/', response.clone()))
 | 
			
		||||
                         .then(() => response);
 | 
			
		||||
      }
 | 
			
		||||
 | 
			
		||||
      throw null;
 | 
			
		||||
    }).catch(() => asyncCache.then(cache => cache.match('/'))));
 | 
			
		||||
    event.respondWith(asyncResponse.then(
 | 
			
		||||
      response => asyncCache.then(cache => cache.put('/', response.clone()))
 | 
			
		||||
                            .then(() => response),
 | 
			
		||||
      () => asyncCache.then(cache => cache.match('/'))));
 | 
			
		||||
  } else if (url.pathname === '/auth/sign_out') {
 | 
			
		||||
    const asyncResponse = fetch(event.request);
 | 
			
		||||
    const asyncCache = openWebCache();
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user