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

Update app.py #93

Merged
merged 1 commit into from
Jul 26, 2023
Merged

Update app.py #93

merged 1 commit into from
Jul 26, 2023

Conversation

Harushii18
Copy link
Contributor

Using Share=True for gradio causes a security risk as it creates a url that is publicly accessible by anyone. This link can be exploited. also, the proxy gradio uses by this is flagged as a virus by antiviruses so ends up getting deleted at times According to Gradio:

Keep in mind, however, that these links are publicly accessible, meaning that anyone can use your model for prediction! Therefore, make sure not to expose any sensitive information through the functions you write, or allow any critical changes to occur on your device.

Using Share=True for gradio causes a security risk as it creates a url that is publicly accessible by anyone. This link can be exploited. also, the proxy gradio uses by this is flagged as a virus by antiviruses so ends up getting deleted at times
According to Gradio: 
>>> Keep in mind, however, that these links are publicly accessible, meaning that anyone can use your model for prediction! Therefore, make sure not to expose any sensitive information through the functions you write, or allow any critical changes to occur on your device.
@haoheliu haoheliu merged commit dda0f54 into haoheliu:main Jul 26, 2023
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