cocagne/zed

Thin wrapper for integrating ZeroMQ sockets into the Twisted reactor

zed's Issues

`identity` property does not work with string datatype in Python 3.

After upgrade to python 3, setting a string value to identity property raises an exception. This worked perfectly fine in Python 2.

Sample code:
socket.identity = 'mysocket'

Traceback:
lambda zs,v: zs._zsock.setsockopt(i,totype(v)) ) File "zmq/backend/cython/socket.pyx", line 413, in zmq.backend.cython.socket.Socket.set TypeError: unicode not allowed, use setsockopt_string

It looks like we need to call setsockopt_string instead of setsockopt for str datatypes.

This fix should also be applicable to other string properties in this class - for e.g. socket.subscribe.