mikhailnov / getcourse-video-downloader

Скрипт для скачивания видео с GetCourse
96 stars 14 forks source link

Fix regexp for detecting .ts urls #2

Closed udovichenko closed 1 year ago

udovichenko commented 1 year ago

У меня регулярка из вашего скрипта не работала. Мб дело в разной обработке регэксов на разых системах или версиях баша (у меня MacOS). Отладчик показал, что слэши должны экранироваться. Поменял на такой вариант https://regex101.com/r/t5dv6C/1 - заработало. Ну и определение наличия https сделал полаконичней)

Спасибо за скрипт!

mikhailnov commented 1 year ago

Различаются не версии баша, а версии grep, регулярка же им обрабатывается. В Линуксах GNU grep, а в Mac OS, наверное, позаимствованный из FreeBSD вариант. В пулл-реквест https://github.com/mikhailnov/getcourse-video-downloader/pull/1 тоже была проблема с конструкцией (s|), может быть, у @etoosamoe тоже макось. Предложенный Вами вариант работает в GNU grep, смерживаю, спасибо!