e-mission / e-mission-data-collection

Repository for our own data collection
BSD 3-Clause "New" or "Revised" License
4 stars 19 forks source link

Consent was revoked when updating to a new version #141

Open shankari opened 8 years ago

shankari commented 8 years ago

While testing the upgrade from 1.2.0 to 1.3.0, the consent appeared to be revoked.

2016-11-02 23:08:23.990 emission[634:468206] reqConsent = 2016-07-14, currConfig.approval_date = (null)
2016-11-02 23:08:23.991 emission[634:468206] DEBUG: reqConsent = 2016-07-14, currConfig.approval_date = (null)

But while restarting with the same version, it worked.

2016-11-02 23:19:35.435 emission[638:470332] BEMDataCollection:pluginInitialize singleton -> initialize statemachine and delegate
2016-11-02 23:19:35.438 emission[638:470332] data has 114 bytes, str has size 114
2016-11-02 23:19:35.465 emission[638:470332] reqConsent = 2016-07-14, currConfig.approval_date = 2016-07-14
2016-11-02 23:19:35.465 emission[638:470332] DEBUG: reqConsent = 2016-07-14, currConfig.approval_date = 2016-07-14
2016-11-02 23:19:35.481 emission[638:470332] isConsented = YES
2016-11-02 23:19:35.481 emission[638:470332] DEBUG: isConsented = YES
shankari commented 8 years ago

missing_consent_start_state.gz

shankari commented 8 years ago

Trying to upgrade to 1.4.0 to see if this is reproducible.

shankari commented 8 years ago

Not reproducible after bumping to 1.4.0.

2016-11-02 23:26:06.972 emission[642:471685] reqConsent = 2016-07-14, currConfig.approval_date = 2016-07-14
2016-11-02 23:26:06.972 emission[642:471685] DEBUG: reqConsent = 2016-07-14, currConfig.approval_date = 2016-07-14

I am going to ensure that the other phone(s) have the release version and will see if the app store update reproduces this. if so, I have to make an emergency fix.

shankari commented 8 years ago

Happened on android as well, bumping up from 1.2.0 -> 1.4.0. Don't see anything in the logs, may need to add more logging.

no_consent_android_either.gz

shankari commented 8 years ago
shankari commented 8 years ago

Like an idiot, I reconsented instead of debugging further. We'll have to see if it happens again. Tests to try when that happens: