Merge branch '6.4' into 7.0

* 6.4: (32 commits)
  [Validator] Fix registering "is_valid()" for `#[Expression]`
  [Scheduler] Trigger unique messages at runtime
  [Scheduler] Fix CHANGELOG
  [Clock] Add `DatePoint`: an immutable DateTime implementation with stricter error handling and return types
  [Scheduler] Allow modifying the schedule at runtime and recalculate heap
  [Cache] Fix Redis6Proxy
  [Finder] Disable failing test about open_basedir
  Fix merge
  Fix merge
  Minor CS fixes
  Deprecate `Kernel::stripComments()`
  Remove setAccessible reflection call in tests
  [Notifier] Telegram Bridge add escaping for \
  [Component][AssertMapper] add type hint of an argument in asset mapper command
  [Translation] [Phrase] Refacto ReadConfig and WriteConfig into arrays
  [Routing] Fix routing collection defaults when adding a new route to a collection
  [Messenger] Fix cloned TraceableStack not unstacking the stack independently
  [Translation] Add `--as-tree` option  to `translation:pull` command
  [Mime] Allow to add some headers as a strings
  [Translation] Give current locale to locale switcher runWithLocale callback
  ...

Author: nicolas-grekas

SmsmodeTransport.php

@@ -69,7 +69,10 @@ protected function doSend(MessageInterface $message): SentMessage
         }
 
         $response = $this->client->request('POST', $endpoint, [
-            'headers' => ['X-Api-Key' => $this->apiKey],
+            'headers' => [
+                'X-Api-Key' => $this->apiKey,
+                'Accept' => 'application/json',
+            ],
             'json' => array_filter($options),
         ]);
 

Tests/SmsmodeTransportTest.php

@@ -80,6 +80,27 @@ public function testNoInvalidArgumentExceptionIsThrownIfFromIsValid(string $from
         self::assertSame('foo', $sentMessage->getMessageId());
     }
 
+    public function testHttpClientHasMandatoryHeaderAccept()
+    {
+        $message = new SmsMessage('+33612345678', 'Hello!');
+
+        $response = $this->createMock(ResponseInterface::class);
+        $response->expects(self::exactly(2))->method('getStatusCode')->willReturn(201);
+        $response->expects(self::once())->method('getContent')->willReturn(json_encode(['messageId' => 'foo']));
+
+        $transport = $this->createTransport(new MockHttpClient(function (string $method, string $url, array $options) use ($response): ResponseInterface {
+            $this->assertSame('POST', $method);
+            $this->assertSame('https://rest.smsmode.com/sms/v1/messages', $url);
+            $this->assertSame('Accept: application/json', $options['normalized_headers']['accept'][0]);
+
+            return $response;
+        }), 'foo');
+
+        $result = $transport->send($message);
+
+        $this->assertSame('foo', $result->getMessageId());
+    }
+
     public static function toStringProvider(): iterable
     {
         yield ['smsmode://rest.smsmode.com?from=test_from', self::createTransport()];