Fix ShortestPathAttacker

[mrkickling]

It is broken, finds paths that are not possible.

[hidde-jan]

I just ran into an issues where the ShortestPathAttacker would always choose the same action in https://github.com/mal-lang/mal-simulator/blob/main/examples/example_find_low_ttc_path.py

Iteration 11766
Agent Attacker1 chose action: Host:0:access

[mrkickling]

Interesting, did you run that file as is? Which version of the simulator?

I can not reproduce that issue when running the file.

[mrkickling]

Oh, the ShortestPathAttacker. Agreed, I get the same problem and the policy is not working as it should. I believe it is missing some checks about a step being legal since it should need to run authenticate before it runs access. Will take a look into it.