api.schedule.ProposalC3VOCPublishingWebhook: fix logic error

emfcamp · Sep 1, 2024 · f6116cc · f6116cc
1 parent 18d6c7e
commit f6116cc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/apps/api/schedule.py b/apps/api/schedule.py
@@ -216,10 +216,10 @@ def post(self):
             if payload["youtube"]["enabled"]:
                 if payload["youtube"]["urls"]:
                     # Please do not overwrite existing youtube urls
+                    youtube_url = payload["youtube"]["urls"][0]
+                    if not youtube_url.startswith("https://www.youtube.com/watch"):
+                        abort(406, message="youtube url must start with https://www.youtube.com/watch")
                     if not proposal.youtube_url:
-                        youtube_url = payload["youtube"]["urls"][0]
-                        if not youtube_url.startswith("https://www.youtube.com/watch"):
-                            abort(406, message="youtube url must start with https://www.youtube.com/watch")
                         # c3voc will send us a list, even though we only have one
                         # video.
                         app.logger.info(f"C3VOC webhook set youtube_url for {proposal.id=} to {youtube_url}")