Skip to content

Commit

Permalink
increase "changeAccountOrder" api call timeout
Browse files Browse the repository at this point in the history
* 10 seconds might not be enough on low-performance systems in case of "sensitive" key derivation function used
  • Loading branch information
vladimiry committed Jul 31, 2018
1 parent 78847f8 commit ef3cf61
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/web/src/app/+options/options.effects.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import {ACCOUNTS_OUTLET, ACCOUNTS_PATH, SETTINGS_OUTLET, SETTINGS_PATH} from "sr
import {CORE_ACTIONS, NAVIGATION_ACTIONS, OPTIONS_ACTIONS} from "src/web/src/app/store/actions";
import {ElectronService} from "src/web/src/app/+core/electron.service";
import {getZoneNameBoundWebLogger, logActionTypeAndBoundLoggerWithActionType} from "src/web/src/util";
import {ONE_SECOND_MS} from "src/shared/constants";
import {OptionsSelectors} from "src/web/src/app/store/selectors";
import {OptionsService} from "./options.service";
import {ProgressPatch, State} from "src/web/src/app/store/reducers/options";
Expand Down Expand Up @@ -127,7 +128,7 @@ export class OptionsEffects {
map(logActionTypeAndBoundLoggerWithActionType({_logger})),
concatMap(({payload}) => merge(
of(this.buildPatchProgress({changingAccountOrder: true})),
this.electronService.ipcMainClient()("changeAccountOrder")({login: payload.login, index: payload.index}).pipe(
this.electronService.ipcMainClient()("changeAccountOrder", {timeoutMs: ONE_SECOND_MS * 20})(payload).pipe(
map((settings) => OPTIONS_ACTIONS.GetSettingsResponse(settings)),
catchError((error) => of(CORE_ACTIONS.Fail(error))),
finalize(() => this.dispatchProgress({changingAccountOrder: false})),
Expand Down

0 comments on commit ef3cf61

Please sign in to comment.