add a comment to warn that it is not secure

Open YYTVicky opened this issue 5 years ago • 3 comments

Mar 01 '20 02:03 YYTVicky

Hi,

could I know why my PR didn't pass the checks, the PR is a comment. Our tool report that keeps checkClientTrusted and checkServerTrusted empty body is not secure.

Thanks!

Apr 30 '20 01:04 YYTVicky

Hi @YYTVicky ,

Thank you for this PR!

I think that was an issue with the build with all pull requests failing at that time. Can you please rebase your PR on master HEAD? The "Update branch" button should be sufficient.

Apr 30 '20 17:04 bessbd

When I try to apply this patch I am getting merge conflicts. I don't there actually are any, just that the fork needs to be brought up to date. Can you please do this?

Oct 08 '22 04:10 rgoers