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

OpenSSL Doc Links #9327

Closed
kobelb opened this issue Oct 27, 2016 · 2 comments
Closed

OpenSSL Doc Links #9327

kobelb opened this issue Oct 27, 2016 · 2 comments
Labels
doc Issues and PRs related to the documentations. openssl Issues and PRs related to the OpenSSL dependency.

Comments

@kobelb
Copy link
Contributor

kobelb commented Oct 27, 2016

The following pull-request moved the OpenSSL docs breaking a lot of links in the node docs: openssl/openssl#1758

There are now separate pages for different versions and master. For example https://www.openssl.org/docs/apps/ca.html is no longer accessible, but it's available at
https://www.openssl.org/docs/man1.0.2/apps/ca.html and https://www.openssl.org/docs/manmaster/man1/ca.html

I was working on a pull-request to address #9025 when I came across this. I'm willing to go through and fix the existing documentation, but I was wondering whether you all would prefer the links be changed to 1.0.2 or master?

@sam-github
Copy link
Contributor

Should be changed to version of OpenSSL we use in node, I think.

@mscdex mscdex added doc Issues and PRs related to the documentations. openssl Issues and PRs related to the OpenSSL dependency. labels Oct 27, 2016
@targos
Copy link
Member

targos commented Jan 8, 2017

Fixed in #9338

@targos targos closed this as completed Jan 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. openssl Issues and PRs related to the OpenSSL dependency.
Projects
None yet
Development

No branches or pull requests

4 participants