Помогите улучшить способ загрузки файлов (инфраструктура Pyramid)

В настоящее время я использую следующий метод для загрузки файлов (через HTML-форму) в Pyramid.

if request.params.get('form.submitted'):

    upload_directory = os.getcwd() + '/myapp/static/uploads/'

    my_file = request.POST.get('thumbnail')
    saved_file = str(upload_directory) + str(my_file.filename)

    perm_file = open(saved_file, 'w')

    shutil.copyfileobj(my_file.file, perm_file)
    my_file.file.close()
    perm_file.close()

Мне просто интересно, это хороший способ сохранить загрузки файлов, есть ли проблемы с безопасностью в моем методе? Как еще я могу улучшить свой метод. Спасибо.

Ответ 1

Вы хотите использовать что-то вроде werkzug safe_join, а не просто добавлять каталог загрузки в указанное имя файла. Злоумышленник может создать POST с именем файла ../../../some/important/path и заставить этот script перезаписать некоторый файл вне вашего upload_directory.