Page MenuHomePhorge
Differential D313

Remove logging of plaintext event
ClosedPublic
Actions

Authored by tusooa on Wed, Jun 24, 8:22 AM.

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rL1e6c1c9bdc43: Remove logging of plaintext event
Summary

A log of plaintext event was accidentally added. This is now removed.

Type: fix

Test Plan

Verify that encrypting an event will not log its plain text.

Diff Detail

Repository
rL libkazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 642
Build 1266: Invoke BuildbotBuildbot build #1901
Build 1265: arc lint + arc unit

Unit TestsFailed
View All

TimeTest
0 msCoverage (build:coverage)

https://b.lily-is.land/#/builders/42/builds/83

0 msCoverage (build:coverage)

https://b.lily-is.land/#/builders/42/builds/84

Event Timeline

tusooa created this revision.Wed, Jun 24, 8:22 AM
Owners added a reviewer: O1: the Kazv Project.Wed, Jun 24, 8:22 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Jun 24, 8:57 AM
Harbormaster failed remote builds in B642: Diff 985!
tusooa requested review of this revision.Wed, Jun 24, 2:00 PM
nannanko accepted this revision.Sat, Jun 27, 7:02 PM
This revision is now accepted and ready to land.Sat, Jun 27, 7:02 PM
This revision was landed with ongoing or failed builds.Sat, Jun 27, 9:08 PM
Closed by commit rL1e6c1c9bdc43: Remove logging of plaintext event. · Explain Why
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rL1e6c1c9bdc43: Remove logging of plaintext event.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)Packages
src/
client/
actions/
1 line64%
Loading...
O1: the Kazv Project

Diff 985

View Options

src/client/actions/encryption.cpp

Loading...