Revert changes in Crashlytics version - as it's causing an issue with the version associated with the crash upload

Crashlytics/Crashlytics/Components/FIRCLSContext.m

@@ -17,8 +17,6 @@
 #include <stdlib.h>
 #include <string.h>
 
-#import "Crashlytics/Shared/FIRCLSConstants.h"
-
 #import "Crashlytics/Crashlytics/Models/FIRCLSFileManager.h"
 #import "Crashlytics/Crashlytics/Models/FIRCLSInstallIdentifierModel.h"
 #import "Crashlytics/Crashlytics/Models/FIRCLSInternalReport.h"
@@ -402,9 +400,9 @@ static bool FIRCLSContextRecordIdentity(FIRCLSFile* file, const FIRCLSContextIni
 
   FIRCLSFileWriteHashStart(file);
 
-  FIRCLSFileWriteHashEntryString(file, "generator", FIRCLSSDKGeneratorName().UTF8String);
-  FIRCLSFileWriteHashEntryString(file, "display_version", FIRCLSSDKVersion().UTF8String);
-  FIRCLSFileWriteHashEntryString(file, "build_version", FIRCLSSDKVersion().UTF8String);
+  FIRCLSFileWriteHashEntryString(file, "generator", CLS_SDK_GENERATOR_NAME);
+  FIRCLSFileWriteHashEntryString(file, "display_version", CLS_SDK_DISPLAY_VERSION);
+  FIRCLSFileWriteHashEntryString(file, "build_version", CLS_SDK_DISPLAY_VERSION);
   FIRCLSFileWriteHashEntryUint64(file, "started_at", time(NULL));
 
   FIRCLSFileWriteHashEntryString(file, "session_id", initData->sessionId);

Crashlytics/Crashlytics/Controllers/FIRCLSReportUploader.m

@@ -318,15 +318,15 @@ - (NSMutableURLRequest *)mutableRequestWithURL:(NSURL *)url timeout:(NSTimeInter
 
   NSString *localeId = [self localeIdentifier];
 
-  [request setValue:FIRCLSSDKGeneratorName() forHTTPHeaderField:FIRCLSNetworkUserAgent];
+  [request setValue:@CLS_SDK_GENERATOR_NAME forHTTPHeaderField:FIRCLSNetworkUserAgent];
   [request setValue:FIRCLSNetworkApplicationJson forHTTPHeaderField:FIRCLSNetworkAccept];
   [request setValue:FIRCLSNetworkUTF8 forHTTPHeaderField:FIRCLSNetworkAcceptCharset];
   [request setValue:localeId forHTTPHeaderField:FIRCLSNetworkAcceptLanguage];
   [request setValue:localeId forHTTPHeaderField:FIRCLSNetworkContentLanguage];
   [request setValue:FIRCLSDeveloperToken forHTTPHeaderField:FIRCLSNetworkCrashlyticsDeveloperToken];
   [request setValue:FIRCLSApplicationGetSDKBundleID()
       forHTTPHeaderField:FIRCLSNetworkCrashlyticsAPIClientId];
-  [request setValue:FIRCLSSDKVersion()
+  [request setValue:@CLS_SDK_DISPLAY_VERSION
       forHTTPHeaderField:FIRCLSNetworkCrashlyticsAPIClientDisplayVersion];
   [request setValue:[[self dataSource] googleAppID]
       forHTTPHeaderField:FIRCLSNetworkCrashlyticsGoogleAppId];

Crashlytics/Crashlytics/DataCollection/FIRCLSDataCollectionArbiter.m

@@ -101,7 +101,7 @@ + (BOOL)isCrashlyticsCollectionEnabledWithApp:(FIRApp *)app withAppInfo:(NSDicti
       [firebaseCrashlyticsCollectionEnabled isKindOfClass:[NSNumber class]]) {
     return [firebaseCrashlyticsCollectionEnabled boolValue];
   }
-#ifdef CRASHLYTICS_1P
+#ifdef CRASHLYTICS_INTERNAL
   return NO;
 #else
   return [app isDataCollectionDefaultEnabled];

Crashlytics/Crashlytics/FIRCrashlytics.m

@@ -37,7 +37,6 @@
 
 #import "Crashlytics/Crashlytics/Helpers/FIRCLSLogger.h"
 #import "Crashlytics/Shared/FIRCLSByteUtility.h"
-#import "Crashlytics/Shared/FIRCLSConstants.h"
 #import "Crashlytics/Shared/FIRCLSFABHost.h"
 
 #import "Crashlytics/Crashlytics/Controllers/FIRCLSReportManager.h"
@@ -95,7 +94,7 @@ - (instancetype)initWithApp:(FIRApp *)app
 
     FIRCLSProfileMark mark = FIRCLSProfilingStart();
 
-    NSLog(@"[Firebase/Crashlytics] Version %@", FIRCLSSDKVersion());
+    NSLog(@"[Firebase/Crashlytics] Version %@", @CLS_SDK_DISPLAY_VERSION);
 
     FIRCLSDeveloperLog("Crashlytics", @"Running on %@, %@ (%@)", FIRCLSHostModelInfo(),
                        FIRCLSHostOSDisplayVersion(), FIRCLSHostOSBuildVersion());

Crashlytics/Crashlytics/Helpers/FIRCLSDefines.h

@@ -17,6 +17,8 @@
 #include <TargetConditionals.h>
 
 // macro trickiness
+#define STR_HELPER(x) #x
+#define STR(x) STR_HELPER(x)
 #define CONCAT_EXPANDED(a, b) a##b
 #define CONCAT(a, b) CONCAT_EXPANDED(a, b)
 
@@ -45,6 +47,10 @@
 #define CLS_TARGET_OS_OSX (TARGET_OS_MAC && !TARGET_OS_IPHONE)
 #define CLS_TARGET_OS_HAS_UIKIT (TARGET_OS_IOS || TARGET_OS_TV)
 
+#define CLS_SDK_DISPLAY_VERSION STR(DISPLAY_VERSION)
+
+#define CLS_SDK_GENERATOR_NAME (STR(CLS_SDK_NAME) "/" CLS_SDK_DISPLAY_VERSION)
+
 // arch definitions
 #if defined(__arm__) || defined(__arm64__) || defined(__arm64e__)
 #include <arm/arch.h>

Crashlytics/Crashlytics/Models/FIRCLSSettings.m

@@ -304,7 +304,7 @@ - (BOOL)collectReportsEnabled {
 }
 
 - (BOOL)shouldUseNewReportEndpoint {
-#ifdef CRASHLYTICS_1P
+#ifdef CRASHLYTICS_INTERNAL
   return YES;
 #else
   NSNumber *value = [self appSettings][@"report_upload_variant"];

Crashlytics/Crashlytics/Settings/FIRCLSSettingsOnboardingManager.m

@@ -84,7 +84,7 @@ - (void)beginSettingsAndOnboardingWithGoogleAppId:(NSString *)googleAppID
   // backwards compatibility
   // TODO(b/141747635)
   self.kitVersionsByKitBundleIdentifier = @{
-    FIRCLSApplicationGetSDKBundleID() : FIRCLSSDKVersion(),
+    FIRCLSApplicationGetSDKBundleID() : @CLS_SDK_DISPLAY_VERSION,
   };
 
   [self beginSettingsDownload:token waitForCompletion:waitForCompletion];

Crashlytics/Crashlytics/Settings/Operations/FIRCLSDownloadAndSaveSettingsOperation.m

@@ -88,7 +88,7 @@ - (NSMutableURLRequest *)mutableRequestWithDefaultHTTPHeaderFieldsAndTimeoutForU
       forHTTPHeaderField:@"X-Crashlytics-OS-Build-Version"];
   [request setValue:FIRCLSHostOSDisplayVersion()
       forHTTPHeaderField:@"X-Crashlytics-OS-Display-Version"];
-  [request setValue:FIRCLSSDKVersion() forHTTPHeaderField:@"X-Crashlytics-API-Client-Version"];
+  [request setValue:FIRCLSVersion forHTTPHeaderField:@"X-Crashlytics-API-Client-Version"];
 
   return request;
 }

Crashlytics/Crashlytics/Settings/Operations/FIRCLSNetworkOperation.m

@@ -74,16 +74,15 @@ - (NSMutableURLRequest *)mutableRequestWithDefaultHTTPHeadersForURL:(NSURL *)url
   [request setValue:FIRCLSDeveloperToken forHTTPHeaderField:FIRCLSNetworkCrashlyticsDeveloperToken];
   [request setValue:FIRCLSApplicationGetSDKBundleID()
       forHTTPHeaderField:FIRCLSNetworkCrashlyticsAPIClientId];
-  [request setValue:FIRCLSSDKVersion()
+  [request setValue:FIRCLSVersion
       forHTTPHeaderField:FIRCLSNetworkCrashlyticsAPIClientDisplayVersion];
   [request setValue:self.googleAppID forHTTPHeaderField:FIRCLSNetworkCrashlyticsGoogleAppId];
 
   return request;
 }
 
 - (NSString *)userAgentString {
-  return
-      [NSString stringWithFormat:@"%@/%@", FIRCLSApplicationGetSDKBundleID(), FIRCLSSDKVersion()];
+  return [NSString stringWithFormat:@"%@/%@", FIRCLSApplicationGetSDKBundleID(), FIRCLSVersion];
 }
 
 - (NSString *)localeIdentifier {

Crashlytics/Shared/FIRCLSConstants.h

@@ -14,10 +14,10 @@
 
 #import <Foundation/Foundation.h>
 
-NS_ASSUME_NONNULL_BEGIN
-
 FOUNDATION_EXPORT NSString *const FIRCLSDeveloperToken;
 
+FOUNDATION_EXPORT NSString *const FIRCLSVersion;
+
 // User Messages
 FOUNDATION_EXPORT NSString *const FIRCLSMissingConsumerKeyMsg;
 FOUNDATION_EXPORT NSString *const FIRCLSMissingConsumerSecretMsg;
@@ -43,9 +43,3 @@ FOUNDATION_EXPORT NSString *const FIRCLSNetworkCrashlyticsGoogleAppId;
 FOUNDATION_EXPORT NSString *const FIRCLSNetworkCrashlyticsOrgId;
 FOUNDATION_EXPORT NSString *const FIRCLSNetworkUserAgent;
 FOUNDATION_EXPORT NSString *const FIRCLSNetworkUTF8;
-
-NSString *FIRCLSSDKGeneratorName(void);
-
-NSString *FIRCLSSDKVersion(void);
-
-NS_ASSUME_NONNULL_END

Crashlytics/Shared/FIRCLSConstants.m

@@ -13,13 +13,14 @@
 // limitations under the License.
 
 #import "Crashlytics/Shared/FIRCLSConstants.h"
-#import "FirebaseCore/Sources/Private/FirebaseCoreInternal.h"
 
 #define STR_HELPER(x) #x
 #define STR(x) STR_HELPER(x)
 
 NSString* const FIRCLSDeveloperToken = @"77f0789d8e230eccdb4b99b82dccd78d47f9b604";
 
+NSString* const FIRCLSVersion = @STR(DISPLAY_VERSION);
+
 // User Messages
 NSString* const FIRCLSMissingConsumerKeyMsg = @"consumer key is nil or zero length";
 NSString* const FIRCLSMissingConsumerSecretMsg = @"consumer secret is nil or zero length";
@@ -46,15 +47,3 @@
 NSString* const FIRCLSNetworkCrashlyticsOrgId = @"X-Crashlytics-Org-Id";
 NSString* const FIRCLSNetworkUserAgent = @"User-Agent";
 NSString* const FIRCLSNetworkUTF8 = @"utf-8";
-
-NSString* FIRCLSSDKGeneratorName(void) {
-  return [NSString stringWithFormat:@"%s / %s", STR(CLS_SDK_NAME), FIRFirebaseVersion().UTF8String];
-}
-
-NSString* FIRCLSSDKVersion(void) {
-#ifdef CRASHLYTICS_1P
-  return [FIRFirebaseVersion() stringByAppendingString:@"_1P"];
-#else
-  return FIRFirebaseVersion();
-#endif
-}

Crashlytics/UnitTests/FIRCLSDataCollectionArbiterTest.m

@@ -52,7 +52,7 @@ - (void)tearDown {
 - (void)testNothingSet {
   self.fakeApp.isDefaultCollectionEnabled = YES;
   FIRCLSDataCollectionArbiter *arbiter = [self arbiterWithDictionary:@{}];
-#ifdef CRASHLYTICS_1P
+#ifdef CRASHLYTICS_INTERNAL
   XCTAssertFalse([arbiter isCrashlyticsCollectionEnabled]);
 #else
   // It should be YES by default for 3P users.

Crashlytics/UnitTests/FIRCLSSettingsTests.m

@@ -489,7 +489,7 @@ - (void)testLegacyReportEndpointSettings {
   [self.settings cacheSettingsWithGoogleAppID:TestGoogleAppID currentTimestamp:currentTimestamp];
 
   XCTAssertNil(error, "%@", error);
-#ifdef CRASHLYTICS_1P
+#ifdef CRASHLYTICS_INTERNAL
   XCTAssertTrue(self.settings.shouldUseNewReportEndpoint);
 #else
   XCTAssertFalse(self.settings.shouldUseNewReportEndpoint);

FirebaseCrashlytics.podspec

@@ -62,6 +62,7 @@ Pod::Spec.new do |s|
   s.ios.pod_target_xcconfig = {
     'GCC_C_LANGUAGE_STANDARD' => 'c99',
     'GCC_PREPROCESSOR_DEFINITIONS' =>
+      'DISPLAY_VERSION=' + s.version.to_s + ' ' +
       'CLS_SDK_NAME="Crashlytics iOS SDK" ' +
       # For nanopb:
       'PB_FIELD_32BIT=1 PB_NO_PACKED_STRUCTS=1 PB_ENABLE_MALLOC=1',
@@ -71,6 +72,7 @@ Pod::Spec.new do |s|
   s.osx.pod_target_xcconfig = {
     'GCC_C_LANGUAGE_STANDARD' => 'c99',
     'GCC_PREPROCESSOR_DEFINITIONS' =>
+      'DISPLAY_VERSION=' + s.version.to_s + ' ' +
       'CLS_SDK_NAME="Crashlytics Mac SDK" ' +
       # For nanopb:
       'PB_FIELD_32BIT=1 PB_NO_PACKED_STRUCTS=1 PB_ENABLE_MALLOC=1',
@@ -80,6 +82,7 @@ Pod::Spec.new do |s|
   s.tvos.pod_target_xcconfig = {
     'GCC_C_LANGUAGE_STANDARD' => 'c99',
     'GCC_PREPROCESSOR_DEFINITIONS' =>
+      'DISPLAY_VERSION=' + s.version.to_s + ' ' +
       'CLS_SDK_NAME="Crashlytics tvOS SDK" ' +
       # For nanopb:
       'PB_FIELD_32BIT=1 PB_NO_PACKED_STRUCTS=1 PB_ENABLE_MALLOC=1',
@@ -89,6 +92,7 @@ Pod::Spec.new do |s|
   s.watchos.pod_target_xcconfig = {
     'GCC_C_LANGUAGE_STANDARD' => 'c99',
     'GCC_PREPROCESSOR_DEFINITIONS' =>
+      'DISPLAY_VERSION=' + s.version.to_s + ' ' +
       'CLS_SDK_NAME="Crashlytics watchOS SDK" ' +
       # For nanopb:
       'PB_FIELD_32BIT=1 PB_NO_PACKED_STRUCTS=1 PB_ENABLE_MALLOC=1',