node-red / node-red-nodes

Extra nodes for Node-RED
Other
964 stars 592 forks source link

simpleParser takes an optional parameter checksumAlgo. Specify the … #1050

Closed JaysonHurst closed 4 months ago

JaysonHurst commented 4 months ago

…FIPS compliant sha256 hash algorithm here so that it will be used instead of the default of md5.

node-red: v3.1.5 node: v18.18.2 OS: Rocky Linux 9.3. npm v9.8.1 node-red-node-email v2.1.0

Proposed changes

Allows node-red-node-email node to correctly work when FIPS crypto policy has been enabled on the operating system by setting the checksumAlgo for simpleParser to SHA256 which is FIPS compliant.

Checklist

linux-foundation-easycla[bot] commented 4 months ago

CLA Signed

The committers listed above are authorized under a signed CLA.

Steve-Mcl commented 4 months ago

@JaysonHurst there is an issue with your commit. See the above comment regarding EasyCLA. Once the EasyCLA is sorted & signed the PR can run tests and be reviewed.

hardillb commented 4 months ago

@JaysonHurst commits must be made with an email address that is linked to your GitHub Account for the CLA tool to match. We can't merge the change until the CLA tool passes or we can re-implement the change.

You can either add the email address linked to the commit to your GH account (email address can be seen here or change the email address linked to the commit to one already linked to your GH account and force push to your local branch.