search

kine / NVRAppDevOps

Navertica DevOps scripts for managing Microsoft Dynamics 365 Business Central Apps
31 stars 29 forks source link

Update Compile-ALProjectTree.ps1 #2

Closed MJECloud closed 5 years ago

MJECloud commented 5 years ago

Added check if signtool is available else throw exception. Use full signtool path instead of system32 shortcut.

kine commented 5 years ago

Thanks! I am now using in Azure DevOps the Sign-NAVContainerApp from navcontainerhelper. I recommend to use it too. I will try to update the script to use the container to sign the app if possible to prevent need of installing the SDK.

kine commented 5 years ago

I moved the code into the if block to test the path only if $CertPath is set and thus the tool will be used.

MJECloud commented 5 years ago

Hello Kamil,

thanks for your quick response. You say that you already use Sign-NAVContainerApp. Do you know when you will publish that in your Code? Otherwise i can edit the module, if its ok.

Greetings, Maurice

kine commented 5 years ago

I have just published new version. If CertPath and CertPwd is filled in, the app is signed in container. If only CertPath is filled (no CertPwd - this means that it will use account permissions inside the certificate - new windows allows you to set permissions for domain accounts in the certificate) the app is signed on host, because navcontianerhelper does not support this (it is out of domain). Not yet tested...

MJECloud commented 5 years ago

Thank you! I'll give it a try today.