Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Question]: Trusted Signing settings ignored in settings.json file #1285

Open
mcebailly opened this issue Oct 30, 2024 · 7 comments
Open

[Question]: Trusted Signing settings ignored in settings.json file #1285

mcebailly opened this issue Oct 30, 2024 · 7 comments
Labels
Fix Ready Fix Ready question Further information is requested

Comments

@mcebailly
Copy link

AL-Go version

6.0

Describe the issue

I have run the new trusted signing build with settings and they are being ignored.

My current sign process continues to state:

Invoking signing tool using clientId/clientSecret

I've added the following settings (my params x'd out) to my settings.json inside my .AL-Go folder. I've tried all of the casing options including TrustedSigning, trustedSigning and capital letters for Endpoint, Account, CertificateProfile etc. Am I doing something wrong here?

"trustedSigning": {
"endpoint": "xx"
"account": "xx",
"certificateProfile": "xx"
}

Expected behavior

The trusted signing section should be read into the Sign process to sign a trusted app file

Steps to reproduce

Add a trustedSigning section to settings.json under .AL-Go folder

Run a CI/CD on an AppSource app

Additional context (logs, screenshots, etc.)

No response

@mcebailly mcebailly added the bug Something isn't working label Oct 30, 2024
@freddydk
Copy link
Contributor

Did you run Update AL-Go System Files with AL-Go preview?
Please supply a log of the build for us to see what might be wrong.

It should also work with the project settings, but I can see that I added it under repo setting here:

https://github.com/microsoft/bcsamples-bingmaps.appsource/blob/main/.github/AL-Go-Settings.json

@mcebailly
Copy link
Author

Log file attached, can confirm I pulled preview logs_30302768542.zip

@freddydk
Copy link
Contributor

freddydk commented Oct 30, 2024

Looking in the logs, it says: microsoft/AL-Go-Actions/[email protected]

Which is v6.0 and not preview.

preview would be: microsoft/AL-Go/Actions/Sign@<ref> (where ref is the github ref of the preview build)

@mcebailly
Copy link
Author

mcebailly commented Oct 31, 2024 via email

@freddydk
Copy link
Contributor

You need to run Update AL-Go System Files with microsoft/AL-Go-AppSource@preview
You should not change the .yaml files yourself.

@mcebailly
Copy link
Author

This got me a lot further. It is now showing trusted signing in the Sign build with success.

When I download the artifact, it does not appear to be signed appropriately though. I would expect to see a tab that says “Digital Signature” correct?

@freddydk
Copy link
Contributor

No, you would only see that if you have the NAVSIP.dll installed on your computer.

If the sign step passes and you have something like this in the log:

  Invoking signing tool using trusted signing
  info: Sign.Core.ISigner[0]
        Submitting D:\a\bcsamples-bingmaps.appsource\bcsamples-bingmaps.appsource\Library Apps\.buildartifacts\Apps\Freddy Kristiansen_FreddyDK.Licensing_4.8.207.0.app for signing.
  info: Sign.Core.ISigner[0]
        SignAsync called for D:\a\bcsamples-bingmaps.appsource\bcsamples-bingmaps.appsource\Library Apps\.buildartifacts\Apps\Freddy Kristiansen_FreddyDK.Licensing_4.8.207.0.app. Using C:\Users\runneradmin\AppData\Local\Temp\txl3onlb.dsx\biufpkrp.app locally.
  info: Sign.Core.IDataFormatSigner[0]
        Signing SignTool job with 1 files.
  info: Sign.Core.IDataFormatSigner[0]
        Signing C:\Users\runneradmin\AppData\Local\Temp\txl3onlb.dsx\biufpkrp.app.
  info: Sign.Core.IDataFormatSigner[0]
        Signing C:\Users\runneradmin\AppData\Local\Temp\txl3onlb.dsx\biufpkrp.app succeeded.
  info: Sign.Core.ISigner[0]
        Completed in 2796 ms.

then you are good.

@freddydk freddydk added question Further information is requested and removed bug Something isn't working labels Oct 31, 2024
@freddydk freddydk changed the title [Bug]: Trusted Signing settings ignored in settings.json file [Question]: Trusted Signing settings ignored in settings.json file Oct 31, 2024
@freddydk freddydk added the Fix Ready Fix Ready label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Ready Fix Ready question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants