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

fix(sec): upgrade streamlit to 1.11.1 #4040

Merged

Conversation

chncaption
Copy link
Contributor

What happened?

There are 1 security vulnerabilities found in streamlit 1.9.0

What did I do?

Upgrade streamlit from 1.9.0 to 1.11.1 for vulnerability fix

What did you expect to happen?

Ideally, no insecure libs should be used.

The specification of the pull request

PR Specification from OSCS

@paddle-bot
Copy link

paddle-bot bot commented Dec 7, 2022

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Dec 7, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@sijunhe sijunhe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chncaption Thank you for the contribution! Would you sign the license agreement so we can potentially merge this PR?

@chncaption
Copy link
Contributor Author

@chncaption Thank you for the contribution! Would you sign the license agreement so we can potentially merge this PR?

👌

@sijunhe sijunhe requested a review from w5688414 December 8, 2022 08:07
@sijunhe
Copy link
Collaborator

sijunhe commented Dec 8, 2022

@w5688414 would you plz review?

@w5688414
Copy link
Contributor

w5688414 commented Dec 9, 2022

I need to test my ui files first, it may take some time

Copy link
Contributor

@w5688414 w5688414 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sijunhe
Copy link
Collaborator

sijunhe commented Dec 30, 2022

@wawltor 这个PR有点老,好像statuses有点问题,麻烦force merge一下

@wawltor wawltor merged commit 2e9dfce into PaddlePaddle:develop Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants