Add DOWNLOAD_MISSING_BEATMAPS env var

Author: Syriiin

Difficalcy/Services/WebBeatmapProvider.cs

@@ -9,13 +9,17 @@ namespace Difficalcy.Services
     public class WebBeatmapProvider(IConfiguration configuration) : IBeatmapProvider
     {
         private readonly string _beatmapDirectory = configuration["BEATMAP_DIRECTORY"];
+        private readonly string _downloadMissingBeatmaps = configuration["DOWNLOAD_MISSING_BEATMAPS"];
         private readonly HttpClient _httpClient = new();
 
         public async Task EnsureBeatmap(string beatmapId)
         {
             var beatmapPath = GetBeatmapPath(beatmapId);
             if (!File.Exists(beatmapPath))
             {
+                if (_downloadMissingBeatmaps != "true")
+                    throw new BadHttpRequestException("Beatmap not found");
+
                 using var response = await _httpClient.GetAsync($"https://osu.ppy.sh/osu/{beatmapId}");
                 if (!response.IsSuccessStatusCode || response.Content.Headers.ContentLength == 0)
                     throw new BadHttpRequestException("Beatmap not found");

Dockerfile

@@ -4,9 +4,10 @@ LABEL org.opencontainers.image.source https://github.com/Syriiin/difficalcy
 
 WORKDIR /app
 EXPOSE 80
-ENV ASPNETCORE_URLS=http://+:80
-ENV ASPNETCORE_ENVIRONMENT=Production
-ENV BEATMAP_DIRECTORY=/beatmaps
+ENV ASPNETCORE_URLS="http://+:80"
+ENV ASPNETCORE_ENVIRONMENT="Production"
+ENV BEATMAP_DIRECTORY="/beatmaps"
+ENV DOWNLOAD_MISSING_BEATMAPS="true"
 
 VOLUME ${BEATMAP_DIRECTORY}
 # chmod 777 so that this volume can be read/written by other containers that might use different uids

docs/docs/configuration.md

@@ -6,9 +6,10 @@ However it would be a good idea to consider these configuration options for a re
 
 ## Environment Variables
 
-| Environment variable  | Default | Description                                                                                     |
-| --------------------- | ------- | ----------------------------------------------------------------------------------------------- |
-| `REDIS_CONFIGURATION` |         | The address of the redis server to use for beatmap caching. By default, there will be no cache. |
+| Environment variable        | Default | Description                                                                                     |
+| --------------------------- | ------- | ----------------------------------------------------------------------------------------------- |
+| `REDIS_CONFIGURATION`       |         | The address of the redis server to use for beatmap caching. By default, there will be no cache. |
+| `DOWNLOAD_MISSING_BEATMAPS` | true    | Whether or not to attempt to download missing beatmaps.                                         |
 
 ## Docker volumes