Elgato_dark/appwrite
1
0
Fork 0
mirror of https://github.com/appwrite/appwrite synced 2026-05-23 08:58:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Simplify country code span logging for internal SMS messages

Browse source 
Internal SMS always has a single recipient, so remove the unnecessary
count-based tracking and simplify to just log the country code directly.
This commit is contained in:
Chirag Aggarwal 2026-02-04 15:18:28 +05:30
parent ce49b8ecb8
commit 4ef43aacd9
1 changed files with 4 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/Appwrite/Platform/Workers/Messaging.php
View file

@ -432,18 +432,11 @@ class Messaging extends Action
private function sendInternalSMSMessage(Document $message, Document $project, array $recipients, Log $log): void
{
Span::add('providerType', 'sms');
Span::add('recipientsCount', \count($recipients));
// Extract country codes from phone numbers
$countryCodes = [];
foreach ($recipients as $recipient) {
$countryCode = $this->extractCountryCode($recipient);
if ($countryCode !== null) {
$countryCodes[$countryCode] = ($countryCodes[$countryCode] ?? 0) + 1;
}
}
foreach ($countryCodes as $code => $count) {
Span::add('countryCode_' . $code, $count);
// Extract country code from the single recipient phone number
$countryCode = $this->extractCountryCode($recipients[0] ?? '');
if ($countryCode !== null) {
Span::add('countryCode', $countryCode);
}
if ($this->adapter === null) {