Skip to content
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

Feature flag values are inaccessible from extensions that don't have network access #300

Open
jayhickey opened this issue Aug 4, 2023 · 2 comments

Comments

@jayhickey
Copy link

jayhickey commented Aug 4, 2023

Describe the bug

I have an app with a Safari extension and it can't access any LaunchDarkly feature flag values. Safari extensions don't have network access and the SDK is caching flags in a custom user default suite that is inaccessible from app extensions. This is non-configurable from what I can tell (this protocol and implementation is internal):

self.keyedValueCache = serviceFactory.makeKeyedValueCache(cacheKey: "com.launchdarkly.client.\(cacheKey)")

func makeKeyedValueCache(cacheKey: String?) -> KeyedValueCaching {
UserDefaults(suiteName: cacheKey)!
}

There should be some way to configure this UserDefaults suite to point to the app group ID so extensions can also read from the defaults.

Am I overlooking something? How can I access cached LD flags from extensions that don't have network access?

To reproduce

  1. Create an iOS app that has a main app target and a Safari extension.
  2. Set up the LDClient in both the main app and the extension.
  3. Launch the main app and log the allFlags value from the client.
  4. Launch the Safari extension target and log the allFlags value from the client.
  5. Observe that the Safari extension doesn't have any of the fetched or cached flag values from the main app target—it only has the default fallback values.

Expected behavior
Flag values fetched from the main app should be cached in such a way that extension targets can also access the cached values.

SDK version
9.0.0

Language version, developer tools
Xcode 14.3.1

OS/platform
iOS 16.6

@louis-launchdarkly
Copy link
Contributor

Hello @jayhickey, thank you for reaching out. Currently, we do not have testing with the extension targets with the SDK, so we are not sure if will this work. It likely will take the engineers some time, but we will look into this and give you an update once we have more information.

Filed Internally as 212134.

@jayhickey
Copy link
Author

jayhickey commented Aug 10, 2023

Hello @jayhickey, thank you for reaching out. Currently, we do not have testing with the extension targets with the SDK, so we are not sure if will this work. It likely will take the engineers some time, but we will look into this and give you an update once we have more information.

Filed Internally as 212134.

Thanks for the reply and for filing an internal ticket. I can tell you confidently that this does not work because you are storing the cached flag data in UserDefaults in a way that makes them inaccessible from extensions (see my code pointers above.) You will need to make the UserDefaults suiteName configurable, but that's a bit tricky since it looks like you are creating multiple UserDefaults databases, one for each mobileKey.

As a new LaunchDarkly customer I'm pretty shocked to see this isn't supported and hasn't come up in the past, pretty much every app has extensions these days 😃 This makes it impossible to use LaunchDarkly out of the box in Widgets, Content Blockers, Safari extensions, iMessage extensions, Keyboards extensions, Spotlight, and literally everything else listed here: https://developer.apple.com/app-extensions/

If you have an ability to do so, please prioritize this! Thank you 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants