[BUG] YouTube video link with playlist parameter is mistaken as playlist link

[SaptarshiSarkar12]

Describe the bug

A person clicked on a video from a YouTube playlist and when the link to the video is pasted in Drifty,

• Drifty GUI tries to fetch the video name indefinitely. It also shows a popup window mentioning how many videos are there in the playlist but, it fails to download any. So, it keeps on detecting filename and computer heats up.
• Drifty CLI shows an error that it fails to detect the filename and prompts the user to exit.

Steps To Reproduce

1. Open Drifty GUI application.
2. Paste a YouTube video link (containing both watch?v= and list= parameters), for example, this link - https://www.youtube.com/watch?v=uy_PEGgUF4U&list=PL0lo9MOBetEFGPccyxyfex8BYF_PQUQWn&index=1
3. You can see that Drifty GUI tries to fetch the filename and the progress bar moves continuously indefinitely.
4. Repeat the same in Drifty CLI to see that bug also.

Expected Behavior

Drifty must recognize that the URL is of a YouTube video and not of any playlists.

Screenshots

Drifty CLI

Drifty GUI

File.Name.detection.going.on.for.indefinite.time.mp4

Additional information

A regex match to identify if the provided link is a YouTube playlist or a specific YouTube video link, must be added.
Yt-dlp fails to identify if a specific video is asked rather than the full playlist as seen when this code fragment is executed.

Drifty/GUI/src/main/java/ui/GetFilename.java

Lines 172 to 192 in 3fff0e5

	String command = Program.get(YT_DLP);
	String[] args = new String[]{command, "--flat-playlist", "--skip-download", "-P", dir, link};
	ProcessBuilder pb = new ProcessBuilder(args);
	pb.redirectErrorStream(true);
	Process process = pb.start();
	StringJoiner joiner = new StringJoiner(lineFeed);
	try {
	try (
	InputStream inputStream = process.getInputStream();
	BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream))
	)
	{
	String line;
	while ((line = reader.readLine()) != null) {
	if (this.isCancelled()) {
	break;
	}
	joiner.add(line);
	feedback.setValue(joiner.toString());
	}
	}

A possible and reliable soluyion is to truncate the url upto the specific video id (the resultant url will be of the form https://www.youtube.com/watch?v=uy_PEGgUF4U).

If anyone wants to work on this issue, please comment below.

[github-actions]

Hello 👋! Thank you very much for raising an issue 🙌! The maintainers will get back to you soon for discussion over the issue! 🚀

Meanwhile you can also discuss about the project in our Discord Server 😀

[Tarunkumarkanakam]

I want to work on this issue. Is anyone working on this?
What is the intended behavior? Does the full playlist need to be downloaded or only the first video of the playlist? As per what I read above I think the application should ignore the playlist urls right?

[SaptarshiSarkar12]

I want to work on this issue. Is anyone working on this?

@Tarunkumarkanakam Thank you for your interest in the issue. This issue is still looking for contributors. I am assigning this issue to you.

What is the intended behavior? Does the full playlist need to be downloaded or only the first video of the playlist?

Only the video (identified from the ?watch?v=<videoID>) needs to be downloaded. Not the whole playlist.

As per what I read above I think the application should ignore the playlist urls right?

Yeah, you are right. The YouTube url provided must either have watch?v=<videoID> or playlist?list=<playlistID>. Not both. So, you will need to remove either of them based on the following rule:
watch?v=<videoID> gets precedence over playlist?list=<playlistID>

[Tarunkumarkanakam]

Well, I am a Windows user. To run in a docker environment I need to execute xhost +local:docker right. So initially, I need to set up X Server on my machine right?

If yes can you give steps to do that?

[Tarunkumarkanakam]

Can you help me with this issue. I added DISPLAY variable in my WSL bash, and I also downloaded VcXSrv, Tested it with xclock working without any issue. But still this error is still there in my console