From 13cefeda05f973e7b6378897b5197fddf5cc88ed Mon Sep 17 00:00:00 2001
From: davide-pi <davide.piccinini@engageitservices.it>
Date: Sat, 14 Sep 2024 20:01:09 +0200
Subject: [PATCH] remove snapcraft configurations, move into base class

---
 config/custom-environment-variables.yml | 2 --
 config/default.yml                      | 2 --
 core/server/server.js                   | 3 ---
 doc/server-secrets.md                   | 6 ------
 services/snapcraft/snapcraft-base.js    | 5 ++---
 5 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/config/custom-environment-variables.yml b/config/custom-environment-variables.yml
index f3d923fb70f4a..6686b245c8faa 100644
--- a/config/custom-environment-variables.yml
+++ b/config/custom-environment-variables.yml
@@ -56,8 +56,6 @@ public:
       authorizedOrigins: 'OBS_ORIGINS'
     pypi:
       baseUri: 'PYPI_URL'
-    snapcraft:
-      baseUri: 'SNAPCRAFT_URL'
     sonar:
       authorizedOrigins: 'SONAR_ORIGINS'
     teamcity:
diff --git a/config/default.yml b/config/default.yml
index e8cfb01240fcb..cea2edad7b60d 100644
--- a/config/default.yml
+++ b/config/default.yml
@@ -24,8 +24,6 @@ public:
       authorizedOrigins: 'https://api.opensuse.org'
     pypi:
       baseUri: 'https://pypi.org'
-    snapcraft:
-      baseUri: 'https://api.snapcraft.io/v2/snaps/info'
     weblate:
       authorizedOrigins: 'https://hosted.weblate.org'
     trace: false
diff --git a/core/server/server.js b/core/server/server.js
index 83686c9e390d5..f5621d9411257 100644
--- a/core/server/server.js
+++ b/core/server/server.js
@@ -142,9 +142,6 @@ const publicConfigSchema = Joi.object({
     pypi: {
       baseUri: requiredUrl,
     },
-    snapcraft: {
-      baseUri: requiredUrl,
-    },
     sonar: defaultService,
     teamcity: defaultService,
     weblate: defaultService,
diff --git a/doc/server-secrets.md b/doc/server-secrets.md
index ca2b154c4b420..d7d66fc3cdaca 100644
--- a/doc/server-secrets.md
+++ b/doc/server-secrets.md
@@ -290,12 +290,6 @@ Create an account, sign in and obtain generate a key on your
 `PYPI_URL` can be used to optionally send all the PyPI requests to a Self-hosted Pypi registry,
 users can also override this by query parameter `pypiBaseUrl`.
 
-### Snapcraft
-
-- `SNAPCRAFT_URL` (yml: `public.snapcraft.baseUri`)
-
-`SNAPCRAFT_URL` can be used to optionally send all the Snapcraft requests to a Self-hosted Snapcraft registry.
-
 ### SymfonyInsight (formerly Sensiolabs)
 
 - `SL_INSIGHT_USER_UUID` (yml: `private.sl_insight_userUuid`)
diff --git a/services/snapcraft/snapcraft-base.js b/services/snapcraft/snapcraft-base.js
index 59288236ec5b7..c8986a0f8647a 100644
--- a/services/snapcraft/snapcraft-base.js
+++ b/services/snapcraft/snapcraft-base.js
@@ -1,4 +1,3 @@
-import config from 'config'
 import { BaseJsonService, pathParam } from '../index.js'
 
 export const snapcraftPackageParam = pathParam({
@@ -8,10 +7,10 @@ export const snapcraftPackageParam = pathParam({
 
 export const snapcraftBaseParams = [snapcraftPackageParam]
 
+const snapcraftBaseUrl = 'https://api.snapcraft.io/v2/snaps/info'
+
 export default class SnapcraftBase extends BaseJsonService {
   async fetch(schema, { packageName }) {
-    const snapcraftBaseUrl =
-      config.util.toObject().public.services.snapcraft.baseUri
     return await this._requestJson({
       schema,
       url: `${snapcraftBaseUrl}/${packageName}`,