News for hackers. Stuff that matters.
Re:Klarna users are being signed in to random accounts by akamia on Thursday May 27, 2021 @12:12PM (#27304728)
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.

Re:Klarna users are being signed in to random accounts by auggierose on Thursday May 27, 2021 @02:22PM (#27306320)
"An engineer was assigned"

Nope. That definitely wasn't an engineer.

Re:Klarna users are being signed in to random accounts by akamia on Thursday May 27, 2021 @04:02PM (#27307563)
Mistakes 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.
Re:Klarna users are being signed in to random accounts by _vertigo on Thursday May 27, 2021 @03:49PM (#27307372)
No true Scottish engineer would have made that error!
Re:Klarna users are being signed in to random accounts by auggierose on Thursday May 27, 2021 @04:47PM (#27308285)
:-)
Re:Klarna users are being signed in to random accounts by beckingz on Thursday May 27, 2021 @02:57PM (#27306737)
Real software engineers don't make mistakes?
Re:Klarna users are being signed in to random accounts by auggierose on Friday May 28, 2021 @02:19PM (#27318762)
Not mistakes like that.
Re:Klarna users are being signed in to random accounts by anaganisk on Friday May 28, 2021 @04:33AM (#27313027)
Elitism alert!