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

No audio with video downloads? #43

Open
retalieight opened this issue Jan 26, 2024 · 4 comments
Open

No audio with video downloads? #43

retalieight opened this issue Jan 26, 2024 · 4 comments

Comments

@retalieight
Copy link

Bug report

What is the current behavior?

I have no audio with the music videos that are downloaded. Is this normal? Is it supposed to only grab the video without audio?

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Download the music video with both video and audio.

Other relevant information:

  • Spotivy version: 0.6.0
  • Node.js version: v20.11.0
  • Operating System: Windows 10
@retalieight
Copy link
Author

retalieight commented Jan 26, 2024

I also received an error message during a download of a track:

[Downloading track] Get Scared - Sarcasm
                          Preparing download... C:\Users\chown\AppData\Roaming\npm\node_modules\spotivy\node_modules\miniget\dist\index.js:206
                let err = new Miniget.MinigetError(Status code: ${res.statusCode}, res.statusCode);
                          ^

MinigetError: Status code: 410
    at ClientRequest.<anonymous> (C:\Users\chown\AppData\Roaming\npm\node_modules\spotivy\node_modules\miniget\dist\index.js:206:27)
    at Object.onceWrapper (node:events:633:26)
    at ClientRequest.emit (node:events:518:28)
    at HTTPParser.parserOnIncomingClient (node:_http_client:693:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)
    at TLSSocket.socketOnData (node:_http_client:535:22)
    at TLSSocket.emit (node:events:518:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23)
Emitted 'error' event on Stream instance at:
    at Stream.onerror (node:internal/streams/readable:1026:14)
    at Stream.emit (node:events:518:28)
    at Stream._send (C:\Users\chown\AppData\Roaming\npm\node_modules\spotivy\node_modules\highland\lib\index.js:998:18)
    at Stream.write (C:\Users\chown\AppData\Roaming\npm\node_modules\spotivy\node_modules\highland\lib\index.js:1658:18)
    at PassThrough.streamEndCb (C:\Users\chown\AppData\Roaming\npm\node_modules\spotivy\node_modules\highland\lib\index.js:494:20)
    at Object.onceWrapper (node:events:633:26)
    at PassThrough.emit (node:events:518:28)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  statusCode: 410
}

@danguilherme
Copy link
Owner

@retalieight could you please share the command you're running?

@retalieight
Copy link
Author

retalieight commented Jan 27, 2024

@danguilherme spotivy playlist chownsauce myplaylistidhere -f video. I read that ytdl-core doesn't merge the video and audio together like youtube-dl or yt-dlp would using ffmpeg. Is it not merging or am I missing something? Is it supposed to be a video without audio?

@danguilherme
Copy link
Owner

Is it supposed to be a video without audio?

No, it's definitely supposed to be a video with audio. I'll check that out as soon as possible

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

2 participants