From aa484f834ce0aaa0304624b5364e94018ee8dc1a Mon Sep 17 00:00:00 2001
From: Artem <aopoltorzhicky@gmail.com>
Date: Sun, 22 Oct 2023 14:20:25 +0200
Subject: [PATCH] Fix: validate contract path

---
 src/router/index.js                           | 57 +++++++++++++------
 .../contract/TransfersTab/TransfersTab.vue    | 40 -------------
 2 files changed, 41 insertions(+), 56 deletions(-)
 delete mode 100644 src/views/contract/TransfersTab/TransfersTab.vue

diff --git a/src/router/index.js b/src/router/index.js
index 1e15e76e..b0a7f8cc 100644
--- a/src/router/index.js
+++ b/src/router/index.js
@@ -19,7 +19,6 @@ import InteractTab from '@/views/contract/InteractTab.vue'
 import ViewsTab from '@/views/contract/ViewsTab/ViewsTab.vue'
 import StorageTab from '@/views/contract/StorageTab.vue'
 import ContractTokensTab from '@/views/contract/TokensTab/TokensTab.vue'
-import TransfersTab from '@/views/contract/TransfersTab/TransfersTab.vue'
 import MetadataTab from '@/views/contract/MetadataTab/MetadataTab.vue'
 import ForkTab from '@/views/contract/ForkTab.vue'
 import EventsTab from '@/views/contract/EventsTab/EventsTab.vue'
@@ -38,6 +37,8 @@ import ConstantTab from "@/views/constant/ConstantTab";
 import Constant from "@/views/constant/Constant";
 import ContractStatsTab from "@/views/contract/StatsTab.vue";
 
+import {checkAddress, isKT1Address} from '@/utils/tz.js'
+
 Vue.use(VueRouter)
 
 function validateNetwork(networks) {
@@ -49,6 +50,27 @@ function validateNetwork(networks) {
   }
 }
 
+
+function validateAddress(networks) {
+  return function(to, _from, next) {
+    if (networks.indexOf(to.params.network) == -1 || !checkAddress(to.params.address)) {
+      next('/not_found');
+    } else {
+      next();
+    }
+  }
+}
+
+function validateContractAddress() {
+  return function(to, _from, next) {
+    if (!isKT1Address(to.params.address)) {
+      next('/not_found');
+    } else {
+      next();
+    }
+  }
+}
+
 export function newRouter(networks) {
   return new Router({
     linkActiveClass: '',
@@ -139,8 +161,8 @@ export function newRouter(networks) {
         ]
       },
       {
-        path: '/:network/:address([0-9A-z]{36})',
-        beforeEnter: validateNetwork(networks),
+        path: '/:network/:address',
+        beforeEnter: validateAddress(networks),
         components: {
           default: Contract,
         },
@@ -173,19 +195,22 @@ export function newRouter(networks) {
             path: 'interact/:entrypoint?',
             name: 'interact',
             component: InteractTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'views/:view?',
             name: 'views',
             component: ViewsTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'storage',
             name: 'storage',
             component: StorageTab,
             props: true,
+            beforeEnter: validateContractAddress(),
             children: [
               {
                 path: 'big_map/:ptr(\\d+)',
@@ -219,13 +244,15 @@ export function newRouter(networks) {
             path: 'code',
             name: 'code',
             component: CodeTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'metadata',
             name: 'metadata',
             component: MetadataTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'fork',
@@ -234,30 +261,28 @@ export function newRouter(networks) {
               default: ForkTab,
             },
             props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'tokens',
             name: 'tokens',
             component: ContractTokensTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'stats',
             name: 'contract_stats',
             component: ContractStatsTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
           {
             path: 'details',
             name: 'details',
             component: DetailsTab,
-            props: true
-          },
-          {
-            path: 'transfers',
-            name: 'transfers',
-            component: TransfersTab,
-            props: true
+            props: true,
+            beforeEnter: validateContractAddress(),
           },
         ]
       },
diff --git a/src/views/contract/TransfersTab/TransfersTab.vue b/src/views/contract/TransfersTab/TransfersTab.vue
deleted file mode 100644
index 5ff5d7bb..00000000
--- a/src/views/contract/TransfersTab/TransfersTab.vue
+++ /dev/null
@@ -1,40 +0,0 @@
-<template>
-  <v-container class="canvas fill-canvas pa-8 ma-0" fluid>
-    <v-row no-gutters>
-      <v-col cols="12" class="pa-2">
-        <InfoCard>
-          <span
-            >The information you are looking for is no longer available on BCD,
-            please use
-          </span>
-          <a v-if="network !== 'jakartanet'" :href="tzkt_link" target="_blank">TzKT.io</a>
-          <span v-else></span>
-          <span> for this particular case</span>
-        </InfoCard>
-      </v-col>
-    </v-row>
-  </v-container>
-</template>
-
-<script>
-import InfoCard from "@/components/Cards/InfoCard.vue";
-
-let sanitizeUrl = require("@braintree/sanitize-url").sanitizeUrl;
-
-export default {
-  name: "TransfersTab",
-    components: {
-    InfoCard,
-  },
-  props: {
-    contract: Object,
-    address: String,
-    network: String,
-  },
-  computed: {
-    tzkt_link() {
-      return sanitizeUrl(`https://${this.network}.tzkt.io/${this.address}/balances`);
-    }
-  }
-};
-</script>