Skip to content

Commit

Permalink
doc: fix http.IncomingMessage.socket documentation
Browse files Browse the repository at this point in the history
Remove the reference to net.Socket.verifyPeer().
That was removed in ea540c9 and was missed in the 032f80e.

Refer to the net.Socket instance by the .socket property.
This avoids unneeded confusion.
'.socket' is the variant that is used internally.

Add a markdown link to net.Socket.getPeerCertificate().

PR-URL: nodejs/node#1867
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
ChALkeR authored and andrewdeandrade committed Jun 3, 2015
1 parent f7d9a47 commit e8ac882
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions doc/api/http.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -1068,9 +1068,8 @@ The HTTP response status message (reason phrase). E.G. `OK` or `Internal Server

The `net.Socket` object associated with the connection.

With HTTPS support, use request.connection.verifyPeer() and
request.connection.getPeerCertificate() to obtain the client's
authentication details.
With HTTPS support, use [request.socket.getPeerCertificate()][] to obtain the
client's authentication details.


['checkContinue']: #http_event_checkcontinue
Expand Down Expand Up @@ -1098,5 +1097,6 @@ authentication details.
[socket.setKeepAlive()]: net.html#net_socket_setkeepalive_enable_initialdelay
[socket.setNoDelay()]: net.html#net_socket_setnodelay_nodelay
[socket.setTimeout()]: net.html#net_socket_settimeout_timeout_callback
[request.socket.getPeerCertificate()]: tls.html#tls_tlssocket_getpeercertificate_detailed
[stream.setEncoding()]: stream.html#stream_stream_setencoding_encoding
[url.parse()]: url.html#url_url_parse_urlstr_parsequerystring_slashesdenotehost

0 comments on commit e8ac882

Please sign in to comment.