Concordium / concordium-desktop-wallet

Desktop wallet for the Concordium blockchain. For Windows, macOS and Linux.
Apache License 2.0
2 stars 3 forks source link

Identity issuance crashes #296

Closed shjortConcordium closed 2 years ago

shjortConcordium commented 2 years ago

Bug Description

When entering the external flow in the identity issuance, this error appears: image (2)

(Happens consistenly in @mh-concordium's mac)

Versions

orhoj commented 2 years ago

@mh-concordium does this also happen to you on the current release (1.3.1)?

mh-concordium commented 2 years ago

I would have to try this specific release. I have used the previous one made for Delegation and that was working.

orhoj commented 2 years ago

What version was that?

mh-concordium commented 2 years ago

I believe that the failing one (this issue) was 1.4.0-beta.2. Working one I have tested before were 1.4.0-beta.0 and 1.4.0-alpha.0.

shjortConcordium commented 2 years ago

A user have encountered this error on 1.3.1 (macOS)

abizjak commented 2 years ago

Please supply steps/context to reproduce, or close if the issue is deemed to be resolved.

tschudid commented 2 years ago
bugreport

Similar issue after upgrading from DW 1.3.1 to 1.4.1 on macos (intel).

orhoj commented 2 years ago

This happens due to the way windows are handled on macOS. If you close the main window (e.g. by pressing the X in the upper-left corner), then we have special macOS handling that keeps the application alive as this is the standard on macOS. However, as the window is killed when closed we now have a bunch of listeners with a reference to a destroyed window. This is why we get an error when trying to register a duplicate handler, and it is also the issue behind the image shown in this issue as it refers to a destroyed object.

mh-concordium commented 2 years ago

Verified on MacOS 1.4.2. First I have successfully reproduced both issues on v. 1.4.1, afterwards I have installed 1.4.2 and all seemed ok. I didn't see warning reported by Daniel neither warning I saw before. I was also able to create new identity on 1.4.2