✨Fix bad url in Dota 2 web api get_real_stats

Author: Aluerie

src/beta_base.py

@@ -1,9 +1,11 @@
 """
 Base class for BetaCog for `modules.beta.py`.
 
-It's a silly way to quickly beta-test some things, but this way
-`modules.beta.py` can be done in the least amount of lines as all the imports and chores
-are handled in this base file instead.
+In `modules.beta.py` I quickly beta-test some things. It's a bit silly but very efficient.
+
+The purpose of the current file is to minimize amount of imports and lines of code
+for anything we're going to do while beta-testing. The performance cost of extra imports is probably
+negligible compared to annoyance to type them manually out every time we need them.
 
 For an example of how `modules.beta.py` looks - you can look in `examples` folder.
 """

src/core/bot.py

@@ -531,7 +531,7 @@ async def event_error(self, payload: twitchio.EventErrorPayload) -> None:
             embed = self.add_args_field(embed, f"Extra {payload.error.__class__.__name__} Debug Data", payload.error.data)
         await self.exc_manager.register_error(payload.error, embed=embed)
 
-    # SHORTCUTS
+    # SHORTCUTS AND UTILITIES
 
     def webhook_from_url(self, url: str) -> discord.Webhook:
         """A shortcut function with filled in discord.Webhook.from_url args."""
@@ -552,8 +552,6 @@ def error_ping(self) -> str:
         """Error Role ping used to notify the developer(-s) about some errors."""
         return "<@&1337106675433340990>" if self.test else "<@&1116171071528374394>"
 
-    # UTILITIES AND SHORTCUTS
-
     def is_online(self, user_id: str) -> bool:
         """Whether the user is online.
 

src/utils/dota/web_api.py

@@ -35,11 +35,12 @@ async def invoke(
     ) -> Any:
         """Invoke a request to Steam Web API."""
         queries = "&".join(f"{k}={v}" for k, v in kwargs.items())
-        url = f"https://api.steampowered.com/{endpoint}/?key={self.api_key}{queries}"
+        url = f"https://api.steampowered.com/{endpoint}/?key={self.api_key}&{queries}"
         max_failures = 5
         for _ in range(max_failures):
             async with self.session.get(url) as resp:
-                # encoding='utf-8' errored out one day
+                # encoding='utf-8' errored out one day, it seems Valve have misconfigured some servers' content types
+                # Or maybe they have to because all the unique characters in player names?
                 result = await resp.json(loads=orjson.loads, content_type=None)
                 if result:
                     break