Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343706

Summary: improve logging / error messages from CertificateChecker
Product: [Eclipse Project] Equinox Reporter: Volker Wegert <eclipse>
Component: p2Assignee: Pascal Rapicault <pascal>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: pascal
Version: unspecified   
Target Milestone: Juno M2   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
proposed patch
pascal: iplog+
example of trace output after applying the patch none

Description Volker Wegert CLA 2011-04-24 12:27:52 EDT
Build Identifier: 20110301-1815

If something goes wrong when checking the certificates (org.eclipse.equinox.internal.p2.engine.phases.CertificateChecker), virtually no information is available about the cause of the problem. In my case, it took me forever to find out why the p2.director of a headless build was unable to install bundles. 

Proposal: Add another trace switch (similar to org.eclipse.equinox.p2.core/generator/parsing, ...) and log all unsigned and untrusted objects in CertificateChecker.checkCertificates() before returning the status if that trace option is set. 

Reproducible: Always
Comment 1 Volker Wegert CLA 2011-04-24 18:14:50 EDT
Created attachment 193974 [details]
proposed patch
Comment 2 Volker Wegert CLA 2011-04-24 18:15:16 EDT
Created attachment 193975 [details]
example of trace output after applying the patch
Comment 3 Pascal Rapicault CLA 2011-05-17 20:14:38 EDT
Hi, I'm sorry to only get to this now. It felt through the cracks. Would you be able to provide an updated version of the patch since it does not seem to apply anymore. Thx in advance.
Comment 4 Pascal Rapicault CLA 2011-08-12 19:07:38 EDT
I managed to apply the patch.
Comment 5 Pascal Rapicault CLA 2011-08-12 19:08:13 EDT
Comment on attachment 193974 [details]
proposed patch

Thx for the contribution