[API] Update dependencies, fix error checking

Typescript now defaults to unknown instead of any for Errors (catch).
Implemented helper function to check AxiosErros.

.eslintrc.js

@@ -75,7 +75,11 @@ module.exports = {
 
   // add your custom rules here
   rules: {
-    'prefer-promise-reject-errors': 'off',
+    // VueStuff
+    'vue/multi-word-component-names': 'off',
+
+    // Misc
+    'prefer-promise-reject-errors': ["error", {"allowEmptyReject": true}],
 
     // TypeScript
     quotes: ['warn', 'single', { avoidEscape: true }],

api/package.json

@@ -1,6 +1,6 @@
 {
   "license": "MIT",
-  "version": "1.0.0-alpha.1",
+  "version": "1.0.0-alpha.2",
   "name": "@flaschengeist/api",
   "author": "Tim Gröger <flaschengeist@wu5.de>",
   "homepage": "https://flaschengeist.dev/Flaschengeist",
@@ -19,21 +19,21 @@
     "pinia": "^2.0.0-alpha.19"
   },
   "devDependencies": {
-    "@flaschengeist/types": "^1.0.0-alpha.1",
-    "@types/node": "^12.20.13",
-    "@typescript-eslint/eslint-plugin": "^4.24.0",
-    "@typescript-eslint/parser": "^4.24.0",
-    "eslint": "^7.26.0",
+    "@flaschengeist/types": "^1.0.0-alpha.4",
+    "@types/node": "^12.20.37",
+    "@typescript-eslint/eslint-plugin": "^5.3.1",
+    "@typescript-eslint/parser": "^5.3.1",
+    "eslint": "^8.2.0",
     "eslint-config-prettier": "^8.3.0",
-    "eslint-plugin-vue": "^7.9.0",
-    "eslint-webpack-plugin": "^2.5.4",
-    "prettier": "^2.3.0",
-    "typescript": "^4.2.4"
+    "eslint-plugin-vue": "^8.0.3",
+    "eslint-webpack-plugin": "^3.1.0",
+    "prettier": "^2.4.1",
+    "typescript": "^4.4.4"
   },
   "prettier": {
     "singleQuote": true,
     "semi": true,
-    "printWidth": 120,
+    "printWidth": 100,
     "arrowParens": "always"
   }
 }

api/src/stores/index.ts

@@ -1,3 +1,23 @@
+import { AxiosError } from 'axios';
+
+/**
+ * Check if error is an AxiosError, and optional if a specific status was returned
+ *
+ * @param error Thrown error to check
+ * @param status If set, check if this error has set thouse status code
+ */
+export function isAxiosError(error: unknown, status?: number) {
+  // Check if it is an axios error (with axios 1.0 `error instanceof AxiosError` will be possible)
+  if (!error || typeof error !== 'object' || !('isAxiosError' in <object>error)) return false;
+  // Check status code if status was given
+  if (status !== undefined)
+    return (
+      (<AxiosError>error).response !== undefined && (<AxiosError>error).response?.status === status
+    );
+
+  return true;
+}
+
 export * from './main';
 export * from './session';
 export * from './user';

api/src/stores/main.ts

@@ -81,7 +81,7 @@ export const useMainStore = defineStore({
       } catch (error) {
         return false;
       } finally {
-        this.handleLoggedOut( );
+        this.handleLoggedOut();
       }
       return true;
     },

api/src/stores/session.ts

@@ -1,7 +1,7 @@
-import { AxiosError, AxiosResponse } from 'axios';
+import { AxiosResponse } from 'axios';
 import { defineStore } from 'pinia';
 import { api } from '../internal';
-import { useMainStore } from '.';
+import { isAxiosError, useMainStore } from '.';
 
 export const useSessionStore = defineStore({
   id: 'sessions',
@@ -46,8 +46,8 @@ export const useSessionStore = defineStore({
         await api.delete(`/auth/${token}`);
         return true;
       } catch (error) {
-        if (!error || !('response' in error) || (<AxiosError>error).response?.status != 401)
-          throw error;
+        // Ignore 401, as this means we are already logged out, throw all other
+        if (!isAxiosError(error, 401)) throw error;
       }
       return false;
     },

api/src/stores/user.ts

@@ -1,7 +1,6 @@
 import { defineStore } from 'pinia';
-import { AxiosError } from 'axios';
 import { api } from '../internal';
-import { useMainStore } from '.';
+import { isAxiosError, useMainStore } from '.';
 
 export const useUserStore = defineStore({
   id: 'users',
@@ -30,8 +29,8 @@ export const useUserStore = defineStore({
           else this.users[idx] = data;
           return data;
         } catch (error) {
-          if (!error || !('response' in error) || (<AxiosError>error).response?.status !== 404)
-            throw error;
+          // Ignore 404, throw all other
+          if (!isAxiosError(error, 404)) throw error;
         }
       } else {
         return this.users[idx];

src-cordova/cordova-flag.d.ts

@@ -1,9 +1,9 @@
 /* eslint-disable */
 // THIS FEATURE-FLAG FILE IS AUTOGENERATED,
 //  REMOVAL OR CHANGES WILL CAUSE RELATED TYPES TO STOP WORKING
-import "quasar/dist/types/feature-flag";
+import 'quasar/dist/types/feature-flag';
 
-declare module "quasar/dist/types/feature-flag" {
+declare module 'quasar/dist/types/feature-flag' {
   interface QuasarFeatureFlags {
     cordova: true;
   }

src-electron/electron-flag.d.ts

@@ -1,9 +1,9 @@
 /* eslint-disable */
 // THIS FEATURE-FLAG FILE IS AUTOGENERATED,
 //  REMOVAL OR CHANGES WILL CAUSE RELATED TYPES TO STOP WORKING
-import "quasar/dist/types/feature-flag";
+import 'quasar/dist/types/feature-flag';
 
-declare module "quasar/dist/types/feature-flag" {
+declare module 'quasar/dist/types/feature-flag' {
   interface QuasarFeatureFlags {
     electron: true;
   }

src/config.ts

@@ -1,4 +1,4 @@
-import {computed} from 'vue';
+import { computed } from 'vue';
 import { LocalStorage } from 'quasar';
 
 const config = {
@@ -6,9 +6,7 @@ const config = {
   pollingInterval: 30000,
 };
 
-const baseURL = computed(() =>
-LocalStorage.getItem<string>('baseURL') || config.baseURL
-);
+const baseURL = computed(() => LocalStorage.getItem<string>('baseURL') || config.baseURL);
 
-export {baseURL}
+export { baseURL };
 export default config;