Closed Bug 1629590 Opened 4 months ago Closed 4 months ago

issues with standalone page for certificate viewer

Categories

(Firefox :: Security, defect, P1)

77 Branch
defect

Tracking

()

RESOLVED FIXED
Firefox 77
Tracking Status
firefox76 --- unaffected
firefox77 --- fixed

People

(Reporter: soeren.hentzschel, Assigned: johannh)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

In bug 1585620 a standalone page for the certificate viewer was implemented. But the implementation has several issues. This ticket is kind of a meta issue.

  1. It takes several seconds to load the tabs. For ~ 5 seconds only the headline is visible, nothing else.
  2. The caption of the first tab is "Unknown".
  3. The content of the first tab is completely empty (expected state is unknown because of 2).
  4. The content of the "Your Certificates" tab is completely empty (empty state is missing*).
  5. The content of the "People" tab is completely empty (empty state is missing*).
  6. The order of the tabs is different than on the preferences screen (should be unified).

*) On the preferences screen there is alo no "empty state" text but there is an empty table and it's clear that there are no entries. But a completely empty content area looks like an error because it's not clear if content is missing or not.

Thanks for reporting! We should probably fix some of these before going to release with the new standalone page :)

Blocks: cert-viewer
Priority: -- → P1
Assignee: nobody → jhofmann
Status: NEW → ASSIGNED
Flags: needinfo?(jhofmann)

These two patches should fix all of your issues, though I'm not 100% sure about the perf thing (#1). There was an error in the way we got the list of certificates which made us end up sending and rendering at least twice as many certs as we should have, so that likely had a perf impact, but it would be good if you could verify the improvement once this is in Nightly. If things are still slow please file a bug with a Firefox Performance Profile attached if possible (https://profiler.firefox.com/). Thanks!

Carolina, do you have time to review these two small-ish patches or should I find someone else? :)

Thanks!

Flags: needinfo?(carolina.jimenez.g)

Hey Johann! Yes!, I'll take a look soon :)

Flags: needinfo?(carolina.jimenez.g)
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ea828bc0282f
Correctly determine certificate types for the about:certificate standalone page. r=carolina.jimenez.g
https://hg.mozilla.org/integration/autoland/rev/7e982ad6250b
Don't show certificate categories on about:certificate if they're empty. r=carolina.jimenez.g
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77

Thanks for fixing the issues!

I'm not 100% sure about the perf thing (#1). There was an error in the way we got the list of certificates which made us end up sending and rendering at least twice as many certs as we should have, so that likely had a perf impact, but it would be good if you could verify the improvement once this is in Nightly.

Maybe there is a minor improvement, but it still takes several seconds on a Late 2018 MacBook Pro (2,6 GHz 6-Core Intel Core i7, 16 GB 2400 MHz DDR4).

(In reply to Sören Hentzschel from comment #9)

Thanks for fixing the issues!

I'm not 100% sure about the perf thing (#1). There was an error in the way we got the list of certificates which made us end up sending and rendering at least twice as many certs as we should have, so that likely had a perf impact, but it would be good if you could verify the improvement once this is in Nightly.

Maybe there is a minor improvement, but it still takes several seconds on a Late 2018 MacBook Pro (2,6 GHz 6-Core Intel Core i7, 16 GB 2400 MHz DDR4).

Huh, I have a similar MacBook but I can not reproduce that at all. Do you have a lot of custom certificates imported? Again, it would be really nice to see a profile of this (and it shouldn't be hard to record).

You need to log in before you can comment on or make changes to this bug.