-
Notifications
You must be signed in to change notification settings - Fork 63
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
Remove appIDHasFeaturesEnabled #41
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -74,7 +74,6 @@ public struct PlanningOperationDidFinish: SessionMessage, Equatable { | |
/// The source data to send to the client for it to generate the provisioning task inputs for the service. | ||
public struct ProvisioningTaskInputsSourceData: Serializable, Equatable, Sendable { | ||
public let configurationName: String | ||
public let appIDHasFeaturesEnabled: Bool | ||
public let provisioningProfileSupport: ProvisioningProfileSupport | ||
public let provisioningProfileSpecifier: String | ||
public let provisioningProfileUUID: String | ||
|
@@ -95,9 +94,8 @@ public struct ProvisioningTaskInputsSourceData: Serializable, Equatable, Sendabl | |
public let localSigningStyle: String | ||
public let enableCloudSigning: Bool | ||
|
||
public init(configurationName: String, appIDHasFeaturesEnabled: Bool, provisioningProfileSupport: ProvisioningProfileSupport, provisioningProfileSpecifier: String, provisioningProfileUUID: String, provisioningStyle: ProvisioningStyle, teamID: String?, bundleIdentifier: String, productTypeEntitlements: PropertyListItem, productTypeIdentifier: String, projectEntitlementsFile: String?, projectEntitlements: PropertyListItem?, signingCertificateIdentifier: String, signingRequiresTeam: Bool, sdkRoot: String, sdkVariant: String?, supportsEntitlements: Bool, wantsBaseEntitlementInjection: Bool, entitlementsDestination: String, localSigningStyle: String, enableCloudSigning: Bool) { | ||
public init(configurationName: String, provisioningProfileSupport: ProvisioningProfileSupport, provisioningProfileSpecifier: String, provisioningProfileUUID: String, provisioningStyle: ProvisioningStyle, teamID: String?, bundleIdentifier: String, productTypeEntitlements: PropertyListItem, productTypeIdentifier: String, projectEntitlementsFile: String?, projectEntitlements: PropertyListItem?, signingCertificateIdentifier: String, signingRequiresTeam: Bool, sdkRoot: String, sdkVariant: String?, supportsEntitlements: Bool, wantsBaseEntitlementInjection: Bool, entitlementsDestination: String, localSigningStyle: String, enableCloudSigning: Bool) { | ||
self.configurationName = configurationName | ||
self.appIDHasFeaturesEnabled = appIDHasFeaturesEnabled | ||
self.provisioningProfileSupport = provisioningProfileSupport | ||
self.provisioningProfileSpecifier = provisioningProfileSpecifier | ||
self.provisioningProfileUUID = provisioningProfileUUID | ||
|
@@ -120,16 +118,15 @@ public struct ProvisioningTaskInputsSourceData: Serializable, Equatable, Sendabl | |
} | ||
|
||
/*! | ||
* @deprecated Use init(configurationName:appIDHasFeaturesEnabled:provisioningProfileSupport:provisioningProfileSpecifier:provisioningProfileUUID:provisioningStyle:teamID:bundleIdentifier:productTypeEntitlements:productTypeIdentifier:projectEntitlementsFile:projectEntitlements:signingCertificateIdentifier:signingRequiresTeam:sdkRoot:sdkVariant:supportsEntitlements:wantsBaseEntitlementInjection:entitlementsDestination:localSigningStyle:enableCloudSigning:) | ||
* @deprecated Use init(configurationName:provisioningProfileSupport:provisioningProfileSpecifier:provisioningProfileUUID:provisioningStyle:teamID:bundleIdentifier:productTypeEntitlements:productTypeIdentifier:projectEntitlementsFile:projectEntitlements:signingCertificateIdentifier:signingRequiresTeam:sdkRoot:sdkVariant:supportsEntitlements:wantsBaseEntitlementInjection:entitlementsDestination:localSigningStyle:enableCloudSigning:) | ||
*/ | ||
public init(configurationName: String, appIDHasFeaturesEnabled: Bool, provisioningProfileSupport: ProvisioningProfileSupport, provisioningProfileSpecifier: String, provisioningProfileUUID: String, provisioningStyle: ProvisioningStyle, teamID: String?, bundleIdentifier: String, productTypeEntitlements: PropertyListItem, productTypeIdentifier: String, projectEntitlementsFile: String?, projectEntitlements: PropertyListItem?, signingCertificateIdentifier: String, signingRequiresTeam: Bool, sdkRoot: String, sdkVariant: String?, supportsEntitlements: Bool, wantsBaseEntitlementInjection: Bool, entitlementsDestination: String, localSigningStyle: String) { | ||
self.init(configurationName: configurationName, appIDHasFeaturesEnabled: appIDHasFeaturesEnabled, provisioningProfileSupport: provisioningProfileSupport, provisioningProfileSpecifier: provisioningProfileSpecifier, provisioningProfileUUID: provisioningProfileUUID, provisioningStyle: provisioningStyle, teamID: teamID, bundleIdentifier: bundleIdentifier, productTypeEntitlements: productTypeEntitlements, productTypeIdentifier: productTypeIdentifier, projectEntitlementsFile: projectEntitlementsFile, projectEntitlements: projectEntitlements, signingCertificateIdentifier: signingCertificateIdentifier, signingRequiresTeam: signingRequiresTeam, sdkRoot: sdkRoot, sdkVariant: sdkVariant, supportsEntitlements: supportsEntitlements, wantsBaseEntitlementInjection: wantsBaseEntitlementInjection, entitlementsDestination: entitlementsDestination, localSigningStyle: localSigningStyle, enableCloudSigning: false) | ||
public init(configurationName: String, provisioningProfileSupport: ProvisioningProfileSupport, provisioningProfileSpecifier: String, provisioningProfileUUID: String, provisioningStyle: ProvisioningStyle, teamID: String?, bundleIdentifier: String, productTypeEntitlements: PropertyListItem, productTypeIdentifier: String, projectEntitlementsFile: String?, projectEntitlements: PropertyListItem?, signingCertificateIdentifier: String, signingRequiresTeam: Bool, sdkRoot: String, sdkVariant: String?, supportsEntitlements: Bool, wantsBaseEntitlementInjection: Bool, entitlementsDestination: String, localSigningStyle: String) { | ||
self.init(configurationName: configurationName, provisioningProfileSupport: provisioningProfileSupport, provisioningProfileSpecifier: provisioningProfileSpecifier, provisioningProfileUUID: provisioningProfileUUID, provisioningStyle: provisioningStyle, teamID: teamID, bundleIdentifier: bundleIdentifier, productTypeEntitlements: productTypeEntitlements, productTypeIdentifier: productTypeIdentifier, projectEntitlementsFile: projectEntitlementsFile, projectEntitlements: projectEntitlements, signingCertificateIdentifier: signingCertificateIdentifier, signingRequiresTeam: signingRequiresTeam, sdkRoot: sdkRoot, sdkVariant: sdkVariant, supportsEntitlements: supportsEntitlements, wantsBaseEntitlementInjection: wantsBaseEntitlementInjection, entitlementsDestination: entitlementsDestination, localSigningStyle: localSigningStyle, enableCloudSigning: false) | ||
} | ||
|
||
public init(from deserializer: any Deserializer) throws { | ||
let count = try deserializer.beginAggregate(20...22) | ||
self.configurationName = try deserializer.deserialize() | ||
self.appIDHasFeaturesEnabled = try deserializer.deserialize() | ||
self.provisioningProfileSupport = count >= 21 ? try deserializer.deserialize() : .unsupported | ||
self.provisioningProfileSpecifier = try deserializer.deserialize() | ||
self.provisioningProfileUUID = try deserializer.deserialize() | ||
|
@@ -157,7 +154,6 @@ public struct ProvisioningTaskInputsSourceData: Serializable, Equatable, Sendabl | |
public func serialize<T: Serializer>(to serializer: T) { | ||
serializer.serializeAggregate(21) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This count should be updated since one of the serialized fields was deleted |
||
serializer.serialize(configurationName) | ||
serializer.serialize(appIDHasFeaturesEnabled) | ||
serializer.serialize(provisioningProfileSupport) | ||
serializer.serialize(provisioningProfileSpecifier) | ||
serializer.serialize(provisioningProfileUUID) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,14 +14,11 @@ public import SWBUtil | |
|
||
public struct ProvisioningSourceData: Serializable, Sendable { | ||
public let configurationName: String | ||
/// Whether the App ID for this target+configuration has any features enabled. | ||
public let appIDHasFeaturesEnabled: Bool | ||
public let provisioningStyle: ProvisioningStyle | ||
public let bundleIdentifierFromInfoPlist: String | ||
|
||
public init(configurationName: String, appIDHasFeaturesEnabled: Bool, provisioningStyle: ProvisioningStyle, bundleIdentifierFromInfoPlist: String) { | ||
public init(configurationName: String, provisioningStyle: ProvisioningStyle, bundleIdentifierFromInfoPlist: String) { | ||
self.configurationName = configurationName | ||
self.appIDHasFeaturesEnabled = appIDHasFeaturesEnabled | ||
self.provisioningStyle = provisioningStyle | ||
self.bundleIdentifierFromInfoPlist = bundleIdentifierFromInfoPlist | ||
} | ||
|
@@ -30,33 +27,18 @@ public struct ProvisioningSourceData: Serializable, Sendable { | |
extension ProvisioningSourceData: Encodable, Decodable { | ||
enum CodingKeys: String, CodingKey { | ||
case configurationName | ||
case appIDHasFeaturesEnabled | ||
case provisioningStyle | ||
case bundleIdentifierFromInfoPlist | ||
} | ||
|
||
public init(from decoder: any Decoder) throws { | ||
let container = try decoder.container(keyedBy: CodingKeys.self) | ||
|
||
let appIDHasFeaturesEnabled: Bool | ||
if let appIDString = try container.decodeIfPresent(String.self, forKey: .appIDHasFeaturesEnabled) { | ||
switch appIDString { | ||
case "true": | ||
appIDHasFeaturesEnabled = true | ||
case "false": | ||
appIDHasFeaturesEnabled = false | ||
default: | ||
throw DecodingError.dataCorruptedError(forKey: .appIDHasFeaturesEnabled, in: container, debugDescription: "\(appIDString) is not a boolean value") | ||
} | ||
} else { | ||
appIDHasFeaturesEnabled = false | ||
} | ||
|
||
guard let provisioningStyle = try ProvisioningStyle(rawValue: container.decode(ProvisioningStyle.RawValue.self, forKey: .provisioningStyle)) else { | ||
throw DecodingError.dataCorruptedError(forKey: .provisioningStyle, in: container, debugDescription: "invalid provisioning style") | ||
} | ||
|
||
self.init(configurationName: try container.decode(String.self, forKey: .configurationName), appIDHasFeaturesEnabled: appIDHasFeaturesEnabled, provisioningStyle: provisioningStyle, bundleIdentifierFromInfoPlist: try container.decode(String.self, forKey: .bundleIdentifierFromInfoPlist)) | ||
self.init(configurationName: try container.decode(String.self, forKey: .configurationName), provisioningStyle: provisioningStyle, bundleIdentifierFromInfoPlist: try container.decode(String.self, forKey: .bundleIdentifierFromInfoPlist)) | ||
} | ||
} | ||
|
||
|
@@ -67,7 +49,6 @@ extension ProvisioningSourceData: PendingSerializableCodable { | |
public func legacySerialize<T: Serializer>(to serializer: T) { | ||
serializer.serializeAggregate(4) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similarly, the field count here should be updated |
||
serializer.serialize(configurationName) | ||
serializer.serialize(appIDHasFeaturesEnabled) | ||
serializer.serialize(provisioningStyle) | ||
serializer.serialize(bundleIdentifierFromInfoPlist) | ||
} | ||
|
@@ -76,7 +57,6 @@ extension ProvisioningSourceData: PendingSerializableCodable { | |
public init(fromLegacy deserializer: any Deserializer) throws { | ||
let count = try deserializer.beginAggregate(4...5) | ||
self.configurationName = try deserializer.deserialize() | ||
self.appIDHasFeaturesEnabled = try deserializer.deserialize() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar to the above, we should check the count here to determine if we need to deserialize and then discard the now-unused field |
||
self.provisioningStyle = try deserializer.deserialize() | ||
if count > 4 { | ||
_ = try deserializer.deserialize() as String // legacyTeamID | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, we should check the count of fields received to determine whether appIDHasFeaturesEnabled is still present - if it is, we can immediately discard it