authz: Stdout logger

authz/audit/audit_logger.go

@@ -79,7 +79,7 @@ type Event struct {
 // logger configuration. Concrete types representing configuration of specific
 // audit loggers must embed this interface to implement it.
 type LoggerConfig interface {
-	loggerConfig()
+	LoggerConfig()
 }
 
 // Logger is the interface to be implemented by audit loggers.

authz/audit/stdout/stdout_logger.go

@@ -0,0 +1,101 @@
+/*
+ *
+ * Copyright 2023 gRPC authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+
+// Package stdout contains a Stdout implementation of audit logging interface
+// The Stdoutlogger prints the audit.Event using log.go in json format
+package stdout
+
+import (
+	"encoding/json"
+	"log"
+	"time"
+
+	"google.golang.org/grpc/authz/audit"
+	"google.golang.org/grpc/grpclog"
+)
+
+var grpcLogger = grpclog.Component("authz-audit")
+
+type stdoutEvent struct {
+	FullMethodName string `json:"fullMethodName"`
+	Principal      string `json:"principal"`
+	PolicyName     string `json:"policyName"`
+	MatchedRule    string `json:"matchedRule"`
+	Authorized     bool   `json:"authorized"`
+	//Timestamp is populated using time.Now() during StdoutLogger.Log call
+	Timestamp string `json:"timestamp"`
+}
+
+// Stdout implementation of audit.Logger
+type StdoutLogger struct {
+}
+
+// Stdout implementation of audit.Logger.Log
+func (logger *StdoutLogger) Log(event *audit.Event) {
+	jsonBytes, err := json.Marshal(convertEvent(event))
+	if err != nil {
+		grpcLogger.Errorf("failed to marshal AuditEvent data to JSON: %v", err)
+	}
+	log.Println(string(jsonBytes))
+}
+
+const (
+	stdName = "stdout"
+)
+
+// Stdout implementation of audit.LoggerConfig
+// Since the logger doesn't support custom configs, it's a no-op one
+type StdoutLoggerConfig struct{}
+
+// No-op implementation of audit.LoggerConfig
+func (StdoutLoggerConfig) LoggerConfig() {}
+
+// Stdout implementation of audit.LoggerBuilder
+type StdoutLoggerBuilder struct{}
+
+// Stdout implementation of audit.LoggerBuilder.Name
+func (StdoutLoggerBuilder) Name() string {
+	return stdName
+}
+
+// Stdout implementation of audit.LoggerBuilder.Build
+// LoggerConfig is ignored so it always returns default StdoutLogger
+func (StdoutLoggerBuilder) Build(audit.LoggerConfig) audit.Logger {
+	return &StdoutLogger{}
+}
+
+// Stdout implementation of audit.LoggerBuilder.ParseLoggerConfig
+// Passed value is ignored but warning is printed
+func (StdoutLoggerBuilder) ParseLoggerConfig(config json.RawMessage) (audit.LoggerConfig, error) {
+	if config != nil {
+		grpcLogger.Warningf("Config value %v ignored, "+
+			"StdoutLogger doesn't support custom configs", string(config))
+	}
+	return &StdoutLoggerConfig{}, nil
+}
+
+func convertEvent(event *audit.Event) stdoutEvent {
+	return stdoutEvent{
+		FullMethodName: event.FullMethodName,
+		Principal:      event.Principal,
+		PolicyName:     event.PolicyName,
+		MatchedRule:    event.MatchedRule,
+		Authorized:     event.Authorized,
+		Timestamp:      time.Now().Format(time.RFC3339),
+	}
+}

authz/audit/stdout/stdout_logger_test.go

@@ -0,0 +1,81 @@
+/*
+ *
+ * Copyright 2023 gRPC authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ */
+
+package stdout
+
+import (
+	"bytes"
+	"encoding/json"
+	"log"
+	"testing"
+	"time"
+
+	"google.golang.org/grpc/authz/audit"
+)
+
+var (
+	content     = json.RawMessage(`{"name": "conf", "val": "to be ignored"}`)
+	builder     = &StdoutLoggerBuilder{}
+	config, _   = builder.ParseLoggerConfig(content)
+	auditLogger = builder.Build(config)
+)
+
+func TestStdoutLoggerBuilder_NilConfig(t *testing.T) {
+	builder = &StdoutLoggerBuilder{}
+	config, err := builder.ParseLoggerConfig(nil)
+	if err != nil {
+		t.Fatalf("unexpected error\n%v", err)
+	}
+	auditLogger = builder.Build(config)
+}
+
+func TestStdoutLogger_Log(t *testing.T) {
+	var buf bytes.Buffer
+	log.SetOutput(&buf)
+	log.SetFlags(0)
+
+	event := &audit.Event{PolicyName: "test policy", Principal: "test principal"}
+	auditLogger.Log(event)
+
+	expected := `{"fullMethodName":"","principal":"test principal","policyName":"test policy","matchedRule":"","authorized":false`
+	if buf.String() != (expected + ",\"timestamp\":\"" + time.Now().Format(time.RFC3339) + "\"}\n") {
+		t.Fatalf("unexpected error\nwant:%v\n got:%v", expected, buf.String())
+	}
+}
+
+func TestStdoutLogger_LogAllEventFields(t *testing.T) {
+	var buf bytes.Buffer
+	log.SetOutput(&buf)
+	log.SetFlags(0)
+
+	event := &audit.Event{
+		FullMethodName: "/helloworld.Greeter/SayHello",
+		Principal:      "spiffe://example.org/ns/default/sa/default/backend",
+		PolicyName:     "example-policy",
+		MatchedRule:    "dev-access",
+		Authorized:     true,
+	}
+	auditLogger.Log(event)
+
+	expected := `{"fullMethodName":"/helloworld.Greeter/SayHello",` +
+		`"principal":"spiffe://example.org/ns/default/sa/default/backend","policyName":"example-policy",` +
+		`"matchedRule":"dev-access","authorized":true`
+	if buf.String() != (expected + ",\"timestamp\":\"" + time.Now().Format(time.RFC3339) + "\"}\n") {
+		t.Fatalf("unexpected error\nwant:%v\n got:%v", expected, buf.String())
+	}
+}