Replace sdkInfo on iOS

[krystofwoldrich]

Description

sentry-react-native/ios/RNSentry.m

Lines 158 to 161 in 86821a4

	resolve(@{
	@"name": sentryOptions.sdkInfo.name,
	@"version": sentryOptions.sdkInfo.version
	});

sdkInfo is deprecated getsentry/sentry-cocoa#1960

PrivateSentrySDKOnly.getSdkName
and
PrivateSentrySDKOnly.getSdkVersionString
to retrieve the information

Cocoa issue

[brustolin]

Instead of this:

resolve(@{ 
     @"name": sentryOptions.sdkInfo.name, 
     @"version": sentryOptions.sdkInfo.version 
         }); 

do this

resolve(@{ 
     @"name": PrivateSentrySDKOnly.getSdkName, 
     @"version": PrivateSentrySDKOnly.getSdkVersionString 
         }); 

[marandaneto]

@brustolin that's possible but still not possible to add a new package.
getsentry/sentry-dart#1056 (comment)

[marandaneto]

on RN iOS, the new package is added on the JS layer.

sentry-react-native/src/js/integrations/sdkinfo.ts

Lines 54 to 61 in 86821a4

	event.sdk = {
	...(event.sdk ?? {}),
	...defaultSdkInfo,
	packages: [
	...((event.sdk && event.sdk.packages) || []),
	...((this._nativeSdkInfo && [this._nativeSdkInfo]) || []),
	...defaultSdkInfo.packages,
	],

via _nativeSdkInfo that is fetch from fetchNativeSdkInfo
So this is possible with the #2530 (comment) alternative.

[brustolin]

Can we close this?

[marandaneto]

Can we close this?

This is not fixed yet, why would we close it?

[github-actions]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀