Closed Bug 1570502 Opened 6 months ago Closed 2 months ago

typo in aboutNetError.js: `failedCertInfo.erroCodeString`

Categories

(Firefox :: Security, defect, P5)

defect

Tracking

()

RESOLVED FIXED
Firefox 72
Tracking Status
firefox-esr68 --- unaffected
firefox68 --- unaffected
firefox69 --- wontfix
firefox70 --- wontfix
firefox71 --- wontfix
firefox72 --- fixed

People

(Reporter: keeler, Assigned: kskim.kim, Mentored)

References

(Regression)

Details

(Keywords: good-first-bug, regression)

Attachments

(1 file, 1 obsolete file)

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task

Thanks for spotting this, but it isn't a bad issue IMO. It will only mean that pages that are affected by the Symantec distrust will get a yellow border instead of no yellow border. Most sites have migrated off at this point.

(this implies we don't have test coverage for this)

No, we don't have test coverage for the symantec distrust error page and I don't think it's worth adding some now. I don't think we would necessarily have tested this specific case (that it doesn't get the yellow border) anyway.

No longer blocks: 1552333
Mentor: jhofmann
Priority: -- → P5
Regressed by: 1552333

I would like to work on it, can I take this up?

Flags: needinfo?(jhofmann)

Hi Megha, I just assigned you to bug 1530781 and I think it's best to focus on a single bug for now. I'll happily assign you to this when you fix the other bug. You can still of course always submit a patch for this bug.

Flags: needinfo?(jhofmann)

Please assign this one to me.

Sure, please let me know if you need any help!

Assignee: nobody → saini.singhbalraj
Status: NEW → ASSIGNED

Happy to take a patch for 70; I'm just marking this fix-optional so that we don't need to keep triaging the bug every week.

(In reply to Johann Hofmann [:johannh] from comment #6)

Sure, please let me know if you need any help!

Thanks. Sorry for my late reply.
Can you help me get started with getting the source code and building it? Where can I find it?

Edit: I'm using macOS, so I found this https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Mac_OS_X_Prerequisites guide to getting the source and building instructions. Is that right?

(In reply to Balraj from comment #9)

Edit: I'm using macOS, so I found this https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Mac_OS_X_Prerequisites guide to getting the source and building instructions. Is that right?

Yup, that should be right.

Hey, are you still working on this or should I unassign you? :)

Thanks!

Flags: needinfo?(saini.singhbalraj)

Unassigning for inactivity, let me know if you want to pick this up again :)

Assignee: saini.singhbalraj → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(saini.singhbalraj)

Hi I am a computer science student from University of Toronto and I am interested in fixing this bug as part of my class project. Can I claim this bug? I am also looking for a mentor who can help me fixing this bug.

Hi Kyle - I assigned you to the bug. Here's build instructions: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions
Here's more general information on contributing: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction
Let me know if you have any questions!

Assignee: nobody → kskim.kim
Mentor: jhofmann → dkeeler

Hi Dana, Could you give me an advice on how to test my bugfix? I am little bit unclear on "Symantec distrust" and how to trigger it.
(In reply to Dana Keeler (she/her) (use needinfo) (:keeler for reviews) from comment #14)

Hi Kyle - I assigned you to the bug. Here's build instructions: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions
Here's more general information on contributing: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Introduction
Let me know if you have any questions!

Flags: needinfo?(dkeeler)
Attached file Bug 1570502 - print hello kyle (obsolete) —

Depends on D52491

Attachment #9107758 - Attachment is obsolete: true
Attachment #9107759 - Attachment description: Bug 1570502 - bugfix 1570502 fixed typo erroCodeString to errorCodeStrng → Bug 1570502 - Fixed typo erroCodeString to errorCodeStrng
Attachment #9107759 - Attachment description: Bug 1570502 - Fixed typo erroCodeString to errorCodeStrng → Bug 1570502 - Fixed typo erroCodeString to errorCodeString

To test this, you could run ./mach mochitest browser/base/content/test/about/browser_aboutNetError.js.
To specifically test the Symantec distrust handling, you could run the SanctionsTestServer[0], add symantec-not-whitelisted-after-cutoff.example.com to your hosts file, add the Symantec test CA [1] as a trusted certificate in Firefox, visit https://symantec-not-whitelisted-after-cutoff.example.com:8443, and check that there are no errors in the web console and the browser console. That's probably more work than necessary, though, since we can verify by inspection that this is correct.

[0] https://searchfox.org/mozilla-central/source/security/manager/ssl/tests/unit/tlsserver/cmd/SanctionsTestServer.cpp
[1] https://searchfox.org/mozilla-central/source/security/manager/ssl/tests/unit/test_sanctions/symantec-test-ca.pem

Flags: needinfo?(dkeeler)

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:kskim.kim, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(kskim.kim)
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/4c60ff4bc048
Fixed typo erroCodeString to errorCodeString r=keeler
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

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