### Beta/Release Uplift Approval Request * **User impact if declined**: The certificate viewer was reporting a wrong OCSP * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: Yes * **Needs manual test from QE?**: Yes * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: I think this patch is not risky since I changed an if statement * **String changes made/needed**:
Bug 1589883 Comment 14 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
### Beta/Release Uplift Approval Request * **User impact if declined**: The certificate viewer was reporting a wrong OCSP * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: I don't know * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: I think this patch is not risky since I changed an if statement * **String changes made/needed**:
### Beta/Release Uplift Approval Request * **User impact if declined**: The certificate viewer was reporting a wrong OCSP * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: I don't know * **Needs manual test from QE?**: I don't know * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: I think this patch is not risky since I changed an if statement * **String changes made/needed**:
### Beta/Release Uplift Approval Request * **User impact if declined**: The certificate viewer was reporting a wrong OCSP * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: I don't know * **Needs manual test from QE?**: I don't know * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: I think this patch is not risky since I changed an if statement * **String changes made/needed**: The OCSP section was showing the result of "Critical" as "Required", which is wrong; now, it only renders the section when `cert.ext.ocspStaple.required` is true
### Beta/Release Uplift Approval Request * **User impact if declined**: The certificate viewer was reporting a wrong OCSP * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: None * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: I think this patch is not risky since I changed an if statement * **String changes made/needed**: None.