MSVSphere Packaging Team
1 year ago
parent
0d8ee2c756
commit
94eefdc003
@ -1,7 +1,6 @@
|
||||
52f2d51d0e17d137571bf3a766f514d34e28e556 SOURCES/cbindgen-vendor.tar.xz
|
||||
6aa448bcbabf2b9410b916e8290b0f58ee725186 SOURCES/firefox-102.11.0esr.b2.processed-source.tar.xz
|
||||
a26fce6c1a21e026f550ee9e4431200ddd041e36 SOURCES/firefox-langpacks-102.11.0esr-20230504.tar.xz
|
||||
2dbf669fa4742e7065cc54cec19f96423032658b SOURCES/firefox-symbolic.svg
|
||||
ad6fa8c1048c81bedd369efcb30642a8bea003d9 SOURCES/firefox-102.12.0esr.processed-source.tar.xz
|
||||
422709954a376fb2cb4d5f89310372e664cb2cf3 SOURCES/firefox-langpacks-102.12.0esr-20230531.tar.xz
|
||||
da39a3ee5e6b4b0d3255bfef95601890afd80709 SOURCES/mochitest-python.tar.gz
|
||||
af58b3c87a8b5491dde63b07efaeb3d7f1ec56c1 SOURCES/nspr-4.34.0-3.el8_1.src.rpm
|
||||
a9dd43799ab2ccbc248cfbba1cc5639c1ab18769 SOURCES/nss-3.79.0-11.el8_1.src.rpm
|
||||
|
||||
@ -1,7 +1,6 @@
|
||||
SOURCES/cbindgen-vendor.tar.xz
|
||||
SOURCES/firefox-102.11.0esr.b2.processed-source.tar.xz
|
||||
SOURCES/firefox-langpacks-102.11.0esr-20230504.tar.xz
|
||||
SOURCES/firefox-symbolic.svg
|
||||
SOURCES/firefox-102.12.0esr.processed-source.tar.xz
|
||||
SOURCES/firefox-langpacks-102.12.0esr-20230531.tar.xz
|
||||
SOURCES/mochitest-python.tar.gz
|
||||
SOURCES/nspr-4.34.0-3.el8_1.src.rpm
|
||||
SOURCES/nss-3.79.0-11.el8_1.src.rpm
|
||||
|
||||
|
After Width: | Height: | Size: 2.2 KiB |
@ -0,0 +1,632 @@
|
||||
diff --git a/security/manager/locales/en-US/security/certificates/certManager.ftl b/security/manager/locales/en-US/security/certificates/certManager.ftl
|
||||
--- a/security/manager/locales/en-US/security/certificates/certManager.ftl
|
||||
+++ b/security/manager/locales/en-US/security/certificates/certManager.ftl
|
||||
@@ -51,9 +51,6 @@ certmgr-cert-name =
|
||||
certmgr-cert-server =
|
||||
.label = Server
|
||||
|
||||
-certmgr-override-lifetime =
|
||||
- .label = Lifetime
|
||||
-
|
||||
certmgr-token-name =
|
||||
.label = Security Device
|
||||
|
||||
@@ -69,6 +66,9 @@ certmgr-email =
|
||||
certmgr-serial =
|
||||
.label = Serial Number
|
||||
|
||||
+certmgr-fingerprint-sha-256 =
|
||||
+ .label = SHA-256 Fingerprint
|
||||
+
|
||||
certmgr-view =
|
||||
.label = View…
|
||||
.accesskey = V
|
||||
diff --git a/security/manager/pki/resources/content/certManager.js b/security/manager/pki/resources/content/certManager.js
|
||||
--- a/security/manager/pki/resources/content/certManager.js
|
||||
+++ b/security/manager/pki/resources/content/certManager.js
|
||||
@@ -64,22 +64,16 @@ var serverRichList = {
|
||||
|
||||
buildRichList() {
|
||||
let overrides = overrideService.getOverrides().map(item => {
|
||||
- let cert = null;
|
||||
- if (item.dbKey !== "") {
|
||||
- cert = certdb.findCertByDBKey(item.dbKey);
|
||||
- }
|
||||
return {
|
||||
hostPort: item.hostPort,
|
||||
- dbKey: item.dbKey,
|
||||
asciiHost: item.asciiHost,
|
||||
port: item.port,
|
||||
originAttributes: item.originAttributes,
|
||||
- isTemporary: item.isTemporary,
|
||||
- displayName: cert !== null ? cert.displayName : "",
|
||||
+ fingerprint: item.fingerprint,
|
||||
};
|
||||
});
|
||||
overrides.sort((a, b) => {
|
||||
- let criteria = ["hostPort", "displayName"];
|
||||
+ let criteria = ["hostPort", "fingerprint"];
|
||||
for (let c of criteria) {
|
||||
let res = a[c].localeCompare(b[c]);
|
||||
if (res !== 0) {
|
||||
@@ -106,10 +100,10 @@ var serverRichList = {
|
||||
_richBoxAddItem(item) {
|
||||
let richlistitem = document.createXULElement("richlistitem");
|
||||
|
||||
- richlistitem.setAttribute("dbKey", item.dbKey);
|
||||
richlistitem.setAttribute("host", item.asciiHost);
|
||||
richlistitem.setAttribute("port", item.port);
|
||||
richlistitem.setAttribute("hostPort", item.hostPort);
|
||||
+ richlistitem.setAttribute("fingerprint", item.fingerprint);
|
||||
richlistitem.setAttribute(
|
||||
"originAttributes",
|
||||
JSON.stringify(item.originAttributes)
|
||||
@@ -120,18 +114,7 @@ var serverRichList = {
|
||||
hbox.setAttribute("equalsize", "always");
|
||||
|
||||
hbox.appendChild(createRichlistItem({ raw: item.hostPort }));
|
||||
- hbox.appendChild(
|
||||
- createRichlistItem(
|
||||
- item.displayName !== ""
|
||||
- ? { raw: item.displayName }
|
||||
- : { l10nid: "no-cert-stored-for-override" }
|
||||
- )
|
||||
- );
|
||||
- hbox.appendChild(
|
||||
- createRichlistItem({
|
||||
- l10nid: item.isTemporary ? "temporary-override" : "permanent-override",
|
||||
- })
|
||||
- );
|
||||
+ hbox.appendChild(createRichlistItem({ raw: item.fingerprint }));
|
||||
|
||||
richlistitem.appendChild(hbox);
|
||||
|
||||
@@ -170,32 +153,6 @@ var serverRichList = {
|
||||
}
|
||||
},
|
||||
|
||||
- viewSelectedRichListItem() {
|
||||
- let selectedItem = this.richlist.selectedItem;
|
||||
- if (!selectedItem) {
|
||||
- return;
|
||||
- }
|
||||
-
|
||||
- let dbKey = selectedItem.getAttribute("dbKey");
|
||||
- if (dbKey) {
|
||||
- let cert = certdb.findCertByDBKey(dbKey);
|
||||
- viewCertHelper(window, cert);
|
||||
- }
|
||||
- },
|
||||
-
|
||||
- exportSelectedRichListItem() {
|
||||
- let selectedItem = this.richlist.selectedItem;
|
||||
- if (!selectedItem) {
|
||||
- return;
|
||||
- }
|
||||
-
|
||||
- let dbKey = selectedItem.getAttribute("dbKey");
|
||||
- if (dbKey) {
|
||||
- let cert = certdb.findCertByDBKey(dbKey);
|
||||
- exportToFile(window, cert);
|
||||
- }
|
||||
- },
|
||||
-
|
||||
addException() {
|
||||
let retval = {
|
||||
exceptionAdded: false,
|
||||
@@ -212,16 +169,8 @@ var serverRichList = {
|
||||
},
|
||||
|
||||
_setButtonState() {
|
||||
- let websiteViewButton = document.getElementById("websites_viewButton");
|
||||
- let websiteExportButton = document.getElementById("websites_exportButton");
|
||||
let websiteDeleteButton = document.getElementById("websites_deleteButton");
|
||||
-
|
||||
- let certKey = this.richlist.selectedItem?.getAttribute("dbKey");
|
||||
- let cert = certKey && certdb.findCertByDBKey(certKey);
|
||||
-
|
||||
websiteDeleteButton.disabled = this.richlist.selectedIndex < 0;
|
||||
- websiteExportButton.disabled = !cert;
|
||||
- websiteViewButton.disabled = websiteExportButton.disabled;
|
||||
},
|
||||
};
|
||||
/**
|
||||
diff --git a/security/manager/pki/resources/content/certManager.xhtml b/security/manager/pki/resources/content/certManager.xhtml
|
||||
--- a/security/manager/pki/resources/content/certManager.xhtml
|
||||
+++ b/security/manager/pki/resources/content/certManager.xhtml
|
||||
@@ -157,18 +157,13 @@
|
||||
|
||||
<listheader equalsize="always">
|
||||
<treecol id="sitecol" data-l10n-id="certmgr-cert-server" primary="true" flex="1"/>
|
||||
- <treecol id="certcol" data-l10n-id="certmgr-cert-name" flex="1"/>
|
||||
- <treecol id="lifetimecol" data-l10n-id="certmgr-override-lifetime" flex="1"/>
|
||||
+ <treecol id="sha256col" data-l10n-id="certmgr-fingerprint-sha-256" flex="1"/>
|
||||
</listheader>
|
||||
<richlistbox ondblclick="serverRichList.viewSelectedRichListItem();" class="certManagerRichlistBox" id="serverList" flex="1" selected="false"/>
|
||||
|
||||
<separator class="thin"/>
|
||||
|
||||
<hbox>
|
||||
- <button id="websites_viewButton"
|
||||
- data-l10n-id="certmgr-view" oncommand="serverRichList.viewSelectedRichListItem();"/>
|
||||
- <button id="websites_exportButton"
|
||||
- data-l10n-id="certmgr-export" oncommand="serverRichList.exportSelectedRichListItem();"/>
|
||||
<button id="websites_deleteButton"
|
||||
data-l10n-id="certmgr-delete" oncommand="serverRichList.deleteSelectedRichListItem();"/>
|
||||
<button id="websites_exceptionButton"
|
||||
diff --git a/security/manager/ssl/nsCertOverrideService.cpp b/security/manager/ssl/nsCertOverrideService.cpp
|
||||
--- a/security/manager/ssl/nsCertOverrideService.cpp
|
||||
+++ b/security/manager/ssl/nsCertOverrideService.cpp
|
||||
@@ -106,8 +106,8 @@ nsCertOverride::GetAsciiHost(/*out*/ nsA
|
||||
}
|
||||
|
||||
NS_IMETHODIMP
|
||||
-nsCertOverride::GetDbKey(/*out*/ nsACString& aDBKey) {
|
||||
- aDBKey = mDBKey;
|
||||
+nsCertOverride::GetFingerprint(/*out*/ nsACString& aFingerprint) {
|
||||
+ aFingerprint = mFingerprint;
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
@@ -118,12 +118,6 @@ nsCertOverride::GetPort(/*out*/ int32_t*
|
||||
}
|
||||
|
||||
NS_IMETHODIMP
|
||||
-nsCertOverride::GetIsTemporary(/*out*/ bool* aIsTemporary) {
|
||||
- *aIsTemporary = mIsTemporary;
|
||||
- return NS_OK;
|
||||
-}
|
||||
-
|
||||
-NS_IMETHODIMP
|
||||
nsCertOverride::GetHostPort(/*out*/ nsACString& aHostPort) {
|
||||
nsCertOverrideService::GetHostWithPort(mAsciiHost, mPort, aHostPort);
|
||||
return NS_OK;
|
||||
@@ -274,7 +268,6 @@ void nsCertOverrideService::RemoveAllTem
|
||||
for (auto iter = mSettingsTable.Iter(); !iter.Done(); iter.Next()) {
|
||||
nsCertOverrideEntry* entry = iter.Get();
|
||||
if (entry->mSettings->mIsTemporary) {
|
||||
- entry->mSettings->mCert = nullptr;
|
||||
iter.Remove();
|
||||
}
|
||||
}
|
||||
@@ -297,18 +297,11 @@
|
||||
nsAutoCString buffer;
|
||||
bool isMore = true;
|
||||
|
||||
- /* file format is:
|
||||
- *
|
||||
- * host:port:originattributes \t fingerprint-algorithm \t fingerprint \t
|
||||
- * override-mask \t dbKey
|
||||
- *
|
||||
- * where override-mask is a sequence of characters,
|
||||
- * M meaning hostname-Mismatch-override
|
||||
- * U meaning Untrusted-override
|
||||
- * T meaning Time-error-override (expired/not yet valid)
|
||||
- *
|
||||
- * if this format isn't respected we move onto the next line in the file.
|
||||
- */
|
||||
+ // Each line is of the form:
|
||||
+ // host:port:originAttributes \t sSHA256OIDString \t fingerprint \t
|
||||
+ // There may be some "bits" identifiers and "dbKey" after the `fingerprint`
|
||||
+ // field in 'fingerprint \t \t dbKey' format, but these are now ignored.
|
||||
+ // Lines that don't match this form are silently dropped.
|
||||
|
||||
while (isMore && NS_SUCCEEDED(lineInputStream->ReadLine(buffer, &isMore))) {
|
||||
if (buffer.IsEmpty() || buffer.First() == '#') {
|
||||
@@ -350,23 +343,10 @@
|
||||
fingerprint.Length() == 0) {
|
||||
continue;
|
||||
}
|
||||
- nsDependentCSubstring bitsString;
|
||||
- if (!parser.ReadUntil(Tokenizer::Token::Whitespace(), bitsString) ||
|
||||
- bitsString.Length() == 0) {
|
||||
- continue;
|
||||
- }
|
||||
- nsDependentCSubstring dbKey;
|
||||
- if (!parser.ReadUntil(Tokenizer::Token::EndOfFile(), dbKey) ||
|
||||
- dbKey.Length() == 0) {
|
||||
- continue;
|
||||
- }
|
||||
- nsCertOverride::OverrideBits bits;
|
||||
- nsCertOverride::convertStringToBits(bitsString, bits);
|
||||
|
||||
AddEntryToList(host, port, attributes,
|
||||
- nullptr, // don't have the cert
|
||||
- false, // not temporary
|
||||
- fingerprint, bits, dbKey, aProofOfLock);
|
||||
+ false, // not temporary
|
||||
+ fingerprint, aProofOfLock);
|
||||
}
|
||||
|
||||
return NS_OK;
|
||||
@@ -412,9 +392,8 @@
|
||||
output.Append(kTab);
|
||||
output.Append(settings->mFingerprint);
|
||||
output.Append(kTab);
|
||||
- output.Append(bitsString);
|
||||
- output.Append(kTab);
|
||||
- output.Append(settings->mDBKey);
|
||||
+ // the "bits" string used to go here, but it no longer exists
|
||||
+ // the "\t dbKey" string used to go here, but it no longer exists
|
||||
output.Append(NS_LINEBREAK);
|
||||
}
|
||||
|
||||
@@ -462,42 +441,16 @@
|
||||
return NS_ERROR_FAILURE;
|
||||
}
|
||||
|
||||
- nsAutoCString nickname;
|
||||
- nsresult rv = DefaultServerNicknameForCert(nsscert.get(), nickname);
|
||||
- if (!aTemporary && NS_SUCCEEDED(rv)) {
|
||||
- UniquePK11SlotInfo slot(PK11_GetInternalKeySlot());
|
||||
- if (!slot) {
|
||||
- return NS_ERROR_FAILURE;
|
||||
- }
|
||||
-
|
||||
- // This can fail (for example, if we're in read-only mode). Luckily, we
|
||||
- // don't even need it to succeed - we always match on the stored hash of the
|
||||
- // certificate rather than the full certificate. It makes the display a bit
|
||||
- // less informative (since we won't have a certificate to display), but it's
|
||||
- // better than failing the entire operation.
|
||||
- Unused << PK11_ImportCert(slot.get(), nsscert.get(), CK_INVALID_HANDLE,
|
||||
- nickname.get(), false);
|
||||
- }
|
||||
-
|
||||
nsAutoCString fpStr;
|
||||
- rv = GetCertSha256Fingerprint(aCert, fpStr);
|
||||
- if (NS_FAILED(rv)) {
|
||||
- return rv;
|
||||
- }
|
||||
-
|
||||
- nsAutoCString dbkey;
|
||||
- rv = aCert->GetDbKey(dbkey);
|
||||
+ nsresult rv = GetCertSha256Fingerprint(aCert, fpStr);
|
||||
if (NS_FAILED(rv)) {
|
||||
return rv;
|
||||
}
|
||||
|
||||
{
|
||||
MutexAutoLock lock(mMutex);
|
||||
- AddEntryToList(aHostName, aPort, aOriginAttributes,
|
||||
- aTemporary ? aCert : nullptr,
|
||||
- // keep a reference to the cert for temporary overrides
|
||||
- aTemporary, fpStr,
|
||||
- (nsCertOverride::OverrideBits)aOverrideBits, dbkey, lock);
|
||||
+ AddEntryToList(aHostName, aPort, aOriginAttributes, aTemporary, fpStr,
|
||||
+ lock);
|
||||
if (!aTemporary) {
|
||||
Write(lock);
|
||||
}
|
||||
@@ -532,10 +485,8 @@
|
||||
|
||||
MutexAutoLock lock(mMutex);
|
||||
AddEntryToList(aHostName, aPort, aOriginAttributes,
|
||||
- nullptr, // No cert to keep alive
|
||||
true, // temporary
|
||||
- aCertFingerprint, (nsCertOverride::OverrideBits)aOverrideBits,
|
||||
- ""_ns, // dbkey
|
||||
+ aCertFingerprint,
|
||||
lock);
|
||||
|
||||
return NS_OK;
|
||||
@@ -632,10 +583,8 @@
|
||||
|
||||
nsresult nsCertOverrideService::AddEntryToList(
|
||||
const nsACString& aHostName, int32_t aPort,
|
||||
- const OriginAttributes& aOriginAttributes, nsIX509Cert* aCert,
|
||||
- const bool aIsTemporary, const nsACString& fingerprint,
|
||||
- nsCertOverride::OverrideBits ob, const nsACString& dbKey,
|
||||
- const MutexAutoLock& aProofOfLock) {
|
||||
+ const OriginAttributes& aOriginAttributes, const bool aIsTemporary,
|
||||
+ const nsACString& fingerprint, const MutexAutoLock& aProofOfLock) {
|
||||
mMutex.AssertCurrentThreadOwns();
|
||||
nsAutoCString keyString;
|
||||
GetKeyString(aHostName, aPort, aOriginAttributes, keyString);
|
||||
@@ -656,11 +605,6 @@
|
||||
settings->mOriginAttributes = aOriginAttributes;
|
||||
settings->mIsTemporary = aIsTemporary;
|
||||
settings->mFingerprint = fingerprint;
|
||||
- settings->mOverrideBits = ob;
|
||||
- settings->mDBKey = dbKey;
|
||||
- // remove whitespace from stored dbKey for backwards compatibility
|
||||
- settings->mDBKey.StripWhitespace();
|
||||
- settings->mCert = aCert;
|
||||
entry->mSettings = settings;
|
||||
|
||||
return NS_OK;
|
||||
diff --git a/security/manager/ssl/nsCertOverrideService.h b/security/manager/ssl/nsCertOverrideService.h
|
||||
--- a/security/manager/ssl/nsCertOverrideService.h
|
||||
+++ b/security/manager/ssl/nsCertOverrideService.h
|
||||
@@ -43,8 +43,6 @@
|
||||
bool mIsTemporary; // true: session only, false: stored on disk
|
||||
nsCString mFingerprint;
|
||||
OverrideBits mOverrideBits;
|
||||
- nsCString mDBKey;
|
||||
- nsCOMPtr<nsIX509Cert> mCert;
|
||||
|
||||
static void convertBitsToString(OverrideBits ob, nsACString& str);
|
||||
static void convertStringToBits(const nsACString& str, OverrideBits& ob);
|
||||
@@ -145,10 +143,8 @@
|
||||
nsresult Write(const mozilla::MutexAutoLock& aProofOfLock);
|
||||
nsresult AddEntryToList(const nsACString& host, int32_t port,
|
||||
const OriginAttributes& aOriginAttributes,
|
||||
- nsIX509Cert* aCert, const bool aIsTemporary,
|
||||
+ const bool aIsTemporary,
|
||||
const nsACString& fingerprint,
|
||||
- nsCertOverride::OverrideBits ob,
|
||||
- const nsACString& dbKey,
|
||||
const mozilla::MutexAutoLock& aProofOfLock);
|
||||
|
||||
// Set in constructor only
|
||||
diff --git a/security/manager/ssl/SSLServerCertVerification.cpp b/security/manager/ssl/SSLServerCertVerification.cpp
|
||||
--- a/security/manager/ssl/SSLServerCertVerification.cpp
|
||||
+++ b/security/manager/ssl/SSLServerCertVerification.cpp
|
||||
@@ -791,8 +791,8 @@
|
||||
aHostName, aPort, aOriginAttributes, aCert, &overrideBits,
|
||||
&isTemporaryOverride, &haveOverride);
|
||||
if (NS_SUCCEEDED(rv) && haveOverride) {
|
||||
- // remove the errors that are already overriden
|
||||
- remainingDisplayErrors &= ~overrideBits;
|
||||
+ // remove all the errors
|
||||
+ remainingDisplayErrors = 0;
|
||||
}
|
||||
}
|
||||
|
||||
diff --git a/security/manager/ssl/nsICertOverrideService.idl b/security/manager/ssl/nsICertOverrideService.idl
|
||||
--- a/security/manager/ssl/nsICertOverrideService.idl
|
||||
+++ b/security/manager/ssl/nsICertOverrideService.idl
|
||||
@@ -33,17 +33,6 @@ interface nsICertOverride : nsISupports
|
||||
readonly attribute int32_t port;
|
||||
|
||||
/**
|
||||
- * Whether or not the override is only used for this
|
||||
- * session (true) or stored persistently (false)
|
||||
- */
|
||||
- readonly attribute boolean isTemporary;
|
||||
-
|
||||
- /**
|
||||
- * The database key for the associated certificate.
|
||||
- */
|
||||
- readonly attribute ACString dbKey;
|
||||
-
|
||||
- /**
|
||||
* A combination of hostname and port in the form host:port.
|
||||
* Since the port can be -1 which is equivalent to port 433 we use an
|
||||
* existing function of nsCertOverrideService to create this property.
|
||||
@@ -51,6 +40,11 @@ interface nsICertOverride : nsISupports
|
||||
readonly attribute ACString hostPort;
|
||||
|
||||
/**
|
||||
+ * The fingerprint for the associated certificate.
|
||||
+ */
|
||||
+ readonly attribute ACString fingerprint;
|
||||
+
|
||||
+ /**
|
||||
* The origin attributes associated with this override.
|
||||
*/
|
||||
[implicit_jscontext]
|
||||
diff --git a/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js b/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js
|
||||
--- a/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js
|
||||
+++ b/security/manager/ssl/tests/mochitest/browser/browser_certificateManager.js
|
||||
@@ -27,9 +27,7 @@ async function checkServerCertificates(w
|
||||
|
||||
expectedValues.forEach((item, i) => {
|
||||
let hostPort = labels[i * 3].value;
|
||||
- let certString = labels[i * 3 + 1].value || labels[i * 3 + 1].textContent;
|
||||
- let isTemporaryString =
|
||||
- labels[i * 3 + 2].value || labels[i * 3 + 2].textContent;
|
||||
+ let fingerprint = labels[i * 3 + 1].value || labels[i * 3 + 1].textContent;
|
||||
|
||||
Assert.equal(
|
||||
hostPort,
|
||||
@@ -38,15 +36,9 @@ async function checkServerCertificates(w
|
||||
);
|
||||
|
||||
Assert.equal(
|
||||
- certString,
|
||||
- item.certName,
|
||||
- `Expected override to have field ${item.certName}`
|
||||
- );
|
||||
-
|
||||
- Assert.equal(
|
||||
- isTemporaryString,
|
||||
- item.isTemporary ? "Temporary" : "Permanent",
|
||||
- `Expected override to be ${item.isTemporary ? "Temporary" : "Permanent"}`
|
||||
+ fingerprint,
|
||||
+ item.fingerprint,
|
||||
+ `Expected override to have field ${item.fingerprint}`
|
||||
);
|
||||
});
|
||||
}
|
||||
@@ -73,41 +73,6 @@
|
||||
);
|
||||
}
|
||||
|
||||
-async function testViewButton(win) {
|
||||
- win.document.getElementById("serverList").selectedIndex = 1;
|
||||
-
|
||||
- Assert.ok(
|
||||
- win.document.getElementById("websites_viewButton").disabled,
|
||||
- "View button should be disabled for override without cert"
|
||||
- );
|
||||
-
|
||||
- win.document.getElementById("serverList").selectedIndex = 0;
|
||||
-
|
||||
- Assert.ok(
|
||||
- !win.document.getElementById("websites_viewButton").disabled,
|
||||
- "View button should be enabled for override with cert"
|
||||
- );
|
||||
-
|
||||
- let loaded = BrowserTestUtils.waitForNewTab(gBrowser, null, true);
|
||||
-
|
||||
- win.document.getElementById("websites_viewButton").click();
|
||||
-
|
||||
- let newTab = await loaded;
|
||||
- let spec = newTab.linkedBrowser.documentURI.spec;
|
||||
-
|
||||
- Assert.ok(
|
||||
- spec.startsWith("about:certificate"),
|
||||
- "about:certificate should habe been opened"
|
||||
- );
|
||||
-
|
||||
- let newUrl = new URL(spec);
|
||||
- let certEncoded = newUrl.searchParams.get("cert");
|
||||
- let certDecoded = decodeURIComponent(certEncoded);
|
||||
- Assert.ok(certDecoded, "should have some certificate as cert url param");
|
||||
-
|
||||
- gBrowser.removeCurrentTab();
|
||||
-}
|
||||
-
|
||||
add_task(async function test_cert_manager_server_tab() {
|
||||
let win = await openCertManager();
|
||||
|
||||
@@ -134,48 +99,13 @@
|
||||
await checkServerCertificates(win, [
|
||||
{
|
||||
hostPort: "example.com:443",
|
||||
- certName: "md5-ee",
|
||||
- isTemporary: false,
|
||||
- },
|
||||
- ]);
|
||||
-
|
||||
- win.document.getElementById("certmanager").acceptDialog();
|
||||
- await BrowserTestUtils.windowClosed(win);
|
||||
-
|
||||
- certOverrideService.rememberTemporaryValidityOverrideUsingFingerprint(
|
||||
- "example.com",
|
||||
- 9999,
|
||||
- {},
|
||||
- "40:20:3E:57:FB:82:95:0D:3F:62:D7:04:39:F6:32:CC:B2:2F:70:9F:3E:66:C5:35:64:6E:49:2A:F1:02:75:9F",
|
||||
- Ci.nsICertOverrideService.ERROR_UNTRUSTED
|
||||
- );
|
||||
-
|
||||
- win = await openCertManager();
|
||||
-
|
||||
- await checkServerCertificates(win, [
|
||||
- {
|
||||
- hostPort: "example.com:443",
|
||||
- certName: "md5-ee",
|
||||
- isTemporary: false,
|
||||
- },
|
||||
- {
|
||||
- hostPort: "example.com:9999",
|
||||
- certName: "(Not Stored)",
|
||||
- isTemporary: true,
|
||||
+ fingerprint: cert.sha256Fingerprint,
|
||||
},
|
||||
]);
|
||||
|
||||
- await testViewButton(win);
|
||||
-
|
||||
- await deleteOverride(win, 2);
|
||||
+ await deleteOverride(win, 1);
|
||||
|
||||
- await checkServerCertificates(win, [
|
||||
- {
|
||||
- hostPort: "example.com:9999",
|
||||
- certName: "(Not Stored)",
|
||||
- isTemporary: true,
|
||||
- },
|
||||
- ]);
|
||||
+ await checkServerCertificates(win, []);
|
||||
|
||||
win.document.getElementById("certmanager").acceptDialog();
|
||||
await BrowserTestUtils.windowClosed(win);
|
||||
diff --git a/security/manager/ssl/tests/unit/test_cert_override_read.js b/security/manager/ssl/tests/unit/test_cert_override_read.js
|
||||
--- a/security/manager/ssl/tests/unit/test_cert_override_read.js
|
||||
+++ b/security/manager/ssl/tests/unit/test_cert_override_read.js
|
||||
@@ -11,19 +11,16 @@ function run_test() {
|
||||
let cert1 = {
|
||||
sha256Fingerprint:
|
||||
"E9:3A:91:F6:15:11:FB:DD:02:76:DD:45:8C:4B:F4:9B:D1:14:13:91:2E:96:4B:EC:D2:4F:90:D5:F4:BB:29:5C",
|
||||
- dbKey: "This isn't relevant for this test.",
|
||||
};
|
||||
// bad_certs/selfsigned.pem
|
||||
let cert2 = {
|
||||
sha256Fingerprint:
|
||||
"51:BC:41:90:C1:FD:6E:73:18:19:B0:60:08:DD:A3:3D:59:B2:5B:FB:D0:3D:DD:89:19:A5:BB:C6:2B:5A:72:A7",
|
||||
- dbKey: "This isn't relevant for this test.",
|
||||
};
|
||||
// bad_certs/noValidNames.pem
|
||||
let cert3 = {
|
||||
sha256Fingerprint:
|
||||
"C3:A3:61:02:CA:64:CC:EC:45:1D:24:B6:A0:69:DB:DB:F0:D8:58:76:FC:50:36:52:5A:E8:40:4C:55:72:08:F4",
|
||||
- dbKey: "This isn't relevant for this test.",
|
||||
};
|
||||
|
||||
let profileDir = do_get_profile();
|
||||
@@ -35,58 +35,42 @@
|
||||
"# This is a generated file! Do not edit.",
|
||||
"test.example.com:443:^privateBrowsingId=1\tOID.2.16.840.1.101.3.4.2.1\t" +
|
||||
cert1.sha256Fingerprint +
|
||||
- "\tM\t" +
|
||||
- cert1.dbKey,
|
||||
+ "\t",
|
||||
"test.example.com:443:^privateBrowsingId=2\tOID.2.16.840.1.101.3.4.2.1\t" +
|
||||
cert1.sha256Fingerprint +
|
||||
+ "\t",
|
||||
+ "test.example.com:443:^privateBrowsingId=3\tOID.2.16.840.1.101.3.4.2.1\t" + // includes bits and dbKey (now obsolete)
|
||||
+ cert1.sha256Fingerprint +
|
||||
"\tM\t" +
|
||||
- cert1.dbKey,
|
||||
+ "AAAAAAAAAAAAAAACAAAAFjA5MBQxEjAQBgNVBAMMCWxvY2FsaG9zdA==",
|
||||
"example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" +
|
||||
cert2.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert2.dbKey,
|
||||
+ "\t",
|
||||
"[::1]:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // IPv6
|
||||
cert2.sha256Fingerprint +
|
||||
- "\tM\t" +
|
||||
- cert2.dbKey,
|
||||
+ "\t",
|
||||
"old.example.com:443\tOID.2.16.840.1.101.3.4.2.1\t" + // missing attributes (defaulted)
|
||||
cert1.sha256Fingerprint +
|
||||
- "\tM\t" +
|
||||
- cert1.dbKey,
|
||||
+ "\t",
|
||||
":443:\tOID.2.16.840.1.101.3.4.2.1\t" + // missing host name
|
||||
cert3.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
+ "\t",
|
||||
"example.com::\tOID.2.16.840.1.101.3.4.2.1\t" + // missing port
|
||||
cert3.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
- "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // wrong fingerprint/dbkey
|
||||
+ "\t",
|
||||
+ "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // wrong fingerprint
|
||||
cert2.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
+ "\t",
|
||||
"example.com:443:\tOID.0.00.000.0.000.0.0.0.0\t" + // bad OID
|
||||
cert3.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
+ "\t",
|
||||
"example.com:443:\t.0.0.0.0\t" + // malformed OID
|
||||
cert3.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
+ "\t",
|
||||
"example.com:443:\t\t" + // missing OID
|
||||
cert3.sha256Fingerprint +
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
- "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // missing fingerprint
|
||||
- "\tU\t" +
|
||||
- cert3.dbKey,
|
||||
- "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // missing override bits
|
||||
- cert3.sha256Fingerprint +
|
||||
- "\t\t" +
|
||||
- cert3.dbKey,
|
||||
- "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t" + // missing dbkey
|
||||
- cert3.sha256Fingerprint +
|
||||
- "\tU\t",
|
||||
+ "\t",
|
||||
+ "example.com:443:\tOID.2.16.840.1.101.3.4.2.1\t", // missing fingerprint
|
||||
];
|
||||
writeLinesAndClose(lines, outputStream);
|
||||
let overrideService = Cc["@mozilla.org/security/certoverride;1"].getService(
|
||||
Loading…
Reference in new issue