Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added parameter to change the type of encryption used #263

Merged
merged 3 commits into from
Jul 28, 2021
Merged

Added parameter to change the type of encryption used #263

merged 3 commits into from
Jul 28, 2021

Conversation

felipehertzer
Copy link
Contributor

I was having problems to perform the handshake with some servers because it is using 384bit encryption, so I found a type that solves my problem the "secp384r1". I added the possibility for the user to choose the best algorithm for each use.

The main problem I had was handshake errors like:
(Caused by SSLError(SSLError(1, '[SSL: SSLV3_ALERT_HANDSHAKE_FAILURE]) sslv3 alert handshake failure (_ssl.c:1108)'))

@VeNoMouS
Copy link
Owner

Hello sir @felipehertzer just have tested it. The repo look patched can't solve cloudflare challenge (v1)
Also the cookie ''cfduid'' has beem removed since 2 month ago.
Do u can fix it to work on new challenge ?

Thanks

image

@VeNoMouS
Copy link
Owner

VeNoMouS commented Jul 28, 2021

@felipehertzer while your're on the right track, your also WAY off ... this will only work for specific sites with specific settings setup... additionally this will cause you problems if your abusing cloudflare BFM...

I have no intention explaining more on this subject btw.

Also if you want me to implement this.... i suggest not doing a PR to the master branch... that is what the dev branch is for.

@felipehertzer felipehertzer changed the base branch from master to dev July 28, 2021 09:38
@felipehertzer
Copy link
Contributor Author

felipehertzer commented Jul 28, 2021

Sorry for the branch. I changed my script to brain… seemed to work correctly so far, but I don't know if you know if this might not work.

@VeNoMouS VeNoMouS merged commit d565f02 into VeNoMouS:dev Jul 28, 2021
@SpangleLabs
Copy link

Also if you want me to implement this.... i suggest not doing a PR to the master branch... that is what the dev branch is for.

Seems like you have to correct people often on that, might be worth adding a little contributors file which explains that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants