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
[TIMOB-16275] iOS Cookie management #5466
Conversation
…props and methods
platforms: [iphone, ipad] | ||
since: "3.3.0" | ||
returns: | ||
type: String |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
String??Also what happens if I call this method on a cookieProxy created from a cookie that already exists in the system? When creating a cookie what values are required for a cookie to be valid
Code Reviewed. REJECTED |
{ | ||
if(self = [super _initWithPageContext:context]) | ||
{ | ||
for(NSString *key in [cookie properties]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
_cookieDict = [[NSMutableDictionary dictionaryWithDictionary:[cookie properties]] retain]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
httpOnly = [cookie isHttpOnly]
and modify the getter for iSHttpOnly to use this boolean
summary: Returns true if the cookie is valid. | ||
description: | | ||
This method checks wheather the cookie is valid. For a cookie to be valid the | ||
minimum properties requiered are `domain`, `name`, `value`, and `path` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually on iOS you need path, name,value and either domain or original Url
Code Reviewed. ACCEPTED |
FR ok. APPROVED |
[TIMOB-16275] iOS Cookie management
[TIMOB-16275] iOS Cookie management
Details in Jira ticket
Test case here