feat(W-23159744): per-user persistent feature flags (Android)#2937
Open
wmathurin wants to merge 2 commits into
Open
feat(W-23159744): per-user persistent feature flags (Android)#2937wmathurin wants to merge 2 commits into
wmathurin wants to merge 2 commits into
Conversation
Implements per-user feature flag storage and hydration so each user account's active features are accurately reflected in the User-Agent string across app restarts. - UserAccount: featureFlags field (Set<String>), serialized to JSON, persisted via AccountManager (KEY_FEATURE_FLAGS, encrypted comma-separated) - SalesforceSDKManager: perUserFeatures map (ConcurrentHashMap<String,ConcurrentSkipListSet>), getUserAgent(qualifier, user) unions global + per-user flags, registerUsedAppFeature/unregisterUsedAppFeature per-user overloads, hydratePerUserFeatures() called from init block, isGlobalFeatureRegistered() helper - LoginActivity: completedViaBrowserTab flag captures Custom Tab completion (covers both regular and Login for Admin); onAuthFlowSuccess promotes BW/WD/QR transient global flags to per-user - AuthenticationUtilities: BA/SL flags registered per-user - SmartStoreSDKManager: SU flag registered per-user - SyncManager/LayoutSyncManager/MetadataSyncManager: MS/Layout/Metadata flags per-user - SalesforceHybridSDKManager: getUserAgent(qualifier, user) override - UserAccountTest, SalesforceSDKManagerTests: new unit tests
Generated by 🚫 Danger |
FEATURE_BROWSER_LOGIN must appear in the UA for requests made during the login session. Register it globally in loadLoginPageInCustomTab() (alongside completedViaBrowserTab = true), then clear global and promote to per-user at onAuthFlowSuccess — matching the WD/QR pattern.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
featureFlagsfield toUserAccount(Set), serialized to JSON and persisted via AccountManager as an encrypted comma-separated string underKEY_FEATURE_FLAGSSalesforceSDKManagergains per-user overloads forregisterUsedAppFeature/unregisterUsedAppFeature,getUserAgent(qualifier, user)(returns global ∪ per-user union),isGlobalFeatureRegistered(), andhydratePerUserFeatures()called frominitLoginActivity:completedViaBrowserTabflag captures Custom Tab completion for both regular browser login and Login for Admin;onAuthFlowSuccesspromotes BW/WD/QR flags to per-userAuthenticationUtilitiesregistered per-user; SU inSmartStoreSDKManager; MS/Layout/Metadata in their respective sync managersSalesforceHybridSDKManageraddsgetUserAgent(qualifier, user)overrideUserAccountTestandSalesforceSDKManagerTestsGUS Stories
Test plan
./gradlew :libs:SalesforceSDK:assembleAndroidTest— BUILD SUCCESSFUL./gradlew :libs:SalesforceSDK:testDebugUnitTest— BUILD SUCCESSFUL./gradlew :libs:SalesforceSDK:lint— BUILD SUCCESSFUL (no new warnings)connectedAndroidTest) — require connected device/emulator, run in CI