Actually we do need to do one thing, for future-proofing, we should check if the URL already contains a "?" character and if it does, use a "&" instead of a "?" when adding the "v=" parameter (and maybe also use a different parameter name that is less likely to collide, like "version=" or "mtime=")