You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
…abbuh)
This PR was merged into the 6.4 branch.
Discussion
----------
[Notifier] send the recipient phone number as an array
| Q | A
| ------------- | ---
| Branch? | 6.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | Fix#57393
| License | MIT
Commits
-------
86480fe send the recipient phone number as an array
Symfony version(s) affected
>=6.4.x
Description
I am using the symfony/clickatell-notifier bridge and i came across an error while trying to send an SMS to any valid phone number.
How to reproduce
config/packages/notifier.yaml
Invalid or missing parameter: to
is returned by the Clickatell APIFull example to test with any number / SMS body :
Notifier configuration :
Symfony command (
app:notifier:sms
) for sending SMS using CLI :Possible Solution
After some digging, i found the issue while reading Clickatell SMS REST API documentation.
In their example, the
to
parameter is an array:However a seen in the https://github.com/symfony/clickatell-notifier/blob/b180e5aeacce4dfd600bc1fcdc358ceac86472b8/ClickatellTransport.php#L64
the
to
parameter is sent as astring
an not anarray
.So the resulting request body is
which is invalid.
I can see that the
to
parameter was changed in this commit symfony/clickatell-notifier@1f6b83e.Solution
Replace the line 64 in https://github.com/symfony/clickatell-notifier/blob/b180e5aeacce4dfd600bc1fcdc358ceac86472b8/ClickatellTransport.php#L64
Additional Context
Without fix
With fix
Suspected commit
symfony/clickatell-notifier@1f6b83e
The text was updated successfully, but these errors were encountered: