Skip to content

certificate_generator/app/routes.py: fix for potential path manipulation #957

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

seowwj
Copy link

@seowwj seowwj commented Sep 29, 2024

Pull Request Template

script name - certificate_generator/app/routes.py

Brief about script
Fixes potential path manipulation in the routes.py file.

  • Checks if the filename is alphanumeric
  • Checks if the filename does not contain '/' or '..'

Issue no.(must) - #951

Self Check(Tick After Making pull Request)

  • This issue was assigned to me.
  • One Change in one Pull Request
  • My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
  • I am following clean code and Documentation and my code is well linted with flake8.
  • I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
  • I have used REPO README TEAMPLATE (Necessary)
  • Just including required dependencies in requirements.txt (Don't include Python version too)

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Signed-off-by: Wen Jie Seow <35338681+seowwj@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant