Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tab symbol error handling #35

Closed
prostopasta opened this issue Jun 22, 2020 · 3 comments
Closed

Tab symbol error handling #35

prostopasta opened this issue Jun 22, 2020 · 3 comments

Comments

@prostopasta
Copy link
Contributor

prostopasta commented Jun 22, 2020

Experienced 'Tab' symbol in title name, got the following error:
"""
Retrieving metadata for:

  1.      What We're Going to Learn
    

Downloading video file for:

  1.      What We're Going to Learn.mp4
    

events.js:292
throw er; // Unhandled 'error' event
^

Error: ENOENT: no such file or directory, open

F:!Programming!\pluralsight-scraper\videos\incident-response-handling-performing\73.
What We're Going to Learn.mp4

Emitted 'error' event on WriteStream instance at:
at errorOrDestroy (internal/streams/destroy.js:108:12)
at WriteStream.onerror (_stream_readable.js:753:7)
at WriteStream.emit (events.js:315:20)
at internal/fs/streams.js:376:12
at FSReqCallback.oncomplete (fs.js:155:23) {
errno: -4058,
code: 'ENOENT',
syscall: 'open',
path: "F:\\!Programming!\\pluralsight-scraper\\videos\\incident-response-handling-performing\\73. \tWhat We're Going to Learn.mp4"
"""
What symbol should I add to exclusion - would work adding [\t] or need some other tabular symbol coding?

@vezaynk
Copy link
Collaborator

vezaynk commented Jun 22, 2020

would work adding [\t] or need some other tabular symbol coding?

I think? I didn't even know this could be an issue. Can other white space characters break things too? shudders

Lets be on the safe side and do something like this: https://stackoverflow.com/a/1981366/4088472

@prostopasta
Copy link
Contributor Author

ok, got ya

@jubayerarefin
Copy link
Contributor

I had the same issue. So, I just trimmed the title and it is working nicely. Created a PR for this.

@vezaynk vezaynk closed this as completed Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants