Skip to content

Conversation

@artalar
Copy link
Contributor

@artalar artalar commented Jan 11, 2022

You can test live reproduce here https://stackblitz.com/edit/nanoid-collision?file=index.js (type node index.js in the console)

@artalar artalar force-pushed the proxy-number-collision branch from 398fd90 to 397c549 Compare January 11, 2022 13:35
@ai ai merged commit 2b7bd93 into ai:main Jan 11, 2022
@ai
Copy link
Owner

ai commented Jan 11, 2022

Thanks. Fixed was released in 3.1.31.

feynmanliang pushed a commit to feynmanliang/beanmachine that referenced this pull request Apr 5, 2022
…arch#1387)

Summary:
### Motivation

Fix vulnerability described in  ai/nanoid#328

Note `docusaurus-plugin-internaldocs-fb` doesn't pin its hard docusaurus version dependency and facebook/docusaurus#6287 introduced a breaking change, so we need to pin docusaurus version to keep the build green.

### Changes proposed

Upgrades pinned JS dep versions from running `yarn upgrade`

Pull Request resolved: facebookresearch#1387

Test Plan:
`yarn build`

### Types of changes
- [x] Docs change / refactoring / dependency upgrade
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

### Checklist
- [ ] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [x] I have read the **[CONTRIBUTING](https://github.com/facebookresearch/beanmachine/blob/main/CONTRIBUTING.md)** document.
- [ ] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
- [x] The title of my pull request is a short description of the requested changes.

**Static Docs Preview: beanmachine**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D35151907/V5/beanmachine/)|

|**Modified Pages**|

Reviewed By: horizon-blue

Differential Revision: D35151907

Pulled By: feynmanliang

fbshipit-source-id: 051b702c83a482fb3d3dacbd8bb3ebe9a0109af3
facebook-github-bot pushed a commit to facebookresearch/beanmachine that referenced this pull request Apr 5, 2022
Summary:
### Motivation

Fix vulnerability described in  ai/nanoid#328

Note `docusaurus-plugin-internaldocs-fb` doesn't pin its hard docusaurus version dependency and facebook/docusaurus#6287 introduced a breaking change, so we need to pin docusaurus version to keep the build green.

### Changes proposed

Upgrades pinned JS dep versions from running `yarn upgrade`

Pull Request resolved: #1387

Test Plan:
`yarn build`

### Types of changes
- [x] Docs change / refactoring / dependency upgrade
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

### Checklist
- [ ] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [x] I have read the **[CONTRIBUTING](https://github.com/facebookresearch/beanmachine/blob/main/CONTRIBUTING.md)** document.
- [ ] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
- [x] The title of my pull request is a short description of the requested changes.

**Static Docs Preview: beanmachine**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D35151907/V5/beanmachine/)|

|**Modified Pages**|

Reviewed By: horizon-blue

Differential Revision: D35151907

Pulled By: feynmanliang

fbshipit-source-id: b4274c7fc713fccddd1d1c64c9024138107a5716
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.

2 participants