6 ms·
> it's not really representative of the sector of developers that make a living with PHP. Anecdote: I was integrating a paid Magento extension (which charges a
by DCoder 10y ago
> it's not really representative of the sector of developers that make a living with PHP.
Anecdote: I was integrating a paid Magento extension (which charges about $1000 for a license) into an existing shop, and some poor code in their templates made me look deeper into their main code. Six hours later, I had:
* 4 different ways to read any file on the server that PHP can read,
* 5 different ways to upload any file to a certain directory on the server, including one that lets you overwrite the webserver's security configuration for that directory and then execute uploaded PHP code,
* a way to delete certain things the administrator has created in the backend,
* a way to overwrite other customers' uploaded files,
* a way to edit other customers' information, which lets you then XSS/XSRF those customers when they view that information.
I have reviewed other Magento extensions in the past, and I found many poor ones, but nothing as atrocious as this.
- benmarks 10y agoIf you (or anyone) ever sees anything like this please email marketplace@magento.com or me at ben@magento.com. Probably you encountered this in the days of Magento Connect. Our new app store (Marketplace) actually hosts the code and has static code tests for many things, including OWASP items.
- DCoder 10y agoThank you for the contact details, I'll send you the details of the security issues later this week. This extension was purchased through Magento Connect just a few months ago. I'm glad to hear the situation is improving now.