[bug] Deprecated warning of "Replacing the data" since version 2.3 of Sentry SDK about nextcloud_sentry

[ChristophWurst]

Yep, noticed that as well. I guess we should only run this once. Would you be interested in learning more about how to contribute a fix?

[SimJoSt]

Generally yes. I am familiar with Git, PRs, etc (project manager with a technical background).
The limiting factor would probably my time, as I am already contributing in a lot of other places ;)

If this is a straight forward problem and the fix is trivial, but the work needs to be done and tested, you can give me some pointers so I or somebody else can tackle it as soon as there is some time.

[ChristophWurst]

We init sentry at

nextcloud_sentry/lib/AppInfo/Application.php

Lines 73 to 76 in 9587f8c

	initSentry([
	'dsn' => $dsn,
	'release' => $config->getSystemValue('version', '0.0.0'),
	]);

. The user data is set at

nextcloud_sentry/lib/Reporter/SentryReporterAdapter.php

Lines 91 to 116 in 9587f8c

	configureScope(function (Scope $scope) use ($context): void {
	if (isset($context['level'])) {
	$scope->setLevel(
	new Severity(self::levels[$context['level']])
	);
	}
	if (isset($context['app'])) {
	$scope->setTag('app', $context['app']);
	}
	$user = $this->userSession->getUser();
	if ($user !== null) {
	// Try to obtain the login name as well
	try {
	$credentials = $this->credentialStoreHelper->getLoginCredentials();
	$username = $credentials->getLoginName();
	} catch (CredentialsUnavailableException $e) {
	$username = null;
	}
	$scope->setUser([
	'id' => $user->getUID(),
	'username' => $username,
	]);
	}
	});

. We might move the latter call to the former, so it's just executed once.