iframe API startAt bug

jaredp_ — Thu, 02 Nov 2023 23:52:36 GMT

Hello!

I was working with the Spotify iframe API found here:

https://open.spotify.com/embed/iframe-api/v1

and noticed a bug related to this.options.startAt found in this script:
https://embed-cdn.spotifycdn.com/_next/static/iframe_api.92ca45a15d3476e3ab9a.js

the bug is found inside the function loadUri:

this.loadUri = (uriString, preferVideo = false, timestampInSeconds = 0)

specifically this block:

if (this.options.startAt || timestampInSeconds) { const startAt = timestampInSeconds !== null && timestampInSeconds !== void 0 ? timestampInSeconds : parseInt((_a = this.options.startAt) !== null && _a !== void 0 ? _a : '0', 10); if (!isNaN(startAt)) { embedURL.searchParams.append('t', startAt.toString()); } }

the bug is that this.options.startAt will never be used since timestampInSeconds defaults to 0.

to fix it, I believe all that needs to be added is && timestampInSeconds !== 0 so we fallback to this.options.startAt if it is 0:

if (this.options.startAt || timestampInSeconds) { const startAt = timestampInSeconds !== null && timestampInSeconds !== void 0 && timestampInSeconds !== 0 ? timestampInSeconds : parseInt((_a = this.options.startAt) !== null && _a !== void 0 ? _a : '0', 10); if (!isNaN(startAt)) { embedURL.searchParams.append('t', startAt.toString()); } }

P.S: I'm not really sure why timestampInSeconds exists since the loadUri function is only called with 2 arguments everytime (uri and preferVideo)

not sure where the correct place to post this was, hopefully this was the place!

Re: iframe API startAt bug

trevorflan — Thu, 02 Nov 2023 23:43:27 GMT

When I was using the iFrame API, I had a similar use case. I saw that there were some stubs for startAt, but they never did anything. I ended up hacking together a solution that just called seek() until the page finished loading, but this would have been much cleaner. +1

Re: iframe API startAt bug

stellaw1 — Fri, 03 Nov 2023 00:03:25 GMT

+2!

topic iframe API startAt bug in Spotify for Developers

iframe API startAt bug

Re: iframe API startAt bug

Re: iframe API startAt bug