Skip to content
This repository was archived by the owner on Jan 21, 2022. It is now read-only.

US388853 : [iOS] MAS credentials should be stored in the keychain as device only#90

Merged
jhoongo merged 3 commits intodevelopfrom
US388853-CredentialsKeychainDeviceOnly
Oct 16, 2017
Merged

US388853 : [iOS] MAS credentials should be stored in the keychain as device only#90
jhoongo merged 3 commits intodevelopfrom
US388853-CredentialsKeychainDeviceOnly

Conversation

@rodrigoareis
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@jhoongo jhoongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please review my comment, and return the ticket back to me.

// Local storage
//
MASIKeyChainStore *localStorage = [MASIKeyChainStore keyChainStoreWithService:_localStorageServiceName];
localStorage.synchronizable = FALSE;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please avoid using TRUE/FALSE, and use YES/NO.

//
MASIKeyChainStore *sharedStorage = [MASIKeyChainStore keyChainStoreWithService:_sharedStorageServiceName accessGroup:self.accessGroup];

sharedStorage.synchronizable = FALSE;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please avoid using TRUE/FALSE, and use YES/NO.

jhoongo
jhoongo previously approved these changes Oct 5, 2017
Copy link
Copy Markdown
Contributor

@jhoongo jhoongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

…ty, by default, the Keychain will not be synchronized through iCloud.
@rodrigoareis rodrigoareis changed the title [iOS] MAS credentials should be stored in the keychain as device only US388853 : [iOS] MAS credentials should be stored in the keychain as device only Oct 11, 2017
Copy link
Copy Markdown
Contributor

@lsanches lsanches left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved!

@jhoongo jhoongo merged commit adc83c6 into develop Oct 16, 2017
@jhoongo jhoongo deleted the US388853-CredentialsKeychainDeviceOnly branch October 16, 2017 23:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants