fix: use temp directory for all YouTube downloads to avoid permission errors

DragMusic/platforms/Youtube.py

@@ -1,4 +1,3 @@
-
 import asyncio
 import os
 import re
@@ -11,6 +10,8 @@ from pyrogram.types import Message
 from youtubesearchpython.__future__ import VideosSearch
 from DragMusic.utils.database import is_on_off
 from DragMusic.utils.formatters import time_to_seconds
+import tempfile
+
 async def shell_cmd(cmd):
     proc = await asyncio.create_subprocess_shell(
         cmd,
@@ -24,6 +25,7 @@ async def shell_cmd(cmd):
         else:
             return errorz.decode("utf-8")
     return out.decode("utf-8")
+
 class YouTubeAPI:
     def __init__(self):
         self.base = "https://www.youtube.com/watch?v="
@@ -31,6 +33,7 @@ class YouTubeAPI:
         self.status = "https://www.youtube.com/oembed?url="
         self.listbase = "https://youtube.com/playlist?list="
         self.reg = re.compile(r"\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])")
+
     async def exists(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -38,6 +41,7 @@ class YouTubeAPI:
             return True
         else:
             return False
+
     async def url(self, message_1: Message) -> Union[str, None]:
         messages = [message_1]
         if message_1.reply_to_message:
@@ -61,6 +65,7 @@ class YouTubeAPI:
         if offset in (None,):
             return None
         return text[offset : offset + length]
+
     async def details(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -77,6 +82,7 @@ class YouTubeAPI:
             else:
                 duration_sec = int(time_to_seconds(duration_min))
         return title, duration_min, duration_sec, thumbnail, vidid
+
     async def title(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -86,6 +92,7 @@ class YouTubeAPI:
         for result in (await results.next())["result"]:
             title = result["title"]
         return title
+
     async def duration(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -95,6 +102,7 @@ class YouTubeAPI:
         for result in (await results.next())["result"]:
             duration = result["duration"]
         return duration
+
     async def thumbnail(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -104,6 +112,7 @@ class YouTubeAPI:
         for result in (await results.next())["result"]:
             thumbnail = result["thumbnails"][0]["url"].split("?")[0]
         return thumbnail
+
     async def video(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -123,6 +132,7 @@ class YouTubeAPI:
             return 1, stdout.decode().split("\n")[0]
         else:
             return 0, stderr.decode()
+
     async def playlist(self, link, limit, user_id, videoid: Union[bool, str] = None):
         if videoid:
             link = self.listbase + link
@@ -139,6 +149,7 @@ class YouTubeAPI:
         except:
             result = []
         return result
+
     async def track(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -159,6 +170,7 @@ class YouTubeAPI:
             "thumb": thumbnail,
         }
         return track_details, vidid
+
     async def formats(self, link: str, videoid: Union[bool, str] = None):
         if videoid:
             link = self.base + link
@@ -194,6 +206,7 @@ class YouTubeAPI:
                         }
                     )
         return formats_available, link
+
     async def slider(
         self,
         link: str,
@@ -211,6 +224,7 @@ class YouTubeAPI:
         vidid = result[query_type]["id"]
         thumbnail = result[query_type]["thumbnails"][0]["url"].split("?")[0]
         return title, duration_min, thumbnail, vidid
+
     async def get_video_info_from_bitflow(self, url: str, video: bool):
         api_url = "https://bitflow.in/api/youtube"
         params = {
@@ -226,6 +240,7 @@ class YouTubeAPI:
                 return response.json()
             else:
                 return {"status": "error", "message": "Failed to fetch data from Bitflow API."}
+
     async def download(
         self,
         link: str,
@@ -244,7 +259,8 @@ class YouTubeAPI:
         loop = asyncio.get_running_loop()
         bitflow_info = await self.get_video_info_from_bitflow(link, video)
         def audio_dl(bitflow_info):
-            xyz = os.path.join("downloads", f"{bitflow_info['videoid']}.{bitflow_info['ext']}")
+            temp_dir = tempfile.gettempdir()
+            xyz = os.path.join(temp_dir, f"{bitflow_info['videoid']}.{bitflow_info['ext']}")
             ydl_optssx = {
                 "format": "bestaudio/best",
                 "outtmpl": xyz,
@@ -259,7 +275,8 @@ class YouTubeAPI:
             x.download([bitflow_info['url']])
             return xyz
         def video_dl(bitflow_info):
-            xyz = os.path.join("downloads", f"{bitflow_info['videoid']}.{bitflow_info['ext']}")
+            temp_dir = tempfile.gettempdir()
+            xyz = os.path.join(temp_dir, f"{bitflow_info['videoid']}.{bitflow_info['ext']}")
             ydl_optssx = {
                 "format": "(bestvideo[height<=?720][width<=?1280][ext=mp4])+(bestaudio[ext=m4a])",
                 "outtmpl": xyz,
@@ -274,8 +291,9 @@ class YouTubeAPI:
             x.download([bitflow_info['url']])
             return xyz
         def song_video_dl():
+            temp_dir = tempfile.gettempdir()
+            fpath = os.path.join(temp_dir, f"{title}")
             formats = f"{format_id}+140"
-            fpath = f"downloads/{title}"
             ydl_optssx = {
                 "format": formats,
                 "outtmpl": fpath,
@@ -289,7 +307,8 @@ class YouTubeAPI:
             x = yt_dlp.YoutubeDL(ydl_optssx)
             x.download([link])
         def song_audio_dl():
-            fpath = f"downloads/{title}.%(ext)s"
+            temp_dir = tempfile.gettempdir()
+            fpath = os.path.join(temp_dir, f"{title}.%(ext)s")
             ydl_optssx = {
                 "format": format_id,
                 "outtmpl": fpath,
@@ -310,11 +329,13 @@ class YouTubeAPI:
             x.download([link])
         if songvideo:
             await loop.run_in_executor(None, song_video_dl)
-            fpath = f"downloads/{title}.mp4"
+            temp_dir = tempfile.gettempdir()
+            fpath = os.path.join(temp_dir, f"{title}.mp4")
             return fpath
         elif songaudio:
             await loop.run_in_executor(None, song_audio_dl)
-            fpath = f"downloads/{title}.mp3"
+            temp_dir = tempfile.gettempdir()
+            fpath = os.path.join(temp_dir, f"{title}.mp3")
             return fpath
         elif video:
             direct = True