Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
Touchpad1117yWhen there's more things happening in the function declaration than in the function you know something is wrong
-
@2lazy2debug beat me to it,
that function is usually found inside debug code, the part of code using this function should at least be surrounded by #if DEBUG
so that when the code is compiled for production that doesn't get included and certificate validate remains untouched -
@2lazy2debug it is just for safety, cuz under pressure sometimes warnings get slipped away and go unnoticed
-
@nzeetee would be fun, but their security overall is so bad they don't event use Auth on the server part. so I would not want too many eyes on them before we switch provider.
-
gitlog57636yAnd then, there are instances that these happen due to laziness of enclosing statements under a block and copy pasting
More: www.gotofail.com -
Meta33346ySometimes you find something like that in methods that *should* be overridden in stuff like abstract classes, but in this case that's just wrong.
Related Rants
Haha 😂, actual code from 3rd party product we use:
rant
security
c#
certificate