CaringCaribou / caringcaribou

A friendly car security exploration tool for the CAN bus
GNU General Public License v3.0
751 stars 197 forks source link

filter extraneous DIDs + valid testing #103

Closed alexdetrano closed 9 months ago

alexdetrano commented 10 months ago

latest code that ensures we only respond to DIDs that we requested. Testing infrastructure has been updated to take this into account. The big change here was updating the mock ECU to respond properly to a DID request. It was not responding with the requested DID and causing tests to fail.

I also rolled in some code cleanup around checking the length of the DID to avoid multiple indentations from nested if statements.

kasperkarlsson commented 9 months ago

Thank you so much for this contribution! 💪🥳