Alter docs, add comments
This commit is contained in:
parent
ab990d8cec
commit
3846aef8a1
1 changed files with 6 additions and 3 deletions
|
@ -157,12 +157,15 @@ export default class DecryptionFailureTracker {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* If there is a failure that should be tracked, call the given trackDecryptionFailure
|
* If there are failures that should be tracked, call the given trackDecryptionFailure
|
||||||
* function with the first failure in the FIFO of failures that should be tracked.
|
* function with the number of failures that should be tracked.
|
||||||
*/
|
*/
|
||||||
trackFailure() {
|
trackFailure() {
|
||||||
if (this.failuresToTrack.length > 0) {
|
if (this.failuresToTrack.length > 0) {
|
||||||
// Remove all failures, and expose the number of failures
|
// Remove all failures, and expose the number of failures for now.
|
||||||
|
//
|
||||||
|
// TODO: Track a histogram of error types to cardinailty to allow for
|
||||||
|
// aggregation by error type.
|
||||||
this.trackDecryptionFailure(this.failuresToTrack.splice(0).length);
|
this.trackDecryptionFailure(this.failuresToTrack.splice(0).length);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue