-
Notifications
You must be signed in to change notification settings - Fork 452
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
Signing the assembly for Windows platforms #3718
Comments
CEF doesn't provide signed binaries. Application distributors (e.g. you, or whomever created the application that you're using) should sign all binaries that are being distributed as part of the application. |
I've tried that but the libcef assembly is not signable for some reason (probably because of the missing PF data). |
I'm not seeing any issues signing libcef. Where are you getting the libcef DLL? Are you using CEF/C++ directly, or some wrapper project? |
It's possible we have an older version here, let me try fetching the latest binaries and try to sign them. [UPDATE] ILSpy reports that "PE file does not contain any managed metadata." |
That's correct. |
Understood. For a Windows user, what would be the correct way to sign the compiled binary since it is unmanaged? |
@magreenblatt Thank you! |
For those of us whose perform security audits (SSF, BlackDuck, etc), this DLL is always flagged because it is unsigned.
Can we get a signed version in the future?
Even ILSpy struggles with it:
![image](https://private-user-images.githubusercontent.com/81915293/341519295-d963bdfd-21f9-43ae-947a-75f19a253f85.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjM3NjYxNjYsIm5iZiI6MTcyMzc2NTg2NiwicGF0aCI6Ii84MTkxNTI5My8zNDE1MTkyOTUtZDk2M2JkZmQtMjFmOS00M2FlLTk0N2EtNzVmMTlhMjUzZjg1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODE1VDIzNTEwNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTUxOTkwZTc0NGMwMzliMjZiODI2OTQ5NWUzNWZmMzY0ZjZjMzA5MmE2NDFhZjQ1MjExNDYyNTMwZTk4NjIzY2EmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.dFm8CVJbvbQqLK_QxmRo4ZOSPa5XLkLQ-tlmU3nnLe0)
The text was updated successfully, but these errors were encountered: