Add platform cehck for Firebase since it is only iOS for now

Resolves: OKTA-350984 OKTA-350984 <<<Jenkins Check-In of Tested SHA: 18f1e16 for [email protected]>>> Artifact: okta-ios-logger
okta · Dec 7, 2020 · d8add28 · d8add28
1 parent 4e6d7d2
commit d8add28
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/OktaLogger.podspec b/OktaLogger.podspec
@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
  s.name = "OktaLogger"
- s.version = "1.0.7"
+ s.version = "1.0.8"
  s.summary = "Logging proxy for standardized logging interface across products"
  s.description = "Standard interface for all logging in Okta apps + SDK. Supports file, console, firebase logging destinations."
  s.homepage = "https://github.com/okta/okta-logger-swift"

diff --git a/OktaLogger/CrashlyticsLogger/OktaLoggerCrashlyticsLogger.swift b/OktaLogger/CrashlyticsLogger/OktaLoggerCrashlyticsLogger.swift
@@ -1,3 +1,5 @@
+// JIRA for proper fix https://oktainc.atlassian.net/browse/OKTA-350983
+#if os(iOS)
 import FirebaseCrashlytics
 
 /**
@@ -79,3 +81,4 @@ open class OktaLoggerCrashlyticsLogger: OktaLoggerDestinationBase {
  return "\(identifier).\(normalizedEventName)"
  }
 }
+#endif