Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add HTTPCookieStorageMonitor #49

Closed
eBardX opened this issue May 6, 2018 · 3 comments
Closed

Add HTTPCookieStorageMonitor #49

eBardX opened this issue May 6, 2018 · 3 comments
Assignees

Comments

@eBardX
Copy link
Owner

eBardX commented May 6, 2018

Feature Name

  • HTTPCookieStorageMonitor -- should live in Foundation folder

Events

  • acceptPolicyChanged(HTTPCookieStorage) -- wraps NSHTTPCookieManagerAcceptPolicyChanged notification
  • cookiesChanged(HTTPCookieStorage) -- wraps NSHTTPCookieManagerCookiesChanged notification

Note: Since there can be multiple instances of HTTPCookieStorage; HTTPCookieStorageMonitor should provide both a cookieStorage property and a cookieStorage initializer parameter.

Platform

iOS/macOS /tvOS/watchOS

@angiemugo
Copy link
Collaborator

I will take this

@eBardX
Copy link
Owner Author

eBardX commented May 14, 2018

@angiemugo Are you still planning to work on #54?

@angiemugo
Copy link
Collaborator

Yeah. I had raised a PR when I assigned this one then I had to revoke it when I noticed a huge mistake I had made. I intend to fix #54 and raise PR by end of the day today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants