Skip to content

Commit c9c7cd2

Browse files
committed
fix: return response with error when encryption disabled
- Add response with error when encryption is disabled but clients request key Signed-off-by: chohee <[email protected]>
1 parent f057124 commit c9c7cd2

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

manager/rpcserver/manager_server_v2.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -980,7 +980,10 @@ func (s *managerServerV2) KeepAlive(stream managerv2.Manager_KeepAliveServer) er
980980
// RequestEncryptionKey implements manager.ManagerServer.
981981
func (s *managerServerV2) RequestEncryptionKey(ctx context.Context, req *managerv2.RequestEncryptionKeyRequest) (*managerv2.RequestEncryptionKeyResponse, error) {
982982
log := logger.WithHostnameAndIP(req.Hostname, req.Ip)
983-
983+
if !s.config.Encryption.Enable {
984+
// TODO: new error code in drgonfly-api?
985+
return nil, status.Error(codes.Unavailable, "encryption is disabled")
986+
}
984987
// Get key from db
985988
var encKey models.EncryptionKey
986989
if err := s.db.WithContext(ctx).First(&encKey).Error; err != nil {

0 commit comments

Comments
 (0)