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

[Snyk] Upgrade react-redux from 7.1.3 to 7.2.0 #121

Merged
merged 1 commit into from
Mar 11, 2020

Conversation

snyk-bot
Copy link
Contributor

@snyk-bot snyk-bot commented Mar 8, 2020

Snyk has created this PR to upgrade react-redux from 7.1.3 to 7.2.0.

ℹ️ Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.
  • The recommended version is 1 version ahead of your current version.
  • The recommended version was released 19 days ago, on 2020-02-18.
Release notes
Package name: react-redux
  • 7.2.0 - 2020-02-18

    This release fixes two bugs, an algorithmic problem with unsubscribing components and a memory leak with connect. It also has optimizations for production bundle size, and adds a couple small improvements to developer readability while debugging.

    Changes

    Bug Fixes

    connect in v7 is implemented using hooks, and the hooks usage captures numerous values from the surrounding scope. We received a PR informing us that the way we were capturing these values would likely result in a copy of the first version of its props being kept alive indefinitely.

    This memory leak has been fixed by extracting a custom hook that receives all the necessary values as arguments, so that they're not captured via closure.

    We also received a PR letting us know that the unsubscribe logic had a quadratic algorithm in it, as removing a subscriber would use an indexOf(listener) check to remove that callback. If there were a large number of subscribers, that line's runtime would increase rapidly, causing slowdowns.

    This algorithm has been replaced with tracking subscribers via a linked list, which drastically improves the runtime of this section of the code even with large numbers of subscribers.

    Thanks to @larrylin28 and @wurstbonbon for finding these bugs and submitting PRs to fix them!

    Bundle Size Improvements

    We've made a number of small tweaks to the codebase to improve the ability of bundlers to shake and minimize the final included size in a bundle. The net result is that react-redux@7.2.0 is smaller than 7.1.3, dropping 1.3K min and 0.6K min+gzip. (In fact, it's even smaller than the pre-hooks 7.0.0 when gzipped!)

    Thanks to @Andarist for doing most of the work on this!

    Debugging Improvements

    The ReactReduxContext instance now has a displayName set, so it should show up in the React DevTools as ReactRedux.Provider.

    Also, when an error is caught in useSelector and re-thrown, we now append the original stack trace.

    Thanks to @pieplu and @r3dm1ke for these!

    Changelog

    v7.1.3...v7.2.0

  • 7.1.3 - 2019-11-06

    Forgot to remove a console statement before I published 7.1.2. Oops!

    Lint your source code before publishing, folks.

    Changes

from react-redux GitHub release notes
Commit messages
Package name: react-redux
  • ec38c1b 7.2.0
  • fa5a7fd Fix memory leak issue with `UseEffect` (#1506)
  • e649fb6 Optimize createListenerCollection (#1523)
  • 3eb5271 Updated links (#1522)
  • b3db96c docs: mapDispatchToProps returning undefined (#1520)
  • 607f1ba Fix: Fix typo in connect.md (#1517)
  • 77a2044 Remove these spaces.
  • cff554d Update connect-dispatching-actions-with-mapDispatchToProps.md (#1501)
  • 9a10b15 Update hooks.md (#1405)
  • 7893b35 Fixed Q&A links (#1500)
  • b3ae892 Fixes broken links (#1494)
  • f9efe46 fix: correct link to Redux Troubleshooting page (#1493)
  • 863128e Fix Usage with React link.
  • 19359fb remove useless semicolon (#1486)
  • 67d5e86 Add Netlify Badge to Docs (#1485)
  • 66b69ec another weird semi colon in docs (#1484)
  • db35c54 unnecessary extra semi colon in docs (#1483)
  • 5bef286 fix broken link in hooks.md (#1465)
  • 24354a6 Fix code example in static-types.md (#1476)
  • 5402f24 Rethrowing original errors thrown in selector functions (#1474)
  • b198f26 Add links to further TS resources
  • f059ad0 WIP: Static Types doc page (#1439)
  • 1b39cdd Pass @babel/runtime version to @babel/plugin-transform-runtime (#1473)
  • 9ebd1a9 Remove invariant in favor of doing NODE_ENV checks directly (#1472)

Compare


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.

For more information:

🧐 View latest project report

🛠 Adjust upgrade PR settings

🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@ryanhefner ryanhefner added this to To Do in Open Sourcery 🔮 Mar 9, 2020
@ryanhefner ryanhefner merged commit a975327 into master Mar 11, 2020
Open Sourcery 🔮 automation moved this from To Do to Done Mar 11, 2020
@ryanhefner ryanhefner deleted the snyk-upgrade-2e5cdfdf18783aaa3276d429e59e5f55 branch March 11, 2020 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants