-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
Level: AdvancedMost developers should anticipate frictionMost developers should anticipate frictionPriority: NormalCruising speedCruising speedSize: MConsumes a day or twoConsumes a day or two
Description
Had an RSS import where some image filenames had weird non-ascii non-encoded narrow no-break spaces.
Not sure if github will keep the NNBS... but:
<image>
<url>https://philosophytalk.org/wp-content/uploads/2022/06/Screenshot-2025-03-21-at-3.05.25 PM.png</url>
<title>Is Optimism Rational?</title>
</image>
First up, the EpisodeImports errored. Probably because URI.parse errors on these (in file_name). We did end up with an Episode and EpisodeImage, but there were issues with the Uncut and Image not firing porter jobs.
Second, you got a 500 trying to go to the episode. Again because the file_name is on the page.
Third, you can't put that un-encoded char into the Content Disposition. (Though you can in the Object Key). So we should probably drop non-ascii from the content disposition?
Metadata
Metadata
Assignees
Labels
Level: AdvancedMost developers should anticipate frictionMost developers should anticipate frictionPriority: NormalCruising speedCruising speedSize: MConsumes a day or twoConsumes a day or two