-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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-15623] tiapp.xml secured and encrypted properties #4900
Conversation
Also added these properties to "listProperties" and "hasProperties". A debug log letting the developer know that these properties cannot be removed on "removeProperty" has also been added
IMPORTANT |
This probably needs to be merged with master to get the changes from #4898 |
jsonData = [NSData dataWithContentsOfFile:tiAppPropertiesPath]; | ||
} | ||
NSError *error = nil; | ||
props = [[NSJSONSerialization JSONObjectWithData:jsonData options:0 error:&error] 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.
Should check for error value in case anything wrong happens.
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.
Should also ensure that the return object type isKindOfClass NSDictionary
There couple of other issues. |
NSError *error = nil; | ||
props = [[NSJSONSerialization JSONObjectWithData:jsonData options:0 error:&error] retain]; | ||
if(error != nil) { | ||
DebugLog(@"[WARN] Could not load tiapp.xml properties, error was %@", [error localizedDescription]); |
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.
[ERROR]
Minor Comments. Starting FR |
FR Passed. Merging. |
[TIMOB-15623] tiapp.xml secured and encrypted properties
No description provided.