Fetch latest release from github only once a day

Author: vishesh92

ui/src/components/page/GlobalFooter.vue

@@ -22,13 +22,13 @@
     </div>
     <div class="line" v-if="$store.getters.userInfo.roletype === 'Admin'">
       CloudStack {{ $store.getters.features.cloudstackversion }}
-      <span v-if="$store.getters.features.cloudstackversion && $store.getters.features.cloudstackversion.split('-')[0] !== $store.getters.latestVersion && $store.getters.latestVersion !== ''">
+      <span v-if="$store.getters.features.cloudstackversion && $store.getters?.latestVersion?.version && $store.getters.features.cloudstackversion.split('-')[0] !== $store.getters.latestVersion.version">
         <a-divider type="vertical" />
         <a
-          :href="'https://github.com/apache/cloudstack/releases/tag/' + $store.getters.latestVersion"
+          :href="'https://github.com/apache/cloudstack/releases/tag/' + $store.getters.latestVersion.version"
           target="_blank">
             <info-circle-outlined />
-            {{ $t('label.new.version.available') + ': ' + $store.getters.latestVersion }}
+            {{ $t('label.new.version.available') + ': ' + $store.getters.latestVersion.version }}
         </a>
       </span>
       <a-divider type="vertical" />

ui/src/store/modules/user.js

@@ -37,6 +37,7 @@ import {
   HEADER_NOTICES,
   DOMAIN_STORE,
   DARK_MODE,
+  LATEST_CS_VERSION,
   CUSTOM_COLUMNS
 } from '@/store/mutation-types'
 
@@ -154,6 +155,7 @@ const user = {
       state.loginFlag = flag
     },
     SET_LATEST_VERSION: (state, version) => {
+      vueProps.$localStorage.set(LATEST_CS_VERSION, version)
       state.latestVersion = version
     }
   },
@@ -200,7 +202,8 @@ const user = {
           commit('SET_2FA_PROVIDER', result.providerfor2fa)
           commit('SET_2FA_ISSUER', result.issuerfor2fa)
           commit('SET_LOGIN_FLAG', false)
-          commit('SET_LATEST_VERSION', '')
+          const latestVersion = vueProps.$localStorage.get(LATEST_CS_VERSION, { version: '', fetchedTs: 0 })
+          commit('SET_LATEST_VERSION', latestVersion)
           notification.destroy()
 
           resolve()
@@ -219,10 +222,12 @@ const user = {
         const cachedCustomColumns = vueProps.$localStorage.get(CUSTOM_COLUMNS, {})
         const domainStore = vueProps.$localStorage.get(DOMAIN_STORE, {})
         const darkMode = vueProps.$localStorage.get(DARK_MODE, false)
+        const latestVersion = vueProps.$localStorage.get(LATEST_CS_VERSION, { version: '', fetchedTs: 0 })
         const hasAuth = Object.keys(cachedApis).length > 0
 
         commit('SET_DOMAIN_STORE', domainStore)
         commit('SET_DARK_MODE', darkMode)
+        commit('SET_LATEST_VERSION', latestVersion)
         if (hasAuth) {
           console.log('Login detected, using cached APIs')
           commit('SET_ZONES', cachedZones)
@@ -236,20 +241,7 @@ const user = {
             const result = response.listusersresponse.user[0]
             commit('SET_INFO', result)
             commit('SET_NAME', result.firstname + ' ' + result.lastname)
-            if (result.rolename === 'Root Admin') {
-              axios.get(
-                'https://api.github.com/repos/apache/cloudstack/releases'
-              ).then(response => {
-                for (const release of response) {
-                  if (release.tag_name.toLowerCase().includes('rc')) {
-                    continue
-                  } else {
-                    commit('SET_LATEST_VERSION', release.tag_name)
-                    break
-                  }
-                }
-              }).catch(ignored => {})
-            }
+            store.dispatch('SetCsLatestVersion', result.rolename)
             resolve(cachedApis)
           }).catch(error => {
             reject(error)
@@ -327,6 +319,7 @@ const user = {
           const result = response.listusersresponse.user[0]
           commit('SET_INFO', result)
           commit('SET_NAME', result.firstname + ' ' + result.lastname)
+          store.dispatch('SetCsLatestVersion', result.rolename)
         }).catch(error => {
           reject(error)
         })
@@ -461,6 +454,22 @@ const user = {
     SetDomainStore ({ commit }, domainStore) {
       commit('SET_DOMAIN_STORE', domainStore)
     },
+    SetCsLatestVersion ({ commit }, rolename) {
+      if (rolename === 'Root Admin' && (+new Date() - store.getters.latestVersion.fetchedTs) > 24 * 60 * 60 * 1000) {
+        axios.get(
+          'https://api.github.com/repos/apache/cloudstack/releases'
+        ).then(response => {
+          for (const release of response) {
+            if (release.tag_name.toLowerCase().includes('rc')) {
+              continue
+            } else {
+              commit('SET_LATEST_VERSION', { version: release.tag_name, fetchedTs: (+new Date()) })
+              break
+            }
+          }
+        }).catch(ignored => {})
+      }
+    },
     SetDarkMode ({ commit }, darkMode) {
       commit('SET_DARK_MODE', darkMode)
     },

ui/src/store/mutation-types.js

@@ -35,6 +35,7 @@ export const USE_BROWSER_TIMEZONE = 'USE_BROWSER_TIMEZONE'
 export const SERVER_MANAGER = 'SERVER_MANAGER'
 export const DOMAIN_STORE = 'DOMAIN_STORE'
 export const DARK_MODE = 'DARK_MODE'
+export const LATEST_CS_VERSION = 'LATEST_CS_VERSION'
 export const VUE_VERSION = 'VUE_VERSION'
 export const CUSTOM_COLUMNS = 'CUSTOM_COLUMNS'