fix: analytics query-string for better browser cache (#1024)

Author: tsi

src/plugins/cloudinary/index.js

@@ -207,7 +207,7 @@ class CloudinaryContext {
         { hasUserPosterOptions: hasUserPosterOptions || null }
       );
 
-      options.queryParams = Object.assign(options.queryParams || {}, options.allowUsageReport ? { _s: `vp-${VERSION}` } : {});
+      options.queryParams = Object.assign(options.queryParams || {}, options.allowUsageReport ? { _s: 'vp' } : {});
 
       if (options.sourceTypes.indexOf('audio') > -1) {
         builtSrc = new AudioSource(publicId, options);

src/plugins/cloudinary/models/video-source/video-source.js

@@ -130,7 +130,10 @@ class VideoSource extends BaseSource {
 
     options.cloudinaryConfig = options.cloudinaryConfig || this.cloudinaryConfig();
     options.resource_type = this.resourceType() || options.resource_type;
-    options.queryParams = this.queryParams();
+
+    if (!isRawUrl(publicId)) {
+      options.queryParams = this.queryParams();
+    }
 
     if (publicId === true) {
       const urlPrefix = getCloudinaryUrlPrefix(options.cloudinaryConfig);

src/utils/fetch-config.js

@@ -25,7 +25,7 @@ const fetchConfig = async (options) => {
 
   const cloudinaryConfig = getCloudinaryConfigFromOptions(options);
   const urlPrefix = getCloudinaryUrlPrefix(cloudinaryConfig) + '/_applet_/video_service';
-  const queryParams = allowUsageReport ? { _s: `vp-${VERSION}` } : null;
+  const queryParams = allowUsageReport ? { _s: 'vp' } : null;
 
   let configUrl;
   if (profile) {

test/unit/videoSource.test.js

@@ -571,6 +571,24 @@ describe('test hasCodec method', () => {
       expect(posterUrl).toContain('/video/upload/');
     });
 
+    it('does not append source queryParams to raw absolute poster URLs', () => {
+      const posterHref = 'https://cdn.example.com/assets/poster.jpg';
+      const source = new VideoSource('sea_turtle', {
+        cloudinaryConfig: cld,
+        queryParams: { _s: 'vp' },
+        poster: posterHref
+      });
+      expect(source.poster().url()).toBe(posterHref);
+    });
+
+    it('appends stable _s to Cloudinary-built poster URLs', () => {
+      const source = new VideoSource('sea_turtle', {
+        cloudinaryConfig: cld,
+        queryParams: { _s: 'vp' }
+      });
+      expect(source.poster().url()).toContain('_s=vp');
+    });
+
     it('should use configured resourceType for seek thumbnails', () => {
       const source = new VideoSource('sample', {
         cloudinaryConfig: cld,