jcryptool / crypto

JCrypTool Crypto Plug-ins
https://www.cryptool.org
Eclipse Public License 1.0
68 stars 39 forks source link

Check all crypto plugins before release #287

Closed grthor closed 4 years ago

grthor commented 4 years ago

Before Release 1.0, the correct function of all plug-ins should be checked.

In the following list all plugins are listed. After you have checked one plugin, you can check it off here, so all others will see which plugins are already processed and there will be no double checks.

Here is a little check list that you can use:

Analysis

Just enter your name in the appropriate column if you have checked or fixed a plugin.

Plugin geprüft von überarbeitet von Kommentar
Entropy Analysis @grthor @grthor Kein Titel und Beschreibung, aber sonst gut. Ich würde es so durchgehen lassen. Das Plugin ist in sich schlüssig.
Frequency Analysis @grthor @grthor (Y-Achse des Graphen verschiebt sich nach oben beim verschieben des Graphen ("Mehr Optionen" auswählen, "Referenzverteilung" aktivieren, Referenztext ändern, Graph verschieben dann tritt das Problem auf) fixed with https://github.com/jcryptool/crypto/commit/9ad835c31529a3a9f121a2d997e3e149a6413f93
Friedman Test @grthor @grthor
Grille Analysis @grthor @grthor
Substitution Analysis @grthor @grthor
Transposition Analysis @grthor @grthor
Vignère Breaker @grthor @grthor Tooltips on the 2nd and 3rd page have random positions fixed with https://github.com/jcryptool/crypto/commit/8a6d35bcb1f5060305811cc2088e01060935e7b6 y-Achse verschwindet bei der Häufigkeitsanalyse (vgl. Frequency Analysis Plugin Problem) fixed with https://github.com/jcryptool/crypto/commit/9ad835c31529a3a9f121a2d997e3e149a6413f93)
Viterbi Analysis @grthor @grthor

Visuals

Just enter your name in the appropriate column if you have checked or fixed a plugin.

Plugin geprüft von überarbeitet von Kommentar
Android Unlock Pattern @grthor @grthor
Ant Colony Optimization @grthor @grthor Letters in the visualization overlap. They are not readable -- fixed
ARC4 / Spritz @grthor @grthor
Certificate Verification @grthor @grthor Changes from https://github.com/jcryptool/crypto/issues/261 included
Chinese Remainder Theorem @grthor @grthor
Diffie-Helman Key Exchange @grthor @grthor
ElGamal Cryptosystem fix before 1.0
Elliptic Curve Calculations @be4 @grthor
Extended Euclidean / Reciprocal Subtraction @grthor @grthor
Extended RSA Cryptosystem @grthor @grthor
Feige Fiat Shamir @grthor @grthor
Fiat Shamir @grthor @grthor
Graph Isomorphism @grthor @grthor
Grille @simlei, @grthor @grthor UTF8, ~resize~, ~minor stuff~
Hash Sensitivity @simlei, @grthor @grthor scrolling, ~context help~
Homomorphic encryption @simlei, @grthor one possible functional bug, rest only minor issues
Huffman Coding @simlei, @grthor ~Context help~, Malfunction (UTF8)
Inner States of the Data Encryption Standard @simlei @simlei ~Context help~, intuitiveness, but overall OK
Kleptography @simlei @simlei ~minor context help issue~, rest ok
Magic Door @simlei, @grthor @grthor pretty good, minor context help problems -- @grthor context help problems fixed
Merkle Signature (XMSS^MT) @simlei) @simlei, @grthor pretty good, awesome graph! :open_mouth:
Merkle-Hemlman Knapsack Cryptosystem @simlei @simlei, @grthor
Multipartite Key Exchange @grthor No context and online help
RSA Cryptosystem @simlei needs thorough fixing and re-testing
Shamirs Secret Sharing (ctd. https://github.com/jcryptool/crypto/issues/291) @simlei @simlei Prio: ?? medium/high + low ??
Shanks Babystep-Giantstep @simlei @simlei Prio: ?? medium (~context help~) + low ??
Signature Demonstration @simlei @simlei Prio: ?? medium (~context help~, orphaned shells) + low ??
Signature Verification @simlei @simlei Prio: ?? medium: orphaned shells, ~context help~, low (focus) ??
Simple Power Analysis / Square and Multiply @simlei @grthor ~Prio: medium for bad scaling in a visual; low/medium(?): textual issues~ fixed with https://github.com/jcryptool/crypto/commit/b745c4162a65dc24c4d06d840e6894dc80b7bfb7 by @grthor
SPHINCS Signature @simlei Prio: low/medium ~context help~
SPHINCS+ Signature @simlei Prio: medium/high visualization + ~context help~
SSl / TLS Handshake @simlei Prio: medium/high bug/glitch**, low/medium textual/~context help~
Verifiable Secret Sharing @simlei @simlei no issues at all found :tada:
WOTS / WOTS+ @simlei Prio: medium for badly-scaled visual

Games

Just enter your name in the appropriate column if you have checked or fixed a plugin.

Plugin geprüft von überarbeitet von Kommentar
Divider Game @tassadarius @tassadarius fixed
Number Shark @tassadarius Prio: medium fix the optimizer output
Sudoku Solver @tassadarius Prio: medium/high fix the StackOverflow error Plugin kommt nicht ins Release 1.0!
Zudo-Ku
simlei commented 4 years ago

El Gamal:

Resolution: reasonably resolve all problems before v1.0

Open issues:

Misc. observations:

simlei commented 4 years ago

Shamirs Secret Sharing

continued here: https://github.com/jcryptool/crypto/issues/291

Preliminary verdict:

English

Both languages

1) [x] Online help (theory, instruction)

Textual:

German

1) [x] online help (theory, instruction)

I1)

Shamir1

I2)

dplnknhcoapdceja

Leikani commented 4 years ago

I can look at at the online help.

Shamirs Secret Sharing

Possibly just Linux: WOTS+ visualization on the right does not work What does Shamir's Secret Sahring have to do with WOTS+?

Best regards, BE

Am 27.03.2020 um 13:51 schrieb Simon Leischnig:

Shamirs Secret Sharing

English

  • Possibly just Linux: WOTS+ visualization on the right does not work (System: Linux, GTK3)

[x] 1) Online help (theory, instruction)

  • [x] has context help
  • [x] has online help
  • [x] has toolbar button that shows context help
  • [o] do online help images scale with page width? [x] 2) Walk-through major functions
  • all fine except the WOTS+ visualization on the right (System: Linux, GTK3)
  • see I1) [x] 3) Walk-through, different order [x] 4) Typos [x] 5) Resize artifacts
  • no other than that in 2) [x] 6) Layout consistency: e.g. Buttons and groupings vertical, horizontal alignment [x] 7) Dialogs: autofocus on first field?

Textual:

  • Online Help:
    • "Here is shown the numerical mode."
    • "By pressing the Select button opens a dialog box "
    • "Press the OK button the dialog box is closed and the polynomial represented as a formula."
    • "The user has now the opportunity There to generate a prime "
    • ... more
    • Online Help is nicely formatted except for the images which are always fixed-width. Better would be percentage of page width.
    • Online Help in English is not ideal as for sentence construction, grammar.

German

Textual:

[x] 1) online help (theory, instruction)

  • [x] has context help
  • [x] has online help
  • [x] has toolbar button that shows context help
  • [o] do online help images scale with page width? [x] 2) walk-through major functions
  • same problem as in English [x] 3) walk-through, different order [x] 4) Typos [x] 5) Resize artifacts [x] 6) Layout consistency: e.g. Buttons and groupings vertical, horizontal alignment [x] 7) Dialogs: autofocus on first field?

I1)

Shamir1

simlei commented 4 years ago

Shanks Babystep-Giantstep

Preliminary Verdict:

English

1) [x] Online help (theory, instruction)

Textual:

(to be fixed in a wider-scope effort)

German

1) [x] online help (theory, instruction)

Signature Demonstration

I1)

babystep

simlei commented 4 years ago

@Leikani

I can look at at the online help.

Thanks!

What does Shamir's Secret Sahring have to do with WOTS+?

as shown in the referenced .gif, on the right there are WOTS+-labeled radiobuttons, and on their right there is a visualization area that doesn't paint. Edit disregard that, these radio buttons are part of the described graphics artifact with the hanging visual. It was framebuffered over from another view.

simlei commented 4 years ago

Signature Demonstration

Preliminary verdict:

Both

English

1) [x] Online help (theory, instruction) 2) [x] Walk-through major functions

Textual:

German

1) [x] online help (theory, instruction)

Textual:

I1)

signaturedemo1

be4 commented 4 years ago

In your gif, the graph with the WOTS shouldn't be there. Maybe a pointer reference problem. There should be a polynom graph.

Under Windows (see screenshot below) I see the polynom graph, but it is wrong: As shown in the online help it's a smooth graph, but here it has edges. And the checked shares should be on the graph and not besides it.

Here a correction seemed to have destroyed something.

There seems to be Am 27.03.2020 um 15:53 schrieb Simon Leischnig:

@Leikani

I can look at at the online help. Thanks!

What does Shamir's Secret Sahring have to do with WOTS+? as shown in the referenced .gif, on the right there are WOTS+-labeled radiobuttons, and on their right there is a visualization area that doesn't paint

simlei commented 4 years ago

Under Windows (see screenshot below) I see the polynom graph

OK so probably a Linux/GTK specific bug. I'd say this is something to be fixed until v1.0 even if not all OS are affected.

[...] but it is wrong: As shown in the online help it's a smooth graph, but here it has edges. And the checked shares should be on the graph and not besides it. Here a correction seemed to have destroyed something.

This should be another issue; I am taking this to the original comment. Please feel free to attach a screenshot there, too.

simlei commented 4 years ago

Signature Verification

Preliminary verdict:

Both languages

English

1) [x] Online help (theory, instruction)

Textual:

German

1) [x] online help (theory, instruction)

I1)

signatureverification

simlei commented 4 years ago

SPHINCS Signature

Preliminary verdict

English

1) [x] Online help (theory, instruction)

German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

SPHINCS+ Signature

Preliminary verdict:

Both languages

English

Textual:

1) [x] Online help (theory, instruction)

German

1) [x] online help (theory, instruction)

I1)

sphincs+

I2)

sphincs+_empty

simlei commented 4 years ago

SSl / TLS Handshake

Preliminary verdict

Both languages

English

1) [x] Online help (theory, instruction)

Textual:

German

1) [x] online help (theory, instruction)

Textual:

I1)

ssl_tls_hs1

I2)

ssl_tls_hs2

I3)

ssl_tls_hs3

simlei commented 4 years ago

Verifiable Secret Sharing

Preliminary verdict:

English

1) [x] Online help (theory, instruction)

German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

WOTS / WOTS+

Preliminary verdict:

Both languages

English

1) [x] Online help (theory, instruction)

German

1) [x] online help (theory, instruction)

I1)

wots1

simlei commented 4 years ago

Simple Power Analysis / Square and Multiply

Preliminary verdict:

English

1) [x] Online help (theory, instruction)

Textual:

German

1) [x] online help (theory, instruction)

Textual:

I1)

squaremulti1

I2)

after executing with insufficient input, only a part of the usual results appears (empty input -> no op?), but nothing notifies the user of the limited value to be had from that result.

squaremultiply2

tassadarius commented 4 years ago

[Game] Divider Game

:heavy_check_mark: looks good :yellow_heart: may need some love :red_circle: critical bug, should be fixed

Update: https://github.com/jcryptool/crypto/pull/290 Tested Platforms:


UI :heavy_check_mark:

Problems

English

Online help (only background, little instruction, quite short) :heavy_check_mark::

Plugin :heavy_check_mark:

German

Online help :heavy_check_mark::

Plugin :heavy_check_mark:

Textual Problems

tassadarius commented 4 years ago

[Game] Number Shark

:heavy_check_mark: looks good :yellow_heart: may need some love :red_circle: critical bug, should be fixed

UI :heavy_check_mark: :

Problems

English

Online help (very detailed) :yellow_heart: :

Plugin :red_circle:

Problems

Textual Problems

German

Online help (very good and detailed online help) :yellow_heart::

Plugin :red_circle: :

Problems

simlei commented 4 years ago

@tassadarius that three-colour system is nice, I think I'll remaster my reviews to use these some time the next days :)

simlei commented 4 years ago

@ [Game] Number Shark

Does not look dead to me

Leikani commented 4 years ago

I have a look at it. The link works, but within JCT it doesn't.

Am 28.03.2020 um 23:47 schrieb Simon Leischnig:

simlei commented 4 years ago

I have a look at it. The link works, but within JCT it doesn't.

I'm already tracking that: https://github.com/jcryptool/crypto/issues/289#issuecomment-605257525

It seems there is a way to do external links that at least under linux they work. It also seems that there is a way to to this that they don't. Not necessarily the same under every OS.

Best place to discuss systematic issues with the plug-ins is there (where it will get sorted into single issues in time) or in the existing issues.

tassadarius commented 4 years ago

[Game] Sudoku Solver

:heavy_check_mark: looks good :yellow_heart: may need some love :red_circle: critical bug, should be fixed

UI :yellow_heart: to :heavy_check_mark: :

Problems

English

Online help :yellow_heart::

Problems

Plugin :red_circle: to :yellow_heart: :

Problems

German

Online help :yellow_heart::

Problems

Plugin :red_circle: to :yellow_heart: :

Problems

For some problems see the English problems also

simlei commented 4 years ago

Huffman Coding

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Labels/Textfields/Buttons:

Online Help

Malfunction:

Text:

Dies ist ein kurzer Beispieltext
in different Languages
with fûnæ ©ħa¶æ¢ŧ€®ẞ

Cut-off labels huffman1 Huffman2

simlei commented 4 years ago

Inner States of DES

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Online Help

isdes

simlei commented 4 years ago

Kleptography

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

Merkele Signature

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

Merkle-Helman Knapsack Cryptosystem

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

Grille

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Malfunction:

Dies ist ein kurzer Beispieltext
in different Languages
with fûnæ ©ħa¶æ¢ŧ€®ẞ

Grille_resize

grille_1

image

simlei commented 4 years ago

Hash Sensitivity

Preliminary verdict:

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

simlei commented 4 years ago

RSA Cryptosystem

Preliminary verdict:

--> Thorough test and bugfix warranted.

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Malfunction:

rsa1

rsa2


rsa2


rsa1

simlei commented 4 years ago

Magic Door

Preliminary verdict:

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Labels/Textfields/Buttons:

magicdoor1

simlei commented 4 years ago

Homomorphic Encryption

Checklist English

1) [x] Online help (theory, instruction)

Checklist German

1) [x] online help (theory, instruction)

Notes

Labels/Textfields/Buttons:

Malfunction:

homomorph1 homomorphic2

simlei commented 4 years ago

I have left this plugin in the milestone tracker for v1.0. However, I would like to close it soon! It served well as a testing organization place, but it is not ideal to separate out assignments of people to single tasks and tracking progress.

I tried to move all unfinished work from this issue into their own issues. If something is missing, please add it to the resp. issue.

grthor commented 4 years ago

Would agree on closing this issue.

It has served its purpose, but has now become too big.

It is much better to handle the remaining problems in individual issues.