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

Merge pull request #266 from guardicore/hotfix/remove-unecessary-set-state 

Some unecessary set states removed to prevent refreshing of config page
This commit is contained in:
itaymmguardicore 2019-02-17 14:08:54 +02:00 committed by GitHub
parent 1fa31340ca d67cb18cae
commit ee7c9d9ea8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
monkey/monkey_island/cc/ui/src/components/Main.js
View File

@ -29,9 +29,11 @@ class AppComponent extends AuthComponent {
updateStatus = () => {
this.auth.loggedIn()
.then(res => {
this.setState({
isLoggedIn: res
});
if (this.state.isLoggedIn !== res) {
this.setState({
isLoggedIn: res
});
}
if (res) {
this.authFetch('/api')

9
monkey/monkey_island/cc/ui/src/components/pages/ConfigurePage.js
View File

@ -141,9 +141,12 @@ class ConfigurePageComponent extends AuthComponent {
.then(res => res.json())
.then(res => {
// This check is used to prevent unnecessary re-rendering
this.setState({
allMonkeysAreDead: (!res['completed_steps']['run_monkey']) || (res['completed_steps']['infection_done'])
});
let allMonkeysAreDead = (!res['completed_steps']['run_monkey']) || (res['completed_steps']['infection_done']);
if (allMonkeysAreDead !== this.state.allMonkeysAreDead) {
this.setState({
allMonkeysAreDead: allMonkeysAreDead
});
}
});
};