Skip to content

Commit

Permalink
Merge pull request #40924 from Expensify/francois-fix-static-thumbnails
Browse files Browse the repository at this point in the history
[CP Staging] Don't include `X-Chat-Attachment-Token` header for static video thumbnails
  • Loading branch information
mountiny authored Apr 24, 2024
2 parents 619ddaa + c53f48d commit 771449b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/components/VideoPlayerPreview/VideoPlayerThumbnail.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ function VideoPlayerThumbnail({thumbnailUrl, onPress, accessibilityLabel}: Video
<Image
source={{uri: thumbnailUrl}}
style={styles.flex1}
isAuthTokenRequired
// The auth header is required except for static images on Cloudfront, which makes them fail to load
isAuthTokenRequired={!CONST.CLOUDFRONT_DOMAIN_REGEX.test(thumbnailUrl)}
/>
</View>
)}
Expand Down

0 comments on commit 771449b

Please sign in to comment.