{"action":"created","issue":{"url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/588","labels_url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/588/labels{/name}","comments_url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/588/comments","events_url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/588/events","html_url":"https://github.com/bitcoin-solutions/multibit-hd/issues/588","id":84600813,"number":588,"title":"CrashReporter: Inserting Trezor with old firmware crashed MBHD","user":{"login":"jim618","id":919427,"avatar_url":"https://avatars.githubusercontent.com/u/919427?v=3","gravatar_id":"","url":"https://api.github.com/users/jim618","html_url":"https://github.com/jim618","followers_url":"https://api.github.com/users/jim618/followers","following_url":"https://api.github.com/users/jim618/following{/other_user}","gists_url":"https://api.github.com/users/jim618/gists{/gist_id}","starred_url":"https://api.github.com/users/jim618/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jim618/subscriptions","organizations_url":"https://api.github.com/users/jim618/orgs","repos_url":"https://api.github.com/users/jim618/repos","events_url":"https://api.github.com/users/jim618/events{/privacy}","received_events_url":"https://api.github.com/users/jim618/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/labels/awaiting%20review","name":"awaiting review","color":"fbca04"}],"state":"closed","locked":false,"assignee":{"login":"jim618","id":919427,"avatar_url":"https://avatars.githubusercontent.com/u/919427?v=3","gravatar_id":"","url":"https://api.github.com/users/jim618","html_url":"https://github.com/jim618","followers_url":"https://api.github.com/users/jim618/followers","following_url":"https://api.github.com/users/jim618/following{/other_user}","gists_url":"https://api.github.com/users/jim618/gists{/gist_id}","starred_url":"https://api.github.com/users/jim618/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jim618/subscriptions","organizations_url":"https://api.github.com/users/jim618/orgs","repos_url":"https://api.github.com/users/jim618/repos","events_url":"https://api.github.com/users/jim618/events{/privacy}","received_events_url":"https://api.github.com/users/jim618/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/milestones/13","html_url":"https://github.com/bitcoin-solutions/multibit-hd/milestones/Release%200.1","labels_url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/milestones/13/labels","id":1007213,"number":13,"title":"Release 0.1","description":"Next iteration to provide bug fixes and the following headline issues:\r\n\r\n* support for an integrated encrypted error reporting system","creator":{"login":"gary-rowe","id":342133,"avatar_url":"https://avatars.githubusercontent.com/u/342133?v=3","gravatar_id":"","url":"https://api.github.com/users/gary-rowe","html_url":"https://github.com/gary-rowe","followers_url":"https://api.github.com/users/gary-rowe/followers","following_url":"https://api.github.com/users/gary-rowe/following{/other_user}","gists_url":"https://api.github.com/users/gary-rowe/gists{/gist_id}","starred_url":"https://api.github.com/users/gary-rowe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gary-rowe/subscriptions","organizations_url":"https://api.github.com/users/gary-rowe/orgs","repos_url":"https://api.github.com/users/gary-rowe/repos","events_url":"https://api.github.com/users/gary-rowe/events{/privacy}","received_events_url":"https://api.github.com/users/gary-rowe/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":68,"state":"open","created_at":"2015-03-05T09:42:55Z","updated_at":"2015-06-03T20:44:31Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-06-03T14:44:10Z","updated_at":"2015-06-03T20:44:31Z","closed_at":"2015-06-03T20:44:31Z","body":"If you have MBHD running and insert a Trezor with old firmware, you see the 'old firmware alert' but then see the CrashReporter. Not only that but the CrashReporter is unusable as it keeps 'refreshing' over and over.\r\nIn the log is the error:\r\n```\r\n{\"@timestamp\":\"2015-06-03T15:41:18.739+01:00\",\"@version\":1,\"message\":\"Uncaught exception\",\"logger_name\":\"org.multibit.hd.core.error_reporting.ExceptionHandler\",\"thread_name\":\"AWT-EventQueue-0\",\"level\":\"ERROR\",\"level_value\":40000,\"stack_trace\":\"java.lang.IllegalStateException: LightBoxPopover should not be called unless a light box is showing\\n\\tat com.google.common.base.Preconditions.checkState(Preconditions.java:176) ~[multibit-hd.jar:na]\\n\\tat org.multibit.hd.ui.views.components.Panels.showLightBoxPopover(Panels.java:365) ~[multibit-hd.jar:na]\\n\\tat org.multibit.hd.ui.views.wizards.AbstractWizardPanelView.checkForEnvironmentEventPopover(AbstractWizardPanelView.java:454) ~[multibit-hd.jar:na]\\n\\tat org.multibit.hd.ui.views.wizards.welcome.WelcomeAttachHardwareWalletPanelView.access$100(WelcomeAttachHardwareWalletPanelView.java:33) ~[multibit-hd.jar:na]\\n\\tat org.multibit.hd.ui.views.wizards.welcome.WelcomeAttachHardwareWalletPanelView$2.run(WelcomeAttachHardwareWalletPanelView.java:211) ~[multibit-hd.jar:na]\\n\\tat \r\n```"},"comment":{"url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/comments/108610374","html_url":"https://github.com/bitcoin-solutions/multibit-hd/issues/588#issuecomment-108610374","issue_url":"https://api.github.com/repos/bitcoin-solutions/multibit-hd/issues/588","id":108610374,"user":{"login":"gary-rowe","id":342133,"avatar_url":"https://avatars.githubusercontent.com/u/342133?v=3","gravatar_id":"","url":"https://api.github.com/users/gary-rowe","html_url":"https://github.com/gary-rowe","followers_url":"https://api.github.com/users/gary-rowe/followers","following_url":"https://api.github.com/users/gary-rowe/following{/other_user}","gists_url":"https://api.github.com/users/gary-rowe/gists{/gist_id}","starred_url":"https://api.github.com/users/gary-rowe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gary-rowe/subscriptions","organizations_url":"https://api.github.com/users/gary-rowe/orgs","repos_url":"https://api.github.com/users/gary-rowe/repos","events_url":"https://api.github.com/users/gary-rowe/events{/privacy}","received_events_url":"https://api.github.com/users/gary-rowe/received_events","type":"User","site_admin":false},"created_at":"2015-06-03T20:44:31Z","updated_at":"2015-06-03T20:44:31Z","body":"Overall this is fine. I shortened some of the crazy-long variable names and added a bit at the end to ensure the application frame is disabled and the dialog is brought to the front in case people lose it.\r\n\r\nI verified the operation with a couple of well placed stacked exceptions and it behaved correctly.\r\n\r\nClosing."}}