this post was submitted on 27 Jul 2023
7 points (100.0% liked)

Programming Humor

2612 readers
11 users here now

Related Communities [email protected] [email protected] [email protected] [email protected]

Other Programming Communities [email protected] [email protected] [email protected] [email protected] [email protected] [email protected] [email protected]

founded 1 year ago
MODERATORS
7
DO NOT TOUCH (lemmy.world)
submitted 1 year ago* (last edited 1 year ago) by [email protected] to c/[email protected]
 

Edit: so im done with my preliminary research into this codebase.

Our corporate SSO provider is changing, so I've been updating our tools to take advantage of the new badges. I found this in a web application that I started on today. The original developer is long gone, and according to our PaaS, this app has been running for just under 3 years without an update.

There is no CI/CD, blue-green deployment, or back ups. The database is an H2 db with ddl-auto set to create-drop on startup, meaning that this database will delete itself if the app is restaged but thanks to this guys code, it won't populate itself. ๐Ÿคท

top 8 comments
sorted by: hot top controversial new old
[โ€“] [email protected] 1 points 1 year ago

Good Lord, this makes my hands sweaty. Why is your entire prod database leaning on one line of code that's prone to human error? There should be 20 extra accidental steps taken to do something like this.

[โ€“] [email protected] 1 points 1 year ago

Dropping the database is not recoverable

This is the real problem

[โ€“] [email protected] 1 points 1 year ago

You wonder why spaceships have self-destruct option?

[โ€“] [email protected] 0 points 1 year ago* (last edited 1 year ago)
[โ€“] [email protected] 0 points 1 year ago (1 children)

Found this in production while migrating SSO providers. Made me chuckle ๐Ÿ™ƒ

[โ€“] [email protected] 1 points 1 year ago* (last edited 1 year ago) (2 children)

DO NOT RUN IN PROD

Found this in production

Classic

[โ€“] [email protected] 1 points 1 year ago

Only thing better is finding commented out code below that which would actually prevent it from running in Prod. Bonus if there's a code comment next to it saying "disabled per email" with no further explanation.

[โ€“] [email protected] 1 points 1 year ago

When I sat down today I thought I was just going to be updating some properties file with oauth end points.

This is so blatantly stupid that I now have to pick through the code base and write up a change request and incident avoidance report ๐Ÿ˜•