Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Merge pull request #6525 from brave/fix/6524
Browse files Browse the repository at this point in the history
Handle cert fingerprints that contain '/'
  • Loading branch information
bsclifton authored Jan 13, 2017
2 parents eab1558 + 9b9ea7f commit b7416d0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion js/about/certerror.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,16 @@ class CertErrorPage extends React.Component {
var validExpiry = new Date()
validStart.setTime(detail.validStart * 1000)
validExpiry.setTime(detail.validExpiry * 1000)
var fingerprint = detail.fingerprint.split('/')
var algorithm = fingerprint.shift()
this.setState({
certDetail: true,
certIssuerName: detail.issuerName,
certSubjectName: detail.subjectName,
certSerialNumber: seperateHex(detail.serialNumber),
certValidStart: validStart.toString(),
certValidExpiry: validExpiry.toString(),
certFingerprint: detail.fingerprint.split('/')
certFingerprint: [algorithm, fingerprint.join('/')]
})
})
}
Expand Down

0 comments on commit b7416d0

Please sign in to comment.