Fix: Extrahiere verwendbare URL aus Webhook-Payload (Case 188021) (#48)

Verwende `repository->html_url` statt `repository->url` aus der
Webhook-Payload.

Historisch hatte die Webhook-Payload unter dem Key `repository->url`
Werte wie `https://github.com/webfactory/baton`. Neuerdings erhalten wir
hier Werte wie `https://api.github.com/repos/webfactory/baton` - diese
URLs werden von composers GitHubDriver nicht unterstützt, so dass damit
keine Repository-Zugriffe mehr möglich sind. Das Drop-in-Replacement mit
gewohnten Werten ist unter dem Schlüssel `repository->html_url` zu
finden.

So, wie wir die URL für Repository-Zugriffe verwenden, könnte der Key
`repository->git_url` mit Werten wie
`git://github.com/webfactory/baton.git` noch besser passen. Allerdings
wird die (letztlich in Project.vcsUrl persistierte) URL auch verwendet,
um in einer Twig-View einen Link auf das Repository auszugeben - dafür
eignet sich die `repository->git_url` im Gegensatz zur
`repository->html_url` nicht. Man könnte überlegen, die beiden URLs nach
ihrer Funktion zu separieren, das ist aber für den Moment out of scope.

(cherry picked from commit ef2839a5ef0f8110ea29ba5f70fcd1da86cf4428)

Author: MalteWunsch

src/AppBundle/Controller/WebhookController.php

@@ -10,14 +10,9 @@
 
 class WebhookController
 {
-    /**
-     * @var ImportProjectTask
-     */
-    private $importProjectTask;
-
-    public function __construct(ImportProjectTask $importProjectTask)
-    {
-        $this->importProjectTask = $importProjectTask;
+    public function __construct(
+        private ImportProjectTask $importProjectTask
+    ) {
     }
 
     /**
@@ -32,9 +27,9 @@ public function updateAction(Request $request)
 
         // This works for the Kiln webhook API as well as for GitHub webhooks of content-type "application/x-www-form-urlencoded"
         if ($payload = $request->get('payload')) {
-            $payload = json_decode($payload);
-            if (isset($payload->repository) && $payload->repository && $payload->repository->url) {
-                $repositoryWasImported = $this->importProjectTask->run($payload->repository->url);
+            $payload = json_decode($payload, flags: JSON_THROW_ON_ERROR);
+            if (isset($payload->repository) && $payload->repository && $payload->repository->html_url) {
+                $repositoryWasImported = $this->importProjectTask->run($payload->repository->html_url);
             }
         }