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

[Auth] Move applicable non-public types away from subclassing NSObject #13676

Merged
merged 1 commit into from
Sep 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import FirebaseCoreExtension

/// Defines configurations to be added to a request to Firebase Auth's backend.
@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthRequestConfiguration: NSObject {
class AuthRequestConfiguration {
/// The Firebase Auth API key used in the request.
let apiKey: String

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ import Foundation
/// Represents the response from the deleteAccount endpoint.
///
/// See https://developers.google.com/identity/toolkit/web/reference/relyingparty/deleteAccount
class DeleteAccountResponse: NSObject, AuthRPCResponse {
override required init() {}
class DeleteAccountResponse: AuthRPCResponse {
required init() {}

func setFields(dictionary: [String: AnyHashable]) throws {}
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@
import Foundation

/// Represents the response from the emailLinkSignin endpoint.
class EmailLinkSignInResponse: NSObject, AuthRPCResponse, AuthMFAResponse {
override required init() {}
class EmailLinkSignInResponse: AuthRPCResponse, AuthMFAResponse {
required init() {}

/// The ID token in the email link sign-in response.
private(set) var idToken: String?
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ private let kErrorKey = "error"

/// Represents the provider user info part of the response from the getAccountInfo endpoint.
/// See https://developers.google.com/identity/toolkit/web/reference/relyingparty/getAccountInfo
class GetAccountInfoResponseProviderUserInfo: NSObject {
class GetAccountInfoResponseProviderUserInfo {
/// The ID of the identity provider.
let providerID: String?

Expand Down Expand Up @@ -57,7 +57,7 @@ class GetAccountInfoResponseProviderUserInfo: NSObject {

/// Represents the firebase user info part of the response from the getAccountInfo endpoint.
/// See https://developers.google.com/identity/toolkit/web/reference/relyingparty/getAccountInfo
class GetAccountInfoResponseUser: NSObject {
class GetAccountInfoResponseUser {
/// The ID of the user.
let localID: String?

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import Foundation

/// Represents the provider user info part of the response from the setAccountInfo endpoint.
/// See https: // developers.google.com/identity/toolkit/web/reference/relyingparty/setAccountInfo
class SetAccountInfoResponseProviderUserInfo: NSObject {
class SetAccountInfoResponseProviderUserInfo {
/// The ID of the identity provider.
var providerID: String?

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
import Foundation

/// A data structure for an APNs token.
class AuthAPNSToken: NSObject {
class AuthAPNSToken {
let data: Data
let type: AuthAPNSTokenType

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@

/// A class to manage APNs token in memory.
@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthAPNSTokenManager: NSObject {
class AuthAPNSTokenManager {
/// The timeout for registering for remote notification.
///
/// Only tests should access this property.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@
import Foundation

/// A class represents a credential that proves the identity of the app.
@objc(FIRAuthAppCredential) class AuthAppCredential: NSObject, NSSecureCoding {
@objc(FIRAuthAppCredential) // objc Needed for decoding old versions
paulb777 marked this conversation as resolved.
Show resolved Hide resolved
class AuthAppCredential: NSObject, NSSecureCoding {
/// The server acknowledgement of receiving client's claim of identity.
var receipt: String

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

/// A class to manage app credentials backed by iOS Keychain.
@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthAppCredentialManager: NSObject {
class AuthAppCredentialManager {
let kKeychainDataKey = "app_credentials"
let kFullCredentialKey = "full_credential"
let kPendingReceiptsKey = "pending_receipts"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@

/// A class represents a credential that proves the identity of the app.
@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthNotificationManager: NSObject {
class AuthNotificationManager {
/// The key to locate payload data in the remote notification.
private let kNotificationDataKey = "com.google.firebase.auth"

Expand Down
4 changes: 1 addition & 3 deletions FirebaseAuth/Sources/Swift/Utilities/AuthErrorUtils.swift
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ private let kFIRAuthErrorMessageMalformedJWT =
"Failed to parse JWT. Check the userInfo dictionary for the full token."

@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthErrorUtils: NSObject {
class AuthErrorUtils {
static let internalErrorDomain = "FIRAuthInternalErrorDomain"
static let userInfoDeserializedResponseKey = "FIRAuthErrorUserInfoDeserializedResponseKey"
static let userInfoDataKey = "FIRAuthErrorUserInfoDataKey"
Expand Down Expand Up @@ -563,5 +563,3 @@ class AuthErrorUtils: NSObject {
return error(code: .recaptchaActionCreationFailed, message: message)
}
}

protocol MultiFactorResolverWrapper: NSObjectProtocol {}
2 changes: 1 addition & 1 deletion FirebaseAuth/Sources/Swift/Utilities/AuthWebUtils.swift
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
import Foundation

@available(iOS 13, tvOS 13, macOS 10.15, macCatalyst 13, watchOS 7, *)
class AuthWebUtils: NSObject {
class AuthWebUtils {
static func randomString(withLength length: Int) -> String {
var randomString = ""
for _ in 0 ..< length {
Expand Down
Loading