delta/node-new.git/src/node_crypto_clienthello.h, branch crypto_set_integer_macro

crypto: move field initialization to class

2018-10-15T00:45:54+00:00

PR-URL: https://github.com/nodejs/node/pull/23610
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen 
Reviewed-By: Refael Ackermann 
Reviewed-By: Colin Ihrig 
Reviewed-By: Сковорода Никита Андреевич 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Ujjwal Sharma

crypto: remove unused header in clienthello.h

2017-12-19T13:36:55+00:00

This commit removes stdlib.h header as it does not seem to be used any
more.

PR-URL: https://github.com/nodejs/node/pull/17752
Reviewed-By: Gireesh Punathil 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen

src: fix -Wundefined-inline warnings

2017-12-15T06:47:42+00:00

Fix compiler warnings introduced in commit 47edfd9c3c ("crypto: move
node_crypto_clienthello-inl.h to cc") by moving constructors around.

They were defined inline but depended on definitions
from `node_crypto_clienthello-inl.h`.

PR-URL: https://github.com/nodejs/node/pull/17649
Reviewed-By: Colin Ihrig 
Reviewed-By: Daniel Bevenius 
Reviewed-By: James M Snell 
Reviewed-By: Timothy Gu 
Reviewed-By: Eugene Ostroukhov 
Reviewed-By: Michael Dawson

src: remove unused include node_crypto_clienthello

2017-12-11T12:01:07+00:00

This commit removes the include of node.h which does not seem to be
used.

PR-URL: https://github.com/nodejs/node/pull/17546
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig

src: move crypto_bio/clienthello to crypto ns

2017-06-30T05:23:25+00:00

Currently, node_crypto_bio and node_crypto_clienthello are not in the
crypto namespace but simply in the node namespace. Not sure if this was
intentional or not, but I think it would make sense to move them to be
consistent.

PR-URL: https://github.com/nodejs/node/pull/13957
Reviewed-By: Refael Ackermann 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell

meta: restore original copyright header

2017-03-10T19:23:48+00:00

A prior io.js era commit inappropriately removed the
original copyright statements from the source. This
restores those in any files still remaining from that
edit.

Ref: https://github.com/nodejs/TSC/issues/174
Ref: https://github.com/nodejs/node/pull/10599
PR-URL: https://github.com/nodejs/node/pull/10155

Note: This PR was required, reviewed-by and approved
by the Node.js Foundation Legal Committee and the TSC.
There is no `Approved-By:` meta data.

crypto: fix undefined behavior in ParseExtension

2016-07-11T06:23:28+00:00

Many extensions are unknown to the `ClientHelloParser::ParseExtension`,
do not cast user-supplied `uint16_t` to `enum`.

PR-URL: https://github.com/nodejs/node/pull/7494
Reviewed-By: Ben Noordhuis

src: add include guards to internal headers

2016-05-25T07:54:24+00:00

For consistency with the newly added src/base64.h header, check that
NODE_WANT_INTERNALS is defined and set in internal headers.

PR-URL: https://github.com/nodejs/node/pull/6948
Refs: https://github.com/nodejs/node/pull/6910
Reviewed-By: Fedor Indutny 
Reviewed-By: James M Snell 
Reviewed-By: Trevor Norris

Remove excessive copyright/license boilerplate

2015-01-12T23:30:28+00:00

The copyright and license notice is already in the LICENSE file.  There
is no justifiable reason to also require that it be included in every
file, since the individual files are not individually distributed except
as part of the entire package.

build,src: remove sslv2 support

2015-01-11T15:07:45+00:00

SSLv2 has been deprecated and known broken for nearly twenty years now.

I made SSLv2 support opt-in well over a year ago in commit 39aa894 and
now this commit removes it entirely.

PR-URL: https://github.com/iojs/io.js/pull/290
Reviewed-By: Fedor Indutny 
Reviewed-By: Rod Vagg