Merge pull request ResearchKit#128 from EricSiegNW/bug/FPHS-625

FPHS 625
Erin-Mounts · Jun 3, 2016 · 4905faa · 4905faa
2 parents cdca9d2 + 9938585
commit 4905faa
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 5 deletions.
diff --git a/APCAppCore/APCAppCore/DataSubstrate/Model/APCUser.h b/APCAppCore/APCAppCore/DataSubstrate/Model/APCUser.h
@@ -58,7 +58,6 @@ typedef NS_ENUM(NSInteger, APCUserConsentSharingScope) {
 
 @property (nonatomic, strong, nullable) NSString * firstName DEPRECATED_ATTRIBUTE;
 @property (nonatomic, strong, nullable) NSString * lastName DEPRECATED_ATTRIBUTE;
-
 @property (nonatomic, strong, nullable) NSString * email;
 @property (nonatomic, strong, nullable) NSString * password;
 @property (nonatomic, strong, nullable) NSString * sessionToken;
@@ -98,9 +97,6 @@ typedef NS_ENUM(NSInteger, APCUserConsentSharingScope) {
 @property (nonatomic, strong, nullable) NSDate *consentSignatureDate;
 @property (nonatomic, strong, nullable) NSData *consentSignatureImage;
 
-@property (nonatomic, strong, nullable) NSDate *downloadDataStartDate; // NOT stored in CoreData
-@property (nonatomic, strong, nullable) NSDate *downloadDataEndDate; // NOT stored in CoreData
-
 @property (nonatomic, getter=isSecondaryInfoSaved) BOOL secondaryInfoSaved;
 
 /*********************************************************************************/
@@ -126,6 +122,14 @@ typedef NS_ENUM(NSInteger, APCUserConsentSharingScope) {
 @property (nonatomic, nullable) NSNumber * savedSharingScope;
 @property (nonatomic, nullable) NSArray <NSString *> * dataGroups;
 
+/*********************************************************************************/
+#pragma mark - Stored In Memory Only
+/*********************************************************************************/
+@property (nonatomic, strong, nullable) NSString * cachedSessionToken; // Memory Only, can nil this out and sessionToken will remain safely in keychain
+@property (nonatomic, strong, nullable) NSDate *downloadDataStartDate; // NOT stored in CoreData
+@property (nonatomic, strong, nullable) NSDate *downloadDataEndDate; // NOT stored in CoreData
+
+
 - (BOOL) isLoggedOut;
 
 /**

diff --git a/APCAppCore/APCAppCore/DataSubstrate/Model/APCUser.m b/APCAppCore/APCAppCore/DataSubstrate/Model/APCUser.m
@@ -275,6 +275,20 @@ -(void)setPassword:(NSString *)password
     [APCKeychainStore setString:[self hashIfNeeded:password] forKey:kPasswordPropertyName];
 }
 
+- (NSString *)sessionToken
+{
+    if (self.cachedSessionToken == nil) {
+        self.cachedSessionToken = [APCKeychainStore stringForKey:kSessionTokenPropertyName];
+    }
+    return self.cachedSessionToken;
+}
+
+-(void)setSessionToken:(NSString *)sessionToken
+{
+    self.cachedSessionToken = sessionToken;
+    [APCKeychainStore setString:[self hashIfNeeded:sessionToken] forKey:kSessionTokenPropertyName];
+}
+
 - (NSString *)externalId
 {
     return [APCKeychainStore stringForKey:kExternalIdPropertyName];

diff --git a/APCAppCore/APCAppCore/Startup/APCAppDelegate.m b/APCAppCore/APCAppCore/Startup/APCAppDelegate.m
@@ -273,7 +273,8 @@ - (void)applicationDidEnterBackground:(UIApplication *) __unused application
         [[NSUserDefaults standardUserDefaults] setObject: [NSNumber numberWithLong:uptime()] forKey:kLastUsedTimeKey];
         [[NSUserDefaults standardUserDefaults] synchronize];
     }
-    self.dataSubstrate.currentUser.sessionToken = nil;
+    // Clear the in memory token while preserving the keychain version
+    self.dataSubstrate.currentUser.cachedSessionToken = nil;
 
     [self showSecureView];
 }