Merge pull request #1088 from rafsanjani/rafsanjani/nullable-os-version

jan-tennert · web-flow · commit b38b5c06d42c · 2025-11-01T18:20:23.000+01:00
fix: Make OS version optional in headers
diff --git a/Supabase/src/commonMain/kotlin/io/github/jan/supabase/PlatformTarget.kt b/Supabase/src/commonMain/kotlin/io/github/jan/supabase/PlatformTarget.kt
@@ -12,7 +12,7 @@ enum class PlatformTarget {
 
 data class OSInformation(
     val name: String,
-    val version: String
+    val version: String?
 ) {
 
     companion object {
diff --git a/Supabase/src/commonMain/kotlin/io/github/jan/supabase/network/KtorSupabaseHttpClient.kt b/Supabase/src/commonMain/kotlin/io/github/jan/supabase/network/KtorSupabaseHttpClient.kt
@@ -115,7 +115,10 @@ class KtorSupabaseHttpClient @SupabaseInternal constructor(
                 append("X-Client-Info", "supabase-kt/${BuildConfig.PROJECT_VERSION}")
                 osInformation?.let {
                     append("X-Supabase-Client-Platform", it.name)
-                    append("X-Supabase-Client-Platform-Version", it.version)
+
+                    it.version?.let { version ->
+                        append("X-Supabase-Client-Platform-Version", version)
+                    }
                 }
             }
             port = HTTPS_PORT

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,10 @@ class KtorSupabaseHttpClient @SupabaseInternal constructor(`
`115`	`115`	`append("X-Client-Info", "supabase-kt/${BuildConfig.PROJECT_VERSION}")`
`116`	`116`	`osInformation?.let {`
`117`	`117`	`append("X-Supabase-Client-Platform", it.name)`
`118`		`- append("X-Supabase-Client-Platform-Version", it.version)`
	`118`	`+`
	`119`	`+ it.version?.let { version ->`
	`120`	`+ append("X-Supabase-Client-Platform-Version", version)`
	`121`	`+ }`
`119`	`122`	`}`
`120`	`123`	`}`
`121`	`124`	`port = HTTPS_PORT`