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

Hostname/IP doesn't match certificate's altnames: "Cert is empty", when subject is empty #47

Closed
evilfer opened this issue Jun 19, 2017 · 2 comments

Comments

@evilfer
Copy link

evilfer commented Jun 19, 2017

Hi,
We have a node app using node-ldapauth-fork (through passport-ldapauth) for authentication. The LDAP certificate recently changed, and the new one doesn't have a "subject", although it has an "altName". This appears to be causing our app to reject the LDAP server.

The problem may be this reported node bug: nodejs/node#11771

So possibly it has nothing to do with node-ldapauth-fork. However, I wanted to ask whether you're seeing this error with similar certificates, and whether there's a workaround for this.

@evilfer evilfer changed the title Hostname/IP doesn't match certificate's altnames: "Cert is empty", with subject is empty Hostname/IP doesn't match certificate's altnames: "Cert is empty", when subject is empty Jun 19, 2017
@vesse
Copy link
Owner

vesse commented Jun 21, 2017

Sorry, I don't have experience on this nor do I know if there's a workaround. The tlsOptions are passed directly through to ldapjs which passes them to tls.connect. If none of the TLS options in Node.js work, then I doubt there's much you could do.

@evilfer
Copy link
Author

evilfer commented Jun 22, 2017

No problem, thank you for confirming that this issue is not related to this package. I'll close this now and keep an eye on the node reported bug :).

@evilfer evilfer closed this as completed Jun 22, 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