TrustPoint-Project / trustpoint-client

MIT License
4 stars 0 forks source link

Dev implicit init and merciless purge #7

Closed BytesWelder closed 1 week ago

BytesWelder commented 1 month ago

Notes Added implicit init and changed the purge method a bit. Also we are using platformdirs now.

Legal

BytesWelder commented 1 month ago

Thank you @Aircoookie for your thorough testing. Especially since you use Windows.

With the platformdirs we actually wanted to avoid the problem for different environments... Funny that we now have a problem because of it, since it worked fine on debian...

The Problem with the two different paths is that we can not set the init Path for devidModule. I will look into it, why there are two different inventories.

AlexHx8472 commented 1 month ago

It is actually intended to have two inventories in this case (if I understand it correctly). The devid-module is a completely separate package that uses its own inventory. It handles keys, certificates and corresponding certificate chains.

The inventory of the client is a layer on top and stores those references to the devid-module as well as separate collections of certificates as truststores.

Aircoookie commented 1 month ago

Thank you! Ah, it seems like I potentially confused the inventory file of the client with that of the devid module and it may be working as intended after all. Perhaps we should also add platformdirs to devid module so that the path structure is similar for both modules

AlexHx8472 commented 1 month ago

Yes, that was the idea to also add it to the devid-module, so both can run concurrently and consistently.