Limit responsive image URLs

HTTPArchive · Sep 13, 2024 · a9f2a5a · a9f2a5a
1 parent 21e084c
commit a9f2a5a
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/dist/responsive_images.js b/dist/responsive_images.js
@@ -305,7 +305,11 @@ function intrinsicOrExtrinsicSizing( computedStyles ) {
 function getImgData( img ) {
 
  const imgData = imgFeatures( img );
- imgData.url = img.currentSrc || img.src;
+ // Limit URLs to 200 chars so data URLs don't take up huge amounts of space.
+ imgData.url =
+ (img.currentSrc || img.src).length > 200 ?
+ (img.currentSrc || img.src).slice(0, 199) + '…' :
+ (img.currentSrc || img.src);
  imgData.totalCandidates = totalNumberOfCandidates( img );
 
  if (imgData.hasHeight) {
@@ -358,7 +362,8 @@ function getImgData( img ) {
  !isFromSource &&
  !( imgData.srcsetHasWDescriptors ) ) {
  srcsetCandidates.push( {
- url: img.getAttribute( 'src' )
+ // Limit URLs to 200 chars so data URLs don't take up huge amounts of space.
+ url: img.getAttribute( 'src' ).length > 200 ? img.getAttribute( 'src' ).slice(0, 199) + '…' : img.getAttribute( 'src' )
  } )
  }