pycoin icon indicating copy to clipboard operation
pycoin copied to clipboard

openssl implementation not working on osx and windows

Open littleskunk opened this issue 8 years ago • 2 comments

https://github.com/richardkiss/pycoin/blob/master/pycoin/ecdsa/native/library.py#L21

This line will only work for linux and doesn't support windows and osx and it should throw a exception if the env variable is set but the library can't be loaded. At the moment i set the env variable and i get no feedback.

I would prefer this implementation: https://github.com/uwdata/termite-stm/blob/master/web2py/gluon/contrib/pbkdf2_ctypes.py#L122

Tested on windows. I will create a pull request later.

littleskunk avatar Mar 01 '16 13:03 littleskunk

Added windows and osx support. I did not add a exception message. I think it is needed but that is your decision.

littleskunk avatar Mar 01 '16 13:03 littleskunk

This line will also fail in the official Python docker image python:3.6-alpine. (See issue).

The implementation of find_library is a hack. It calls ldconfig -p and parses the output, hoping that the library is in the cache. As a fallback it calls gcc to find the library. This is obviously very fragile.

recmo avatar Nov 17 '16 16:11 recmo