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

(cherry picked from commit 771449b)
  • Loading branch information
mountiny authored and OSBotify committed Apr 24, 2024
1 parent 4b3c6ae commit 6d0501b
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 6d0501b

Please sign in to comment.