4 ms·
I worked with a team that owned a service that resizes images. An engineer was assigned a task to add support for auto rotating images. His solution involved sa
by akamia 5y ago
I worked with a team that owned a service that resizes images. An engineer was assigned a task to add support for auto rotating images. His solution involved saving the image to a file and then using a library to handle the rotation. He used a hardcoded value for the file name. In a local environment where requests are sparse this looked fine to him and other engineers on the team missed it in code reviews. It wasn't until it went out to prod that he realized the error in this. Users started seeing other users' images because the file's content was constantly being overwritten.
When you test features like this or caching a response with a JWT it can be very easy to default to the happy path or ignore the impact of a large volume of concurrent users.
- auggierose 5y ago"An engineer was assigned" Nope. That definitely wasn't an engineer.
- beckingz 5y agoReal software engineers don't make mistakes?
- auggierose 5y agoNot mistakes like that.
- _vertigo 5y agoNo true Scottish engineer would have made that error!
- auggierose 5y ago:-)
- akamia 5y agoMistakes happen. I've never met an engineer who has never made a mistake. However, I have met brilliant engineers who have written incredibly complex software and have also managed to make some silly mistakes along the way.
- anaganisk 5y agoElitism alert!