Покритикуйте скрипт рассылки сообщений

Запустил твой код у себя. Вот то что хранится в $output

string(547) "HTTP/1.1 100 Continue HTTP/1.1 401 Access-Control-Allow-Origin: * Access-Control-Allow-Methods: GET, POST, PUT, PATCH, DELETE, COPY, HEAD, OPTIONS, LINK, UNLINK, PURGE Access-Control-Allow-Headers: Authorization, Content-Type, X-Enrollment-Token Content-Type: application/json;charset=UTF-8 Content-Length: 97 Date: Fri, 08 Jun 2018 08:11:22 GMT Server: SMS API Strict-Transport-Security: max-age=16000000; includeSubDomains; preload; {"requestError":{"serviceException":{"messageId":"UNAUTHORIZED","text":"Invalid login details"}}}" 

Все отработало без ошибок. PHP 7.0

спасибо. странно, что у меня при запуске из консоли этого не отобразилось.

Зачем пачкаться об curl, если можно просто делать file_get_contents с указанием контекста. Или взять guzzle, unirest... тысячи их.

Также непонятна страть к array(), вместо [].

Еще мой дежурный перфекционист требует выравнять => в массиве

[
  'key'   => 'val',
  'other' => 'val2',
]

спасибо.

https://dev.tele2messaging.com/docs/send-single-sms

изначальный пример вообще был на HttpRequest, который в убунту1604 вёл себя очень...никак. пришлось перелезать на курл.

Также непонятна страть к array

я новичок. а пример из интернета на array был интуитивно понятен, что и как там делать и располагать.

да собственно тут мало что критиковать можно, скриптик маленький, все более ли менее аккуратно написано

HttpRequest

Какое-то протухшее говно мамонта.

Да и ещё вдогонку. Номер телефона и ключ авторизации лучше не хардкодить. Может у тебя это тестовый пример, но на всякий случай правильнее получать это добро из файла, который добавить в .gitignore

$creds = require('creds.conf.php');
...
// creds.conf.php
return [
  'phone' => '+7555..',
  'key'   => 'HJiuyvIu=',
];