Bug 152477

Summary:

Limit cached redirect chain length

Product:

WebKit

Reporter:

Antti Koivisto <koivisto>

Component:

Page Loading

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, beidson, cdumez

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	kling: review+

Description Antti Koivisto 2015-12-21 06:20:11 PST

Networking layer prevents cycles and limits the redirect chain length so creating cache cycles is difficult. For robustness the network cache should still limit the maximum redirect chain length.

Comment 1 Antti Koivisto 2015-12-21 06:54:13 PST

Created attachment 267749 [details]
patch

Comment 2 Andreas Kling 2015-12-21 11:13:36 PST

Comment on attachment 267749 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=267749&action=review

r=me

> Source/WebKit2/NetworkProcess/NetworkResourceLoader.cpp:53
> +// Networking layer follows over 30 redirects but caching that many seems unnecessary.

This comment sounds platform-specific.

Comment 3 Antti Koivisto 2015-12-21 11:57:30 PST

https://trac.webkit.org/r194337