Hacker Newsnew | past | comments | ask | show | jobs light | darkhn

This tool looks like it unconditionally disables tls verification for upstream requests.

It shells out to mitmproxy with "--set", "ssl_insecure=true"

This took all of 5 minutes to find reading through main.py on my phone.

https://github.com/jmuncor/sherlock/blob/fb76605fabbda351828...

Edit: In case it’s not clear, you should not use this.

reply on default site

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Ah, getting the job done by disabling important validation, if that isn't the most prominent Opus trait...

I wonder how much this will end up costing the industry in aggregate.


Not entirely different from many human engineers...

Just fixed it and implemented a simple http relay, eliminating the mitmproxy and the ssl_insecure=true. The new implementation uses TLS verification, doing last tests and merging it... After the merge can you check it out and tell me if I earned your star? :D

I’m not sure you fully understand the implications of the misconfiguration of mitmproxy there. Effectively you provided an easily accessible front door for remote code execution on a user’s machine.

No offense, but I wouldn’t trust anything else you published.

I think it’s great that you are learning and it is difficult to put yourself out there and publish code, but what you originally wrote had serious implications and could have caused real harm to users.


Ohh my, no offense taken... The next time I will be a lot more careful with the stuff that I put out there. Learning and getting the hang of it, would love if you either comment on the code or here any other things you think could be improved. I am in the process of getting better and appreciate all the blunt and transparent feedback. No one grows out of praise.

No, you’re in the process of vibe coding stuff you don’t understand and you will most likely never understand until you take the time to open a book.

Your comment contains nothing but insults.

This is not a place for you to try and make yourself feel better by disparaging others.


>tell me if I earned your star

Since you asked: Not in a million years, no.

A bug of this type is either an honest typo or a sign that the author(s) don't take security seriously. Even if it were a typo, any serious author would've put a large FIXME right there when adding that line disabling verification. I know I would. In any case a huge red flag for a mitm tool.

Seeing that it's vibe coded leads me believe it's due to AI slop, not a simple typo from debugging.


I love the real feedback tbh, I am still learning, and want to learn as much as possible. Would love if you can review it and tell me bluntly either in the repo or here the things that should be improved. I would love to learn more from you and get better :D

I'm not going to review it in full, sorry. Reviewing is so much more effort compared to producing something with AI. But don't let me deter you, keep on learning and keep on building.

I wish I had the possibilities to learn and build on such a large scale when I started out. AI is a blessing and a curse I guess.

My own early projects were most definitely crap, and I made the exact same mistakes in the past. Honestly my first attempts were surely worse. But my projects were also tiny and incomplete, so I never published them.

However: What little parts I did publish as open-source or PRs were meticulously reviewed before ever hitting send, and I knew these inside and out and they were as good as I could make it.

Vibe-coded software is complete but never as good as you could make it, so the effort in reviewing it is mostly wasted.

I guess what I'm trying to say is I'm a bit tired of seeing student-level projects on HN / Github cosplaying as production ready software built by an experienced engineer. It used to be possible to distinguish these from the README or other cues, but nowadays they all look professional and are unintentionally polluting the software space when I'm actually looking for something.

Please understand that this is not specifically directed at you, it's pent up frustration from reading HN projects over the last months. Old guy yelling at clouds.


The README is really annoying.

You used to be able to tell so easily what was a good well looked after repo by viewing the effort and detail that had gone into the README.

Now it's too easy to slop up a README.


I appreciate that attitude. Keep it up.

unlikely to get that from a throwaway

You can always try right?

Only if you don’t care about your reputation.

“Give me your time for free” is not the kind of request that earns respect.


You don't understand what you're doing, and never will. Throw away all computing devices you've got.



Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact |

Search: