[BUG] Unable start `TypeError: Cannot read properties of null (reading 'includes')`

Describe the bug Can`t start youtubedl-m due error includes null

2022-05-05T08:58:55.945Z INFO: Config items set using ENV variables.
/home/debian/youtubedl-material/youtube-dl.js:93
    if (!is_windows && details_json && details_json['path'].includes('.exe')) {
                                                            ^

TypeError: Cannot read properties of null (reading 'includes')
    at Object.exports.verifyBinaryExistsLinux (/home/debian/youtubedl-material/youtube-dl.js:93:61)
    at startYoutubeDL (/home/debian/youtubedl-material/app.js:671:19)
    at loadConfig (/home/debian/youtubedl-material/app.js:514:11)
    at async setAndLoadConfig (/home/debian/youtubedl-material/app.js:483:5)

I see the PR about checking binary of youtube-dl, seems it solve some weird behaviour thinking run windows? and no .exe extension here. Full path will be node_modules/youtube-dl/bin/youtube-dl

To Reproduce Steps to reproduce the behavior:

1. Follow install guide
2. node ./app.js

Expected behavior Application should start

Screenshots

Environment

• YoutubeDL-Material git master
• Dockerized nodejs 16 self build

Youtube-dl version Youtube-dl itself installed by NPM correctly with version 2021.12.17

[Tzahi12345]

@GlassedSilver no need to overcomplicate it! We can just check if the path is null and handle that case. More ideally we should check all necessary fields to make sure they’re set properly.

I actually expected the details file to be more mangled than this. The hotfix will prevent the error from coming up but I’ll make a more permanent solution that handles more edge cases. Thanks for the info @martadinata666!

[martadinata666]

Technically, yes it works. But can’t really tell specific situation like #601 , as i myself build for my arm64 and not encountering something like that. Maybe some node 12 quirk. I will send PR to test nodejs upgrade.