https://pulp.plan.io/https://pulp.plan.io/favicon.ico2021-07-26T07:28:00ZPulpCertGuard - Issue #9140: RHSM certguard failure messages are lost in log-level debughttps://pulp.plan.io/issues/9140?journal_id=737432021-07-26T07:28:00Zttereshcttereshc@redhat.com
<ul></ul><p>BZ comments from ggainey:</p>
<blockquote>
<p>I suspect this is a combination of <a href="https://github.com/pulp/pulp-certguard/blob/master/pulp_certguard/app/models.py#L42" class="external">https://github.com/pulp/pulp-certguard/blob/master/pulp_certguard/app/models.py#L42</a> and <a href="https://github.com/pulp/pulp-certguard/blob/master/pulp_certguard/app/models.py#L167-L169" class="external">https://github.com/pulp/pulp-certguard/blob/master/pulp_certguard/app/models.py#L167-L169</a>.<br>
In the first, we only log "this doesn't even look like a cert" at debug-level.<br>
In the second, we lose information on the specific error encountered and log "something went wrong".</p>
</blockquote>
<blockquote>
<p>From discussion w/ Matthias:</p>
<p>certguard raises PermissionError, which is a base python error-class. These error-messages appear to not show up in logging. Investigation needed on whether PermissionError is "special" in some way when it comes to being logged by the content-app.<br>
NOTE: we really should not be overloadiung python's file-level PermissionError this way, and instead should have our own cecrtguard-perm-error. May want its own RFE.<br>
See <a href="https://github.com/pulp/pulpcore/blob/master/pulpcore/content/handler.py#L296-L303" class="external">https://github.com/pulp/pulpcore/blob/master/pulpcore/content/handler.py#L296-L303</a> for where we might want to increase log-level.</p>
</blockquote> CertGuard - Issue #9140: RHSM certguard failure messages are lost in log-level debughttps://pulp.plan.io/issues/9140?journal_id=782172022-01-04T12:41:05Zpulpbot
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/78217/diff?detail_id=78929">diff</a>)</li><li><strong>Status</strong> changed from <i>NEW</i> to <i>CLOSED - DUPLICATE</i></li></ul>