Public
- Public
- Network
- Groups
- Featured
- Popular
- People

a line of added code in a pull request that contains a call to a function called sqlEscape under it, a comment from me: "deeply amused that I probably could've merged this without review if only I hadn't properly escaped this one string"

Download link

a line of added code in a pull request that contains a call to a function called sqlEscape under it, a comment from me: "deeply amused that I probably could've merged this without review if only I hadn't properly escaped this one string"
https://sparkly.uni.horse/system/media_attachments/files/113/357/633/112/633/907/original/79823edfa84b89bf.png

Notices where this attachment appears

Embed this notice
emily, emitter of spooky noise (emily@sparkly.uni.horse)'s status on Thursday, 24-Oct-2024 01:20:17 JST emily, emitter of spooky noise

$work has optional code review, based half on "do you, the developer, think this needs review" vibes, and half on a regex that matches words like "sql" with as many false positives as you would expect
which means this was going to happen eventually

In conversation about a month ago from sparkly.uni.horse permalink