Feature/hide play files #753
Conversation
FinemechanicPub
temporarily deployed
to
test_deploy
GitHub Actions
Inactive
| @@ -53,6 +53,11 @@ server { | |||
| add_header Content-disposition "attachment; filename=$1"; | |||
| } | |||
|
|
|||
| location /private-redirect/ { | |||
| internal; | |||
| alias /config/prod/protected_media/; | |||
There was a problem hiding this comment.
А почему тут конфиг в пути?
в докере указывается /code/protected_media/, не знаю пересекаются эти пути или нет
There was a problem hiding this comment.
Для swag задано
- ./protected_media:/config/prod/protected_media/
Для backend задано
- ./protected_media:/code/protected_media/
Таким образом в одна и так же папка на хосте сопоставлена двум папкам:
в контейнере backend это /code/protected_media/, а в контейнере swag это /config/prod/protected_media/.
Это конфиг Nginx, поэтому тут путь указывается так, как он выглядит из контейнера Nginx (Swag).
| @@ -35,7 +35,7 @@ server { | |||
| proxy_pass $upstream_proto://$upstream_app:$upstream_port; | |||
| } | |||
|
|
|||
| location ~^/size([0-9]{3})/media(?:/(.*))?$ { | |||
| location ~ "^/size([0-9]{3})/media(?:/(.*))?$" { | |||
There was a problem hiding this comment.
Ты тут выделил регулярку двойными ковычками, но во всех остальных местах которых редактировал регулярки, не выделял, но почему? Как правильно(корректно)?
There was a problem hiding this comment.
Если в регулярном выражении есть }, то его нужно оковычивать.
Вообще, эта правка не для пьес, а для работы уменьшителя изображений, который мы уже смержили, и в другой редакции. Нужно будет делать разрешение конфликтов.
|
|
||
| def move_to_protected_media(apps, schema_editor): | ||
| if REGULAR_MEDIA.exists(): | ||
| copytree(REGULAR_MEDIA, PROTECTED_MEDIA, copy_function=move, dirs_exist_ok=True) |
There was a problem hiding this comment.
Ой, а объясни что тут происходит?
There was a problem hiding this comment.
Конкретно здесь просто перемещается папка plays со всеми подпапками из папки общедоступной статики в папку защищенной статики, из которой nginx будет выдавать файлы только по разрешению нашего сервера.
А файловое поле в модели пьес меняется на специальное файловое поле. Не могу вспомнить, зачем нужно специальное поле. Предположу, что бы генерировать urls.
FinemechanicPub
temporarily deployed
to
test_deploy
GitHub Actions
Inactive
Тикет вашего PR (если есть):
Не давать скачивать текст неопубликованной пьесы