Bug 28431 - Application Cache doesn't work with HTTP authentication
Summary: Application Cache doesn't work with HTTP authentication
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-18 10:18 PDT by Alexey Proskuryakov
Modified: 2009-08-18 12:04 PDT (History)
0 users

See Also:


Attachments
proposed fix (7.12 KB, patch)
2009-08-18 10:27 PDT, Alexey Proskuryakov
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2009-08-18 10:18:05 PDT
When updating an application cache, the user is not asked for credentials, which makes sense. However, credentials that already exist in credential storage also aren't used.

I'm not sure if it makes huge sense to cache authenticated resources - once they are cached, they become available to anyone who asks, even after browser restart. Also, there is no way to make appcache store several versions of content that depends on user identity (cookies have the same problem). But if web developers want to, we probably shouldn't stop them. This works in Firefox, FWIW.
Comment 1 Alexey Proskuryakov 2009-08-18 10:27:18 PDT
Created attachment 35048 [details]
proposed fix
Comment 2 Darin Adler 2009-08-18 10:29:42 PDT
Comment on attachment 35048 [details]
proposed fix

> +    // For normal resource loading, WebKit client is asked about each resource individually. Since application cache does not belong to any particular document,
> +    // the existing client callback cannot be used, and just assume that any client that enables application cache also wants it to use credential storage.

Should say "so assume" rather than "and just assume".

r=me
Comment 3 Alexey Proskuryakov 2009-08-18 12:04:04 PDT
Committed revision 47439.