Bug 111772 – CFNetwork cache partitioning does not work properly on subdomains

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Vicki Pfau]

Currently, the CFNetwork cache partition is initialized on NSURLRequests with the wrong partition name--it uses the full domain instead of just the top privately-controlled domain. This causes the partitions to get out of sync and/or just not work properly.

[Vicki Pfau]

Created attachment 192082 [details]
Patch

[David Kilzer (:ddkilzer)]

Comment on attachment 192082 [details]
Patch

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

r=me

> Source/WebCore/platform/network/mac/ResourceRequestMac.mm:193
> +    String partition = cachePartition();
> +    if (!partition.isNull() && !partition.isEmpty())
> +        [NSURLProtocol setProperty:[NSString stringWithUTF8String:partition.utf8().data()] forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];

This would be easier to read if you pulled out the UTF-8 string conversion as a separate local variable:

        NSString *partitionValue = [NSString stringWithUTF8String:partition.utf8().data()];
        [NSURLProtocol setProperty:partitionValue forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];

[Vicki Pfau]

<rdar://problem/13374125>

[Vicki Pfau]

Committed r145161: <http://trac.webkit.org/changeset/145161>

[Darin Adler]

Comment on attachment 192082 [details]
Patch

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

>> Source/WebCore/platform/network/mac/ResourceRequestMac.mm:193
>> +        [NSURLProtocol setProperty:[NSString stringWithUTF8String:partition.utf8().data()] forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];
> 
> This would be easier to read if you pulled out the UTF-8 string conversion as a separate local variable:
> 
>         NSString *partitionValue = [NSString stringWithUTF8String:partition.utf8().data()];
>         [NSURLProtocol setProperty:partitionValue forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];

Why is UTF-8 involved at all? Why not just cachedPartition().nsString()?

[Vicki Pfau]

(In reply to comment #5)
> (From update of attachment 192082 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=192082&action=review
> 
> >> Source/WebCore/platform/network/mac/ResourceRequestMac.mm:193
> >> +        [NSURLProtocol setProperty:[NSString stringWithUTF8String:partition.utf8().data()] forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];
> > 
> > This would be easier to read if you pulled out the UTF-8 string conversion as a separate local variable:
> > 
> >         NSString *partitionValue = [NSString stringWithUTF8String:partition.utf8().data()];
> >         [NSURLProtocol setProperty:partitionValue forKey:(NSString *)wkCachePartitionKey() inRequest:nsRequest];
> 
> Why is UTF-8 involved at all? Why not just cachedPartition().nsString()?

It's an unfortunate workaround for <rdar://problem/13361941>, which CFNetwork has said they're not going to fix.