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 HAS_ZLIB in run_crime(), declaration of CERT_COMPRESSION fixed #1282

Merged
merged 1 commit into from Jun 12, 2019

Conversation

Projects
None yet
1 participant
@drwetter
Copy link
Owner

commented Jun 12, 2019

CERT_COMPRESSION was initialized with false in 912cd90 so so that the variable was always false. This PR fixes that. In addition a informational line
that the new TLS extension has been added (if $DEBUG >3).

Also determine_optimal_proto() is not being run if devel mode ($do_tls_sockets) is enabled.

Furthermore as David added HAS_ZLIB in #1279 as a global, run_crime() now makes use of it too.

Added HAS_ZLIB in run_crime(), declaration of CERT_COMPRESSION fixed
CERT_COMPRESSION was declared always with fast in ... so that the variable
was always false. This PR fixes that. In addition a informational line
that the new TLS extension has been added (if $DEBUG >3).

Also determine_optimal_proto() is not being run if devel mode
($do_tls_sockets) is enabled.

Furthermore as David added HAS_ZLIB as a global run_crime() now makes
use of it too.

@drwetter drwetter merged commit 6e4abbf into 3.0 Jun 12, 2019

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@drwetter drwetter deleted the 1279_related branch Jun 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.