search

iceman1001 / ChameleonMini-rebootedGUI

Windows based GUI for Chameleon Mini, the contactless smartcard emulator (NFC/RFID)
GNU General Public License v3.0
290 stars 72 forks source link

draft of v1.3 #111

Closed iceman1001 closed 4 years ago

iceman1001 commented 4 years ago

A draft for v1.3 release of the GUI

@shinhub @bogiton @slurdge

Feel free to update the text to cover all changes.

iceman1001 commented 4 years ago

After testing a bit, I must say the user experience left me with a bad taste in my mouth. The gui was slow. Really slow. The selected slot jumped between operations, the detection code didn't work, the device went offline / online. It was a bizarr experience. Now if this depends on device fw or gui or a combi I dont know. Regardless this needs more proper testing.

securechicken commented 4 years ago

@iceman1001 I think I fixed most of slowness (#112) as well as a slot index selection issue. But this one should still be slower than oldest GUIs because: 1) it checks UID and UIDSIZE based on real FW content to ensure we set a consistent one when switching from confs, most notably from 4 to 7bytes UID (before that, the GUI sometimes appeared with an UID that was not the one in FW memory) 2) clear now really clears whole slot memory (where before, it depended on which bug you triggered), and sometimes slots memories are bigger than before on FW (so, more memory to clear)

As for DETECTION, I think it it should be from FW side, I will look into it. But anyway, DETECTION has always been a pain lately.

securechicken commented 4 years ago

Plus there were some bugs in original code on Serial data reading, that were very visibly triggered (introducing delay till a 1s manual timeout is reached) by an improper use of "SendCommand" I introduced when checking UID sizes. See #113. This should speed everything back to when checks were not made on UID, slots were not reset consistently with FW, and all memory was not used/cleared.

iceman1001 commented 4 years ago

@shinhub Don't forget to add lines in Changelog.md...

iceman1001 commented 4 years ago

@shinhub @grspy @bogiton
With @grspy 's commits we are getting to a much smoother experience. How about you try it out and give some feedback on it?

iceman1001 commented 4 years ago

Thanks to @merlokk we now have Appveyor CI builds. I hooked up the badge on readme, looking good.