search

linuxserver / docker-nzbget

GNU General Public License v3.0
149 stars 83 forks source link

Adding Lxml pip dependency for de-obfuscation scripts required by some indexers #120

Closed tylergibson closed 3 years ago

tylergibson commented 3 years ago

Adding the lxml pip package for python scripts

linuxserver.io

Description:

Adding the lxml pip package for python scripts

Benefits of this PR and context:

Some indexers have begun obfuscating file names in NZBs, requiring new scripts to parse NZBs and decode filenames from the xml metadata. These scripts have a dependency on lxml. While currently limited in affected indexers, other nzb binary downloaders have patched to handle this. This will likely become more pervasive with time.

How Has This Been Tested?

custom docker build on personal Unraid 6.8.2

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-307d3534-pr-120/index.html https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-307d3534-pr-120/shellcheck-result.xml

tylergibson commented 3 years ago

@nemchik - thoughts?

tylergibson commented 3 years ago

@nemchik - I double checked and rebased to current master, but the indentation lines up on both web and local?

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-a7f1c907-pr-120/index.html https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-a7f1c907-pr-120/shellcheck-result.xml

LinuxServer-CI commented 3 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-a7f1c907-pr-120/index.html https://ci-tests.linuxserver.io/lspipepr/nzbget/v21.0-pkg-a7f1c907-pr-120/shellcheck-result.xml

Roxedus commented 3 years ago

These scripts, are they bundled in NZBGet?