Skip to content

Commit db91a77

Browse files
wraithgarlukekarrys
authored andcommitted
fix: remove "hashAlgorithm" from flatOptions
BREAKING CHANGE: the hard-coded "hashAlgorithm" value is no longer being passed through flatOptions Nothing in npm is using this.
1 parent ece52a3 commit db91a77

File tree

2 files changed

+0
-5
lines changed

2 files changed

+0
-5
lines changed

workspaces/config/lib/definitions/index.js

-3
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ const flatten = (obj, flat = {}) => {
2525
: /* istanbul ignore next - not configurable property */ undefined
2626
flat.nodeBin = process.env.NODE || process.execPath
2727

28-
// XXX should this be sha512? is it even relevant?
29-
flat.hashAlgorithm = 'sha1'
30-
3128
return flat
3229
}
3330

workspaces/config/test/definitions/index.js

-2
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ t.test('flatten', t => {
3636
'//foo.bar.com:_authToken': 'foobarbazquuxasdf',
3737
npmBin: '/path/to/npm',
3838
nodeBin: '/path/to/node',
39-
hashAlgorithm: 'sha1',
4039
})
4140

4241
mockGlobals(t, {
@@ -52,7 +51,6 @@ t.test('flatten', t => {
5251
'//foo.bar.com:_authToken': 'foobarbazquuxasdf',
5352
npmBin: '/path/to/npm',
5453
nodeBin: '/usr/local/bin/node.exe',
55-
hashAlgorithm: 'sha1',
5654
})
5755

5856
t.end()

0 commit comments

Comments
 (0)