Skip to content
This repository has been archived by the owner on Nov 4, 2022. It is now read-only.

Add support for an optional, secure gRPC service #202

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

jshlbrd
Copy link
Contributor

@jshlbrd jshlbrd commented Jan 23, 2019

This PR adds a gRPC service to Stenographer and implements a call to retrieve PCAP via stenoread. The gRPC service is designed to only use a secure channel (due to the sensitivity of packet data) with SSL certificates that are managed separately from those created by stenokeys.sh (due to users needing to manage remote access to many servers running Stenographer, though technically users could share the certs used by the local Stenographer HTTP service). This commit explicitly does not address any client-side integration with the gRPC service, but clients can build interaction based on the protobuf and documentation in DESIGN.md. Copyrights have been added per CONTRIBUTING.md and the CLA signed. Below is a summary of the additions and changes:

  • Added rpc/rpc.go
  • Added protobuf/steno.proto
  • Changed config/config.go to support Rpc configuration
  • Changed stenographer.go to check for Rpc configuration and (if found) run gRPC server as a goroutine
  • Updated DESIGN docs to describe Rpc functionality

This commit adds a gRPC service to Stenographer and implements a call to retrieve PCAP via stenoread. The gRPC service is designed to only use a secure channel (due to the sensitivity of packet data) with SSL certificates that are managed separately from those created by stenokeys.sh (due to users needing to manage remote access to many servers running Stenographer, though technically users could share the certs used by the local Stenographer HTTP service). This commit explicitly does not address any client-side integration with the gRPC service, but clients can build interaction based on the protobuf and documentation in DESIGN.md. Below is a summary of the additions and changes:
- Added rpc/rpc.go
- Added protobuf/steno.proto
- Changed config/config.go to support Rpc configuration
- Changed stenographer.go to check for Rpc configuration and (if found) run gRPC server as a goroutine
- Updated DESIGN docs to describe Rpc functionality
@gconnell
Copy link
Contributor

This looks great, merging!

Note that Travis is dead, so the failure there is unrelated to this.

@gconnell gconnell merged commit d678531 into google:master Jan 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants