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

If TLSEnable is false, ssl_context is not initialized #7

Closed
bjerickson opened this issue Feb 2, 2017 · 2 comments
Closed

If TLSEnable is false, ssl_context is not initialized #7

bjerickson opened this issue Feb 2, 2017 · 2 comments

Comments

@bjerickson
Copy link
Contributor

[2017-02-02 11:41:57] Unhandled python exception in read callback: UnboundLocalError: local variable 'ssl_context' referenced before assignment
[2017-02-02 11:41:57] Traceback (most recent call last):
[2017-02-02 11:41:57]   File "/opt/collectd-aerospike/aerospike_plugin.py", line 755, in read
    capath=ca_path, protocols=protocols, cert_blacklist=blacklist, crl_check=crl_check, crl_check_all=crl_check_all, tls_name=tls_name)
[2017-02-02 11:41:57]   File "/opt/collectd-aerospike/aerospike_plugin.py", line 230, in connect
bjerickson pushed a commit to bjerickson/aerospike-collectd that referenced this issue Feb 2, 2017
initialize ssl_context as false
@bjerickson
Copy link
Contributor Author

TLSEnable in collectd, tls_enable in the python code

bjerickson pushed a commit to bjerickson/aerospike-collectd that referenced this issue Feb 2, 2017
Instead of using False, set ssl_context as None to clarify that ssl_context is an object
@whyrg
Copy link
Contributor

whyrg commented Feb 2, 2017

Don't see the issue:

aerospike_plugin.py: line 660
elif node.key == 'TLSEnable': self.tls_enable = node.values[0]

@whyrg whyrg closed this as completed in 490332b Feb 2, 2017
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

No branches or pull requests

2 participants