Use latest cryptography

NginxProxyManager / docker-nginx-full

Docker image with compiled Nginx (OpenResty) and OpenSSL with all the stock Nginx plugins enabled.

72 stars 60 forks source link

Use latest cryptography #16

Open dormancygrace opened 1 year ago

dormancygrace commented 1 year ago

I've built npm with (amd64 and arm64) with cryptography 38, all work as intended. Is there any reason, why we should stay on 2.8?

nginxproxymanagerci[bot] commented 1 year ago

Docker Image for build 1 is available on DockerHub as:

nginxproxymanager/nginx-full:github-pr-16
nginxproxymanager/nginx-full:github-pr-16-certbot
nginxproxymanager/nginx-full:github-pr-16-certbot-node
nginxproxymanager/nginx-full:github-pr-16-acmesh
nginxproxymanager/nginx-full:github-pr-16-acmesh-golang

jc21 commented 1 year ago

Yes there was, it was related to the certbot plugins. Most of them (at the time) were failing because of the crypto package. This might not be an issue anymore, but it would need to be tested at length with the nginx-proxy-manager project