style: sonarqube fixes

Author: jd-apprentice

docker/ARM64.Dockerfile

@@ -22,9 +22,9 @@ FROM base AS release
 COPY --from=install /temp/prod/node_modules node_modules
 COPY --from=prerelease /bun/dist ./dist
 
-RUN mkdir -p /bun/src/image/assets/images
-RUN chown -R bun:bun /bun/src/image/assets/images
-RUN chmod -R 600 /bun/src/image/assets/images
+RUN mkdir -p /bun/src/image/assets/images && \
+    chown -R bun:bun /bun/src/image/assets/images && \
+    chmod -R 600 /bun/src/image/assets/images
 
 USER bun
 EXPOSE 4000

docker/Dockerfile

@@ -22,9 +22,9 @@ FROM base AS release
 COPY --from=install /temp/prod/node_modules node_modules
 COPY --from=prerelease /bun/dist ./dist
 
-RUN mkdir -p /bun/src/image/assets/images
-RUN chown -R bun:bun /bun/src/image/assets/images
-RUN chmod -R 600 /bun/src/image/assets/images
+RUN mkdir -p /bun/src/image/assets/images && \
+    chown -R bun:bun /bun/src/image/assets/images && \
+    chmod -R 600 /bun/src/image/assets/images
 
 USER bun
 EXPOSE 4000

src/image/image-controller.ts

@@ -1,52 +1,62 @@
 // External Modules
 import { Request, Response } from 'express';
+import { LogArgument } from 'rollbar';
 
 // Interal Modules
-import clearTemporaryFiles from '../common/utils/clear';
+import clearTemporaryFiles from 'src/common/utils/clear';
 import ImageService from './image-service';
-import { Query } from '../common/interfaces/types';
+import { Query } from 'src/common/interfaces/types';
+import { rollbar } from 'src/app/config/rollbar';
 
 class ImageController {
   /**
-   * @description Upload a file to cloudinary then saves the url on mongodb
-   * @param { Request } req.body - tag, source, is_nsfw
-   * @param { Request } file.path - path to the file
-   * @returns { Promise<Response> } A success message with a Json response format
+   * @description Uploads a file to Cloudinary and saves the URL in MongoDB
+   * @param { Request } req - The request object containing the file and additional data
+   * @returns { Promise<Response> } A JSON response with a success message
    */
   async uploadFile(req: Request, res: Response): Promise<Response> {
     try {
       const { file } = req;
       const { tag, source, is_nsfw } = req.body;
-      const response = await ImageService.cloudinaryUpload(file?.path);
-      const { public_id, secure_url } = response;
-      ImageService.upload({
+
+      if (!file) {
+        return res.status(400).json({ error: 'File is required' });
+      }
+
+      const uploadResult = await ImageService.cloudinaryUpload(file.path);
+      const { public_id, secure_url } = uploadResult;
+
+      await ImageService.upload({
         source,
         is_nsfw,
         id: public_id,
         url: secure_url,
         tag,
       });
-      clearTemporaryFiles(file?.path ?? 'image/assets/images');
-      return res.json({ url: 'Imagen guardada correctamente' });
+
+      clearTemporaryFiles(file.path ?? 'image/assets/images');
+      return res.json({ message: 'Image saved successfully', url: secure_url });
     } catch (error: unknown) {
-      return res.json({ message: (<Error>error).message });
+      rollbar.error(error as LogArgument);
+      return res.status(500).json({ error: 'An error occurred while uploading the image' });
     }
   }
 
   /**
    * @description Get a random waifu from the collection!
-   * @param { Response } res object with the waifu
+   * @param { Response } res - object with the waifu
    * @query size - number of items to retrieve
    * @query tag_id - tag to filter
    * @returns { Promise<Response> } An url with the waifu image hosted in cloudinary
    */
   async getRandomImage(req: Request, res: Response): Promise<Response> {
     try {
-      const { size, tag_id } = req.query as unknown as Query;
-      const getImages = await ImageService.getImage(size, tag_id);
-      return res.json(getImages);
-    } catch {
-      return res.json({ message: 'No se pudo encontrar alguna imagen' });
+      const { size, tag_id } = req.query as Query;
+      const images = await ImageService.getImage(size, tag_id);
+      return res.json(images);
+    } catch (error: unknown) {
+      rollbar.error(error as LogArgument);
+      return res.status(500).json({ error: 'An error occurred while getting the image' });
     }
   }
 
@@ -58,11 +68,12 @@ class ImageController {
    */
   async getImages(req: Request, res: Response): Promise<Response> {
     try {
-      const { tag_id } = req.query as unknown as Query;
+      const { tag_id } = req.query as Query;
       const images = await ImageService.getAllImages(tag_id);
       return res.json(images);
     } catch (error: unknown) {
-      return res.json({ message: (<Error>error).message });
+      rollbar.error(error as LogArgument);
+      return res.json({ error });
     }
   }
 }

src/image/image-repository.ts

@@ -2,15 +2,15 @@
 import Image from './schema/image-schema';
 import Tag from '../tag/schema/tag-schema';
 import { hasTag } from '../common/utils/ref';
-import { IImage } from './interfaces/image-interface';
+import { ImageProp } from './interfaces/image-interface';
 
 class ImageRepository {
   /**
    * @description Creates a new Image in the database
-   * @param {IImage} image - the image to create in the database
-   * @return { Promise<IImage> } - A new image created
+   * @param { ImageProp } image - the image to create in the database
+   * @return { Promise<ImageProp> } - A new image created
    */
-  async create(image: IImage): Promise<IImage> {
+  async create(image: ImageProp): Promise<ImageProp> {
     const tagExists = await Tag.findOne({ tag_id: { $eq: image.tag } });
     const _idTag = tagExists?._id;
     return Image.create({
@@ -22,9 +22,9 @@ class ImageRepository {
   /**
    * @description Get all images from the database
    * @param tag_id - the id from the tag to retrieve
-   * @return { Promise<IImage[]> } An array of images
+   * @return { Promise<ImageProp[]> } An array of images
    */
-  async findImages(tag_id?: number): Promise<IImage[]> {
+  async findImages(tag_id?: number): Promise<ImageProp[]> {
     const images = await Image.find().populate(hasTag(tag_id));
     return images.filter(image => image.tag !== null);
   }

src/image/image-service.ts

@@ -80,7 +80,7 @@ class ImageService {
           tag: image.tag,
         };
       });
-      const randomArray = urls.sort(randomUrls);
+      const randomArray = urls.toSorted(randomUrls);
       const sizeArray = randomArray.slice(0, size);
       const randomUrl = urls[Math.floor(Math.random() * urls.length)];
       return size === undefined || null ? randomUrl : sizeArray;

src/image/interfaces/image-interface.ts

@@ -20,4 +20,3 @@ export interface ImageTypeResponse extends ImageType {
 }
 
 export type ImageProp = Omit<ImageProps, "save">;
-export type IImage = ImageProp;

src/user/interfaces/user-interface.ts

@@ -1,3 +1,6 @@
+import { Boom } from "@hapi/boom";
+import { NextFunction, Response } from "express";
+
 export interface UsernameType {
   username: string;
 }
@@ -22,3 +25,5 @@ export type UserPicture = Omit<
   IUser,
   "username" | "password" | "save" | "isAdmin"
 >;
+
+export type MiddlewareUser = Boom | NextFunction | Response | void;