-
Notifications
You must be signed in to change notification settings - Fork 173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(api): encrypt/decrypt api #791
Conversation
Codecov ReportBase: 60.39% // Head: 60.39% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #791 +/- ##
=======================================
Coverage 60.39% 60.39%
=======================================
Files 133 133
Lines 8398 8398
=======================================
Hits 5072 5072
Misses 2738 2738
Partials 588 588 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: seeflood <[email protected]>
@kevinten10 Hi, I modified the proto according to our discussion. |
Signed-off-by: seeflood <[email protected]>
Signed-off-by: seeflood <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
message DecryptResponse { | ||
|
||
// Raw plaintext. | ||
bytes plain_text = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我在 #794 中提到了一个例子,去掉status之后似乎没办法表示?
Signed-off-by: seeflood [email protected]
What this PR does:
encrypt/decrypt api
Which issue(s) this PR fixes:
Fixes #794
Special notes for your reviewer:
Does this PR introduce a user-facing change?: