CryptoAnalysis icon indicating copy to clipboard operation
CryptoAnalysis copied to clipboard

Reorganize CrySL Architecture

Open AnakinRaW opened this issue 6 years ago • 1 comments

As discussed in today's meeting:

  • [x] Perform a relabeling fro CrypSL to CrySL java source files
  • [ ] Extract and move (where possible) CrySL specific code to CrySL Repo (e.g.: such as CrySLRulesetSelector, etc.)
  • [ ] Move relevant constants/enums (like the CrySL source file ending) into a separate file

In edit for my suggestion about to referencing CrySL in this Repo: I didn't look carefully enough what exactly was referenced. I was under the assumption that this repo depends on the Ruleset Repo. As this obviously is not the case my comment becomes obsolete. Depending on the CrySL Engine of course makes sense. I'd still agree however to minimize the referenced code basis by splitting the CrySL-Repo into more independent projects

AnakinRaW avatar Nov 08 '19 15:11 AnakinRaW

Since this issue may become relevant in the future, the CrySL specific code concerns all classes in the packages crypto.rules and crypto.cryslhandler. In total, reading the rules is not part of the actual analysis. Instead, the analysis takes a Collection of CrySLRule as input and runs the analysis with them. One can obtain a collection using the RulesetReader. Therefore, the following new tasks arise:

  • [ ] Move the crypto.rules and crypto.cryslhandler into the CrySL Repository (probably into a new submodule)
  • [ ] Move the tests for reading the rules into the CrySL Repo
  • [ ] Remove not needed dependencies from this repository (mainly XText)
  • [ ] Document in CrySL Repository how to use the RulesetReader and obtain the Ruleset from a directory or Zip file

smeyer198 avatar Jul 15 '24 12:07 smeyer198