Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependencies #46

Closed
wants to merge 5 commits into from

Conversation

stusmall
Copy link

PR Checklist:

  • I have run npm test locally and all tests are passing.
  • I have added/updated tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

The keys were generated with 1024 bit keys.  Depending on the system
configuration this can be rejected as unsafe causing the test to fail.
This updates the config to use 4096 keys and regenerates all key
material
This doesn't update all dependencies or get everything to the newest
versions.  The goal was to eliminate current critical and high vulens
in the tree.  There are still a few mediums kicking around that could be
squashed too.  I also opportunistically bump other versions when
possible
These keys predate the ca folder and update script contained in there.
I udpated the tests using them to use one of the keys managed by the
update script and removed them
@cypress-app-bot
Copy link

@stusmall stusmall closed this Oct 11, 2023
@stusmall
Copy link
Author

Sorry about this! Didn't mean to open a second PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants