search

guardian / grid

The Guardian’s image management system
https://www.theguardian.com/info/developer-blog/2015/aug/12/open-sourcing-grid-image-service
Apache License 2.0
1.43k stars 119 forks source link

[ingestion] handle S3 file not existing manifesting as a 403 not just a 404 #4250

Closed twrichards closed 4 months ago

twrichards commented 4 months ago

Observing this in PROD image

so broadening the definition of not existing to cover 403 too

github-actions[bot] commented 4 months ago

Deploy build 12321 to TEST

All deployment options - [Deploy build 12321 to TEST](https://riffraff.gutools.co.uk/deployment/deployAgain?project=media-service%3A%3Agrid%3A%3Aall&build=12321&stage=TEST&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 12321 to TEST by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=media-service%3A%3Agrid%3A%3Aall&build=12321&stage=TEST&updateStrategy=MostlyHarmless)

From guardian/actions-riff-raff.

prout-bot commented 4 months ago

Seen on auth, usage, image-loader, metadata-editor, thrall, leases, cropper, collections, media-api, kahuna (merged by @twrichards 8 minutes and 31 seconds ago) Please check your changes!