Changeset 2873b30 in Klonkt

Timestamp:

06/20/2026 02:54:09 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

557a76f

Parents:

16b0c00

Message:

fix(embed): Spotify directly as bare iframe (instant, no API wait)

The Spotify iFrame API bundle often fails to initialise in practice (CDN
gating), and their player cannot be skinned anyway. Therefore with Spotify
show the bare iframe immediately instead of waiting for the API → instant
rather than seconds of "sluggishness". CSP whitelist + 4s timeout (previous
commit) remain as a fallback for those who do get the API. buster
embed-player.js?v=6.

Co-Authored-By: Claude <noreply@…>

Location:

src

Files:

• assets/js/embed-player.js (modified) (1 diff)
• views/shell.ejs (modified) (1 diff)

src/assets/js/embed-player.js

@@ -290 +290 @@
       if (mounted) { if (adapter) adapter.play(); return; }
       mounted = true;
+      // Spotify: hun speler is toch niet te skinnen (besturing-only) én de
+      // iFrame-API-bundle initialiseert in de praktijk vaak niet (CDN-gating/503)
+      // → niet op de API wachten, meteen het kale Spotify-iframe tonen. Instant.
+      if (provider === 'spotify') { renderFallback(); return; }
       el.classList.add('pcms-embed-busy');
       try {

src/views/shell.ejs

@@ -303 +303 @@
 <!-- Eigen custom media-embeds (YouTube/SoundCloud/Spotify) via de echte
      player-API's + gedeelde mutual-exclusion registry met de site-speler. -->
-<script src="/assets/js/embed-player.js?v=5" defer></script>
+<script src="/assets/js/embed-player.js?v=6" defer></script>
 <% if (site && site.enable_audio_player && audioTracks && audioTracks.length > 0) { %>
   <script>window.PCMS_SITE_TRACKS = <%- JSON.stringify(audioTracks) %>;</script>