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.

Sign up for GitHub

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

Jump to bottom

[Security] fix security problem for _wget_download #61321

Closed
wants to merge 2 commits into from
Closed

[Security] fix security problem for _wget_download #61321

wants to merge 2 commits into from

Conversation

vn-ncvinh
Copy link
Contributor

PR types

Others

PR changes

Others

Description

fix security problem for _wget_download, use the subprocess module

@CLAassistant
Copy link

CLAassistant commented Jan 29, 2024 •
edited

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@paddle-bot paddle-bot bot added the contributor External developers label Jan 29, 2024
@luotao1
Copy link
Contributor

luotao1 commented Jan 30, 2024

image Please fix the CodeStyle problem

@wanghuancoder
Copy link
Contributor

wanghuancoder commented Jan 30, 2024 •
edited by luotao1

Following PR has already fixed this problem

@luotao1 luotao1 closed this Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@luotao1 luotao1

@wanghuancoder wanghuancoder

Labels
contributor External developers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vn-ncvinh @CLAassistant @luotao1 @wanghuancoder