Skip to content
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(signing-event): add support to sign volume events before exporting to server #14

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

mittachaitu
Copy link

@mittachaitu mittachaitu commented Nov 8, 2021

This PR adds support to sign volume events(only if SIGNING_KEY_PATH
env is configured) and export the signature along with data to configured REST
based server.

Integration test cases:

  • Add sanity test case to verify the signature of create & delete volumes events information.

TODO:

  • Adding document about usage of volume signing events.

Signed-off-by: mittachaitu [email protected]

…ng to server

This commit adds support to sign volume events(only if SIGNING_KEY_PATH
env is configured) and export the signature along with data to REST
based server

Signed-off-by: mittachaitu <[email protected]>
@mittachaitu mittachaitu marked this pull request as draft November 9, 2021 12:53
…files

This commit also does following changes:
- Add integration test to verify behavior of volume-event-collector
  when in-appropriate SIGNING_KEY is configured

Signed-off-by: mittachaitu <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2021

Codecov Report

Merging #14 (e82c840) into develop (cdef6be) will increase coverage by 5.38%.
The diff coverage is 53.21%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #14      +/-   ##
===========================================
+ Coverage    26.64%   32.02%   +5.38%     
===========================================
  Files            5       10       +5     
  Lines          334      434     +100     
===========================================
+ Hits            89      139      +50     
- Misses         235      275      +40     
- Partials        10       20      +10     
Impacted Files Coverage Δ
pkg/encrypt/rsa/generate_keys.go 0.00% <0.00%> (ø)
pkg/nfspv/nfsvolume.go 51.35% <45.45%> (-1.00%) ⬇️
pkg/controller/pv_controller_sync.go 20.00% <50.00%> (-5.00%) ⬇️
pkg/encrypt/rsa/private_key.go 50.00% <50.00%> (ø)
pkg/sign/private_key.go 65.51% <65.51%> (ø)
pkg/sign/public_key.go 76.92% <76.92%> (ø)
pkg/encrypt/rsa/public_key.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdef6be...e82c840. Read the comment docs.

Signed-off-by: mittachaitu <[email protected]>
@mittachaitu mittachaitu marked this pull request as ready for review November 16, 2021 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants