peppelinux / pymultildap Goto Github PK

11.0 3.0 1.0 52 KB

OpenLDAP proxy or simple python3 LDAP client to handle multiple LDAP connections, data aggregation and manipulation strategies

License: Other

Python 100.00%

ldap3 json ldap-client gevent slapd-sock openldap slapd rewrite-rules unix-domain-socket backend

pymultildap's Issues

AttributeError: module 'gevent' has no attribute 'signal'

Hi,

I tried this package but I've got an error:
multildapd.py -conf setting.py -port 1234

[2021-07-14 06:35:31,182] [INFO] multildapd: Process run on pid: 322
Traceback (most recent call last):
  File "/usr/local/bin/multildapd.py", line 259, in <module>
    gevent.signal(signal.SIGTERM, stop_app, **{'pidfile': pidfile,
AttributeError: module 'gevent' has no attribute 'signal'

Steps to reproduce:

I use docker (I tried local mashine too but same result - Ubuntu Python 3.8) docker run -it python:3.6 /bin/bash
pip3 install proxyldap
create setting.py from example
multildapd.py -conf setting.py -port 1234
this return error:

[2021-07-14 06:35:31,182] [INFO] multildapd: Process run on pid: 322
Traceback (most recent call last):
  File "/usr/local/bin/multildapd.py", line 259, in <module>
    gevent.signal(signal.SIGTERM, stop_app, **{'pidfile': pidfile,
AttributeError: module 'gevent' has no attribute 'signal'

Did I something wrong? Which Python version should I use? Thanks.

Fanda

Possible bug in _decode_elements

pyMultiLDAP/client.py

Lines 45 to 47 in dd8dcbb

    
           def _decode_elements(self, attr_dict): 
        
               return {k:[e.decode(self.conf['encoding'] if isinstance(e, bytes) else e) for e in v] 
        
                       for k,v in attr_dict.items() }

This seems wrong:

e.decode(self.conf['encoding'] if isinstance(e, bytes) else e)

I am guessing what you wanted was:

e.decode(self.conf['encoding']) if isinstance(e, bytes) else e

progression on this:

use proper names
accumulate results

result = {
    attr_name: decoded_vals
    for attr_name, attr_vals in attr_dict.items()
    for decoded_vals in [[
        decoded_val
        for attr_val in attr_vals
        for decoded_val in [
            attr_val.decode(self.conf['encoding'])
            if isinstance(attr_val, bytes)
            else attr_val
        ]
    ]]
}

Nesting cannot be avoided; it follows the structure of the data. What you can do is split the different things you do in separate functions.

def _decode_elements(self, attr_dict):
    def decode_attr_val(attr_val, encoding):
        value = (
            attr_val.decode(encoding)
            if isinstance(attr_val, bytes)
            else attr_val
        )
        return value

    def decode_attr_vals(attr_vals, encoding):
        values = [decode_attr_val(attr_val, encoding) for attr_val in attr_vals]
        return values

    encoding = self.conf['encoding']
    result = {
        attr_name: decode_attr_vals(attr_vals, encoding)
        for attr_name, attr_vals in attr_dict.items()
    }
    return result

peppelinux / pymultildap Goto Github PK

pymultildap's Issues

AttributeError: module 'gevent' has no attribute 'signal'

Possible bug in _decode_elements

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	def _decode_elements(self, attr_dict):
	return {k:[e.decode(self.conf['encoding'] if isinstance(e, bytes) else e) for e in v]
	for k,v in attr_dict.items() }