Skip to content

Conversation

andrewheard
Copy link
Contributor

Publicly exposed the initWithApp:appCheckProvider: constructor in GACAppCheck for the non-Firebase-specific App Check SDK. Since an AppCheckProvider is explicitly specified when constructing this generic App Check SDK, the AppCheckProviderFactory protocol / pattern / functionality is no longer used. Remaining references to factories (and FIRApp) will be pruned in follow-up PRs.

Added a podspec for AppCheckInterop and added it as a dependency in the existing AppCheck podspec. This is a lightweight, protocol-only, dependency that non-Firebase SDKs can make use of.

#no-changelog

Publicly exposed the `initWithApp:appCheckProvider:` constructor in
GACAppCheck for the non-Firebase-specific App Check SDK.

Added a podspec for AppCheckInterop and added it as a dependency in the
existing AppCheck podspec.
#import "AppCheck/Sources/Public/AppCheck/GACDeviceCheckProvider.h"

@implementation GACDeviceCheckProviderFactory

+ (void)load {
Copy link
Member

Choose a reason for hiding this comment

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

Great to see load going away.

Changed the 'AppCheckInterop' dependency from '~> 10.0' to '~> 10.10' in
the AppCheck podspec.
@andrewheard andrewheard requested a review from ncooke3 May 15, 2023 20:58
@andrewheard andrewheard merged commit 1e1f37b into appcheck-lite May 15, 2023
@andrewheard andrewheard deleted the ah/appcheck-lite branch May 15, 2023 21:04
andrewheard added a commit that referenced this pull request May 19, 2023
Publicly exposed the initWithApp:appCheckProvider: constructor in GACAppCheck for the non-Firebase-specific App Check SDK. Since an AppCheckProvider is explicitly specified when constructing this generic App Check SDK, the AppCheckProviderFactory protocol / pattern / functionality is no longer used.

Added a podspec for AppCheckInterop and added it as a dependency in the existing AppCheck podspec. This is a lightweight, protocol-only, dependency that non-Firebase SDKs can make use of.
andrewheard added a commit that referenced this pull request May 26, 2023
Publicly exposed the initWithApp:appCheckProvider: constructor in GACAppCheck for the non-Firebase-specific App Check SDK. Since an AppCheckProvider is explicitly specified when constructing this generic App Check SDK, the AppCheckProviderFactory protocol / pattern / functionality is no longer used.

Added a podspec for AppCheckInterop and added it as a dependency in the existing AppCheck podspec. This is a lightweight, protocol-only, dependency that non-Firebase SDKs can make use of.
andrewheard added a commit that referenced this pull request May 31, 2023
Publicly exposed the initWithApp:appCheckProvider: constructor in GACAppCheck for the non-Firebase-specific App Check SDK. Since an AppCheckProvider is explicitly specified when constructing this generic App Check SDK, the AppCheckProviderFactory protocol / pattern / functionality is no longer used.

Added a podspec for AppCheckInterop and added it as a dependency in the existing AppCheck podspec. This is a lightweight, protocol-only, dependency that non-Firebase SDKs can make use of.
andrewheard added a commit that referenced this pull request Jun 13, 2023
Publicly exposed the initWithApp:appCheckProvider: constructor in GACAppCheck for the non-Firebase-specific App Check SDK. Since an AppCheckProvider is explicitly specified when constructing this generic App Check SDK, the AppCheckProviderFactory protocol / pattern / functionality is no longer used.

Added a podspec for AppCheckInterop and added it as a dependency in the existing AppCheck podspec. This is a lightweight, protocol-only, dependency that non-Firebase SDKs can make use of.
@firebase firebase locked and limited conversation to collaborators Jun 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants