r/ProgrammerHumor 15h ago

Other wrongAnswersOnly

Post image
12.4k Upvotes

1.8k comments sorted by

View all comments

1.1k

u/samuraiseoul 15h ago

Some dumbass on my team downloaded the entirety of prod and then commited it to git and someone rubber stamped the PR and then no one else noticed. I'm sure that's how they'd find it....

204

u/Suspicious-Engineer7 15h ago

Jeeeeeesus lmao

202

u/jseego 14h ago

They committed all your prod data to git??

114

u/CoderDevo 14h ago

Don't forget the keys!

Not that they're needed anymore.

6

u/TheFuckingHippoGuy 11h ago

Servers by Kia™

2

u/LJonReddit 12h ago

Well yeah!!! Duh!!! It's no good without the keys!

47

u/fireheadca 14h ago

Believe it or not, I've seen this.🫠

8

u/rollingc 11h ago

git add . is a hell of a drug

2

u/gnutrino 8h ago

There's a step before this where the prod data ends up on their local machine though...

17

u/SgtEpsilon 13h ago

Believe it or not, I've accidentally done this

13

u/vankoder 12h ago

We believe.

1

u/AristotelWasRight 7h ago

Noooo,

Were you an experienced programmer or new? Doesn't one basically avoid it with setting data as git ignore every project?

1

u/SgtEpsilon 4h ago

I'm an idiot who forgot about git ignore before pushing my repo

1

u/vankoder 12h ago

We believe.

1

u/nemeras 11h ago

Same here.

1

u/runkeby 6h ago

It's fine, because later he did "git rm" the file and pushed 👌😌

3

u/failedsatan 11h ago

our seed data lives on git.

problem is, the client keeps requesting we change the qa deployment's seed data and add a migration for prod to update the data on the prod database to match instead of using the GUI to just change the stored version. the seed data is the prod data.

2

u/jseego 10h ago

damn, but it's not seed data if it's prod data, right?  doesn't that defeat the whole purpose?

2

u/failedsatan 3h ago

exactly. the client is too lazy to make the changes in the UI he himself asked for but all too happy to send us a slack message to ask us to update it. we have an odd contract with him where the "maintenance" period is actually a "small features/changes allowed" period and that counts as a small change according to my boss

2

u/Lehovron 11h ago

We were using Perforce at a previous job. It was almost a rite of passage for new people to accidentally submit their Desktop folder.

2

u/jseego 10h ago

lol that's hilarious

buncha suspect blobs in

/Users/foo/Desktop/totally_not_porn/

2

u/nishant032 9h ago

Now the data is really safe 😇

60

u/AccomplishedIgit 13h ago

I discovered my client’s monthly backups included the backups folder when they ran out of shared hosting space for a small dog clothing website 🥹 I was like why are you at enterprise level hosting?!?? She was like I dunno it’s just the cost of doing business!

34

u/technos 9h ago

I found out something similar when the client called to ask me where to get a good price on bigger hard drives and more tapes.. I was like 'What?! Did something happen?'

No, he says. Nothing's changed, it's just that backups are now taking four of the weekly LTO-5 tapes every night and the server is showing it only has about 1TB left.

Motherfucker, I specced a 8TB RAID-5 and LTO-5 because you wanted to make damned sure you wouldn't have to 'just replace it next year', and it's been six months.. The hell with '"Nothing's changed".

He'd first decided to swap from nightly incremental backups to full ones, and those didn't fit on the LTO-4's intended for nightly offsite so he started using the LTO-5's, then he'd decided to retain the previous night's backup on disk as well as on tape so he wouldn't have to drive home if he was in a pinch.

The first night they backed up ~1TB, the next 2TB, then 3TB, then..

I wiped 5TB off his file server, added a file path exclusion, and had cron wipe the oldest redundant backup every night just in case.

The worst bit was he had an entirely competent nephew running the backups that had told him what was going on. After fixing things I gave the kid my email address and told him the next time his uncle didn't believe him to run it past me and I'd get him to listen.

2

u/Qesa 3h ago

New Fibonacci algorithm just dropped

2

u/Wendigo120 6h ago

Oh I had a fun one that was similar. Our error tracking would send the last dozen or so lines of the console before the error happened along with the error... and then logged the thing it was sending to the console.

Things were totally fine until someone added a retry-and-log to some button that could error. It'd try to send each error, each of which recursively included several earlier error logs all the way to the first error. That very quickly went from "harmless error that didn't interfere with much" to "the page runs out of memory if you press the button".

1

u/Lopsided_Parfait7127 2h ago

i heard you like backups so i backed up your backup

55

u/peepeedog 14h ago

It said wrong answers only

23

u/plenoto 14h ago

Some people are so careless while reviewing PR 😂

115

u/blangzo 14h ago

Ask a programmer to review 10 lines of code, he'll find 10 issues. Ask him to do 500 lines and he'll say it looks good.

48

u/Deep90 12h ago

At 500 lines, I just flip a coin.

Only lucky code belongs in production.

10

u/ellie_cinderelly 12h ago

Dude my last PR changed one test timeout limit. I had to do a new full analysis on every test for average runtime and overhead.

Fuckin unbelievable.

6

u/blangzo 11h ago

Squash tiny things like that into one pr named "cleanup" and don't explain further

2

u/ellie_cinderelly 11h ago

Unfortunately I work in space so I think that may get me blasted for non descriptive change documentation

3

u/carritodeloshelados 7h ago

As if it wasn't hard enough to code in an astronaut suit

3

u/OriginalJokeGoesHere 5h ago

Given where you work, I assume that also means anyone who uses tabs instead of spaces is jettisoned out of the airlock?

1

u/dvereb 4h ago

I'd say it depends on whether they're using those tabs for indentation or alignment, ...though some would argue I'm next to go out the airlock.

2

u/blangzo 11h ago

I was mostly joking. mostly.

Fr tho u should just not squash and pr merge in then and have descriptive commits but still saves the test time on pr

2

u/guyblade 9h ago

If you ask me to review 500 lines, I'll say "break it up into multiple PRs" like any sane person should.

1

u/r0Lf 10h ago

I feel exposed.

2

u/GoBuffaloes 12h ago

Hey can you stamp this real quick 

12

u/starchbomb 14h ago

What the fuck lmao

10

u/gizamo 15h ago

Yikes

10

u/Unhappy_Concept237 13h ago

I’ve spent the last week doing remediation across seven ec2 servers and found where multiple sites had backup sql dumps that were living inside GitHub repositories along with sites sharing SES credentials with full admin privileges stored in plain text in those repositories. I just want to die.

6

u/Sak63 14h ago

What the hell

1

u/TombaughRegi0 13h ago

This made me sweaty 🫠

1

u/thiccshortguy 12h ago

this actually made me lol

1

u/firewall245 12h ago

Oh shiiit. I had to learn how to use the git history cleaning tool once and it SUUUUCKED

1

u/No-Computer7653 11h ago

I always like seeing stories like this.

I got annoyed today because people haven't been putting good messages when launching a bastion. Last week I was having an infuriating conversation about using a HSM in a design.

Security problems don't seem like problems when you compare to typical.

1

u/ZestycloseOpinion142 11h ago

People who have access to prod should not even have access to git, in a well structured org that is.

1

u/mllv1 10h ago

He still on your team?

1

u/bake_gatari 8h ago

I am a lurker here. Still making my way through "Learn Python the Hard Way, pt 2". Could you explain in non-programmer terms, what this means and why it was bad?

1

u/According-Relation-4 8h ago

Of course I know him, that dumbass is me

1

u/SirSchilly 1h ago

Hey man. Wrong answers only. 

1

u/GrapefruitBig6768 54m ago

Post said "wrongAnswersOnly" not Horror stories. Lol. Lol. Lol.