FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: require redis v3 by compwright · Pull Request #546 · bee-queue/bee-queue · GitHub

fix: require redis v3 - #546

Merged
compwright merged 2 commits into
bee-queue:masterfrom
compwright:redis-3
Nov 22, 2022
Merged

fix: require redis v3#546
compwright merged 2 commits into
bee-queue:masterfrom
compwright:redis-3

Conversation

Copy link
Copy Markdown
Collaborator

Resubmitting #451 from @b-jam to restart CI checks:

As previously discussed in #257 In the interest of security, to utilisize rediss over TLS. I noticed dependabot was also complaining about redis too.

It seemed like no progress is being made towards bee-queue 2.0 or using IOredis so this is necessary.

Node 4 support needs to be dropped as a result, but that was EOL in 2018 (Node 8 is also EOL as is 10) https://endoflife.software/programming-languages/server-side-scripting/nodejs

compwright mentioned this pull request Nov 22, 2022
compwright force-pushed the redis-3 branch 2 times, most recently from 450149d to b803129 Compare November 22, 2022 17:35
compwright merged commit 445521b into bee-queue:master Nov 22, 2022
beequeueci pushed a commit that referenced this pull request Nov 22, 2022
## [1.4.2](v1.4.1...v1.4.2) (2022-11-22)

### Bug Fixes

* require redis v3 ([#546](#546)) ([445521b](445521b))

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 1.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL