Actually remove the algorithmParams when encrypting

PeculiarVentures · Oct 28, 2021 · 2f8934c · 2f8934c
1 parent 03cc200
commit 2f8934c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/.gitignore b/.gitignore
@@ -7,3 +7,6 @@ coverage
 .vs
 .vscode
 docs
+examples/*/bundle.js
+test/s_*.js
+test/browser/*.js
diff --git a/src/EnvelopedData.js b/src/EnvelopedData.js
@@ -942,13 +942,7 @@ export default class EnvelopedData
 					//region Create and encode "ECC-CMS-SharedInfo" structure
 					const eccInfo = new ECCCMSSharedInfo({
 						keyInfo: new AlgorithmIdentifier({
-							algorithmId: aesKWAlgorithm.algorithmId,
-							/*
-							 Initially RFC5753 says that AES algorithms have absent parameters.
-							 But since early implementations all put NULL here. Thus, in order to be
-							 "backward compatible", index also put NULL here.
-							 */
-							algorithmParams: new asn1js.Null()
+							algorithmId: aesKWAlgorithm.algorithmId
 						}),
 						entityUInfo: recipientInfo.value.ukm,
 						suppPubInfo: new asn1js.OctetString({ valueHex: kwLengthBuffer })