Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
From: clelland <git@git.apache.org>
To: dev@cordova.apache.org
Reply-To: dev@cordova.apache.org
References: <git-pr-34-cordova-plugin-inappbrowser@git.apache.org>
In-Reply-To: <git-pr-34-cordova-plugin-inappbrowser@git.apache.org>
Subject: [GitHub] cordova-plugin-inappbrowser pull request: [CB-6389] add
 clearsessi...
Content-Type: text/plain
Message-Id: <20140404024242.DC2CD93DA97@tyr.zones.apache.org>
Date: Fri,  4 Apr 2014 02:42:42 +0000 (UTC)

Github user clelland commented on the pull request:

    https://github.com/apache/cordova-plugin-inappbrowser/pull/34#issuecomment-39527128
  
    Umm... you're absolutely right there. And I'm not sure quite how I missed that.
    
    I'll reopen the original issue (CB-3617) since that's clearly not done. The option name should be changed here, and we should start thinking about parity on other platforms.
    
    Two issues to resolve:
    
    1. Do session *cookies* get cleared when the InAppBrowser is closed? If so, there's never going to be a need to explicitly clear those ones on a `window.open` call.
    2. Does `sharedURLCache` in this case contain cached responses for just the IAB, or for the entire application? (XHRs, internal navigation, etc.) If so, we probably don't want to clear it this way, and should probably just start the InAppBrowser with its own isolated URL cache instead, and clear *that* cache when requested.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---