p34709852
/
monkey
forked from p15670423/monkey
1
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

Revert "Island, UI: Handle the case when version couldn't be fetched" 

This reverts commit 737070f956.
This commit is contained in:
Mike Salvatore 2022-08-12 10:15:44 -04:00
parent 737070f956
commit 2e70b87ee9
2 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
monkey/monkey_island/cc/ui/src/components/logo/VersionComponent.js
View File

@ -28,11 +28,6 @@ class VersionComponent extends AuthComponent {
newerVersionAvailable() { newerVersionAvailable() {
const semverGt = require('semver/functions/gt'); const semverGt = require('semver/functions/gt');
if(this.state.latestVersion === null) {
// Couldn't fetch the latest version.
// Consider showing this state to the user
return false;
}
if(this.state.latestVersion !== undefined && this.state.versionNumber !== undefined) { if(this.state.latestVersion !== undefined && this.state.versionNumber !== undefined) {
return semverGt(this.state.latestVersion, this.state.versionNumber); return semverGt(this.state.latestVersion, this.state.versionNumber);
} }

4
monkey/monkey_island/cc/version.py
View File

@ -63,7 +63,7 @@ class Version:
response = requests.get(url, timeout=LATEST_VERSION_TIMEOUT).json() response = requests.get(url, timeout=LATEST_VERSION_TIMEOUT).json()
except requests.exceptions.RequestException as err: except requests.exceptions.RequestException as err:
logger.warning(f"Failed to connect to {url}, Error: {err}") logger.warning(f"Failed to connect to {url}, Error: {err}")
return None, None return self._version_number, None
try: try:
download_link = response["download_link"] download_link = response["download_link"]
@ -73,6 +73,6 @@ class Version:
f"Failed to fetch version information from {url}, response: {response}." f"Failed to fetch version information from {url}, response: {response}."
f"Most likely deployment {self._deployment.value} is not recognized." f"Most likely deployment {self._deployment.value} is not recognized."
) )
return None, None return self._version_number, None
return latest_version, download_link return latest_version, download_link