[bisq-network/bisq] Improve startup routines (#4583)

chimp1984 notifications at github.com
Thu Oct 1 04:38:00 UTC 2020


Refactorings and cleanups of Bisq startup code.

Removed the crypto policy checks as it is not needed anymore as far I am aware of (not 100% sure though).
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/4583

-- Commit Summary --

  * Refactor: Move AsciiLogo to common
  * Refactor: extract method
  * Refactor: add new setup method to CommonSetup with config (WIP) and call it at doExecute. Move AsciiLogo to CommonSetup
  * Refactor: Rename setup to setupUncaughtExceptionHandler
  * Improve handling of UncaughtExceptionHandler
  * Refactor: Move sig int handlers to CommonSetup
  * Refactor: Move DevEnv setup to CommonSetup
  * Refactor: Add setup method in DevEnv
  * Refactor: Rename onUiReadyHandler to onApplicationStartedHandler
  * Refactor: Move periodic printSystemLoad to commonSetup
  * Refactor: Move AvoidStandbyModeService from BisqApp to BisqExecutable.setupAvoidStandbyMode
  * Remove empty loop
  * Remove checkCryptoSetup as not needed anymore
  * Remove checkCryptoPolicySetup as not needed anymore
  * Remove Remove LimitedKeyStrengthException as not needed anymore
  * Refactor: Rearrange methods
  * Refactor: Move common bases setup code to CommonSetup
  * Remove cryptoSetupFailedHandler
  * Refactor: Move osUserDataDir method to Utilities, rename to getUserDataDir
  * Refactor: Rename startAppSetup to runBisqSetup
  * Improve printSystemLoad
  * Refactor: Move handler code to domain classes

-- File Changes --

    R common/src/main/java/bisq/common/app/AsciiLogo.java (2)
    M common/src/main/java/bisq/common/app/DevEnv.java (7)
    M common/src/main/java/bisq/common/crypto/CryptoUtils.java (10)
    D common/src/main/java/bisq/common/crypto/LimitedKeyStrengthException.java (24)
    M common/src/main/java/bisq/common/setup/CommonSetup.java (73)
    M common/src/main/java/bisq/common/util/Profiler.java (15)
    M common/src/main/java/bisq/common/util/Utilities.java (16)
    M core/src/main/java/bisq/core/app/BisqExecutable.java (67)
    M core/src/main/java/bisq/core/app/BisqHeadlessApp.java (1)
    M core/src/main/java/bisq/core/app/BisqHeadlessAppMain.java (9)
    M core/src/main/java/bisq/core/app/BisqSetup.java (152)
    M core/src/main/java/bisq/core/app/WalletAppSetup.java (80)
    M core/src/main/java/bisq/core/app/misc/ExecutableForAppWithP2p.java (16)
    M core/src/main/java/bisq/core/filter/FilterManager.java (31)
    M core/src/main/java/bisq/core/setup/CoreSetup.java (29)
    M daemon/src/main/java/bisq/daemon/app/BisqDaemonMain.java (3)
    M desktop/src/main/java/bisq/desktop/app/BisqApp.java (13)
    M desktop/src/main/java/bisq/desktop/app/BisqAppMain.java (18)
    M desktop/src/main/java/bisq/desktop/main/MainView.java (4)
    M desktop/src/main/java/bisq/desktop/main/MainViewModel.java (5)
    M seednode/src/main/java/bisq/seednode/SeedNodeMain.java (2)
    M statsnode/src/main/java/bisq/statistics/StatisticsMain.java (4)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/4583.patch
https://github.com/bisq-network/bisq/pull/4583.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4583
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200930/12d82af4/attachment.html>


More information about the bisq-github mailing list