Open jeherve opened 9 years ago
Is this an internal call that references a url, or a third party theme/plugin using tonesque?
It's a third-party plugin, color-posts. Do you think I should handle the 404 in my plugin (probably somewhere around here?
I do see some room for improvement in our code as well, though. We could check for 404s in Jetpack_PostImages::get_image
, or in Tonesque?
The plugin could just prepend an @ to the calls to avoid 404 errors. I'd be fine adding some error handling in to core in theory.
Worth noting that this will happen for any errors. Here is the same problem with a 403: https://twitter.com/helenhousandi/status/768135636230471680
Related discussion: https://github.com/Automattic/jetpack/pull/3742#discussion_r61244909
This issue has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.
This issue has been marked as stale. This happened because:
No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.
If you try to use Tonesque with an image returning a 404 error, it outputs the following warnings:
Would it be possible to fail silently instead?