Use regex

OskarStark · web-flow · commit 7fb9e6d0275d · 2020-12-22T11:50:53.000+01:00
diff --git a/src/Symfony/Component/Notifier/Bridge/Slack/SlackTransport.php b/src/Symfony/Component/Notifier/Bridge/Slack/SlackTransport.php
@@ -36,7 +36,7 @@ final class SlackTransport extends AbstractTransport
 
     public function __construct(string $accessToken, string $channel = null, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)
     {
-        if (!\in_array(substr($accessToken, 0, 5), ['xoxb-', 'xoxp-'], true) && 0 !== strpos($accessToken, 'xoxa-2')) {
+        if (!preg_match('/^xox(b-|p-|a-2)/', $accessToken)) {
             throw new InvalidArgumentException('A valid Slack token needs to start with "xoxb-", "xoxp-" or "xoxa-2". See https://api.slack.com/authentication/token-types for further information.');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ final class SlackTransport extends AbstractTransport`
`36`	`36`
`37`	`37`	`public function __construct(string $accessToken, string $channel = null, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)`
`38`	`38`	`{`
`39`		`- if (!\in_array(substr($accessToken, 0, 5), ['xoxb-', 'xoxp-'], true) && 0 !== strpos($accessToken, 'xoxa-2')) {`
	`39`	`+ if (!preg_match('/^xox(b-\|p-\|a-2)/', $accessToken)) {`
`40`	`40`	`throw new InvalidArgumentException('A valid Slack token needs to start with "xoxb-", "xoxp-" or "xoxa-2". See https://api.slack.com/authentication/token-types for further information.');`
`41`	`41`	`}`
`42`	`42`