I am currently on vacation and will come back to this PR soon.

Understood.
Enjoy your time off! :)

If someone can start with some initial rewiew and testing it would be highly appreciate.

I will begin work on the requested changes.
Since this goes out of scope of my Bachelors Thesis, it might take a bit longer tho as I will be working on it in my free time.
Initially, I did not plan on adding any Python modules to EMBA permanently, just to implement an easy method
to run custom ones when required, but I will gladly expand the module to support this properly.
All minor changes (colorization, name changes, ...) should not take very long.

I will also update the copyright notices accordingly, I did not change them due to a misunderstanding on my end.

Please feel free to contact me for any additional questions or requirements.

To clarify some things:

For testing: where/how do I install pip packages?( does the venv work with the scripts)

As it is currently, the module uses EMBAs venv. PIP packages should work when installed into this environment.
However, I have not tested this. Will confirm that it works.
I also do not know whether this venv even should be used, or whether I need to create a custom one
for this module. That decision is up to you.

Why ist the PYTHON_SCRIPTS array not local?

The PYTHON_SCRIPTS array is not local because it is set in the EMBA profile.
It's supposed to allow a user to specify what Python scripts they want to run.
I can add a local version of the array with default settings, which can then be overwritten by the profile setting.
Would be similar to the SELECT_MODULES arrays functionality.

I am not completely sure how we should handle the logging and we need to link to the results from the python modules.

Right now, the Bash module reads the results from STDOUT of the Python script (each line = 1 results).
An alternative solution would be to use grep (or a similar tool) to parse the results from the Python scripts own log file.
This would require some formatting changes in the embamodule.py output to make it grep-able, but
should not take too much work.

Finally, we need a way to setup a virtual environment dynamically for the container to run the python scripts. Probably we can create a virtual environment in the external directory and mount it via docker-compose into the container.

Instead of using EMBAs venv, this might be possible. I will look into it.

Since this goes out of scope of my Bachelors Thesis, it might take a bit longer tho as I will be working on it in my free time. Initially, I did not plan on adding any Python modules to EMBA permanently, just to implement an easy method to run custom ones when required, but I will gladly expand the module to support this properly.

No hurry with this. I think this could be a very interesting addon for EMBA. Nevertheless, we need to design it in a way it will also fulfill the expectations into a python integration for EMBA.

As the next big change I want to change the logging method around.

As mentioned before, I plan on making the output from the python module grep-able, so it is easier
to determine what is a finding, what is just debugging output and what is an error.
The goal is to add a better result counter that does not have to rely on the amount of lines output by a Python module.

I could either implement this by reading STDOUT from each Python module directly like I am doing at the moment, or
by parsing the module specific log file after it has finished.

Is there a preferred method to go about this?

As the next big change I want to change the logging method around.

As mentioned before, I plan on making the output from the python module grep-able, so it is easier to determine what is a finding, what is just debugging output and what is an error. The goal is to add a better result counter that does not have to rely on the amount of lines output by a Python module.

Probably it makes sense to log from the python modules as from the main EMBA modules. All relevant output into a text file that we can then link from the main EMBA s28 module. With this we should be able to easily generate the html report page. I think this could be around line 37 of your module. There we will need a write_link after your print_output. This links to the original log file from your python module.

The grep-able thing is additionally a useful mechanism.

I could either implement this by reading STDOUT from each Python module directly like I am doing at the moment, or by parsing the module specific log file after it has finished.

Is there a preferred method to go about this?

See my notes before. I would recommend to log all relevant details in a nice way to a dedicated log file and link to this log file via write_link from the main s28 module.

That makes sense, I did not know about the write_link function.

That makes sense, I did not know about the write_link function.

The web reporter automatically picks it up and is doing the html stuff for you ... hopefully ;) You should see a link with the [REF] marker in the text report. See also here

I majorly screwed up my git history just now, which caused this pull request to close. :,)
All the changes are still backed up locally, so that is not too much of an issue.
I will see that I reopen a new pull request/issue as soon as I can, tho I might have to create a new fork of the repository.