Make telemetry explicit (#668)

README.md

@@ -44,9 +44,6 @@ Digger has 2 main components:
 
 Digger also stores PR-level locks and plan cache in your cloud account (DynamoDB + S3 on AWS, equivalents in other cloud providers)
 
-## Telemetry
-No sensitive or personal / identifyable data is logged. You can see what is tracked in [`pkg/usage/usage.go`](https://github.com/diggerhq/digger/blob/main/pkg/usage/usage.go)
-
 ## Contributing
 
 We love contributions. Check out our [contribiting guide](CONTRIBUTING.md) to get started. 

pkg/usage/usage.go

@@ -2,13 +2,12 @@ package usage
 
 import (
 	"bytes"
-	"crypto/sha256"
-	"encoding/hex"
 	"encoding/json"
-	configuration "github.com/diggerhq/lib-digger-config"
 	"log"
 	"net/http"
 	"os"
+
+	configuration "github.com/diggerhq/lib-digger-config"
 )
 
 var collect_usage_data = true
@@ -22,12 +21,8 @@ type UsageRecord struct {
 }
 
 func SendUsageRecord(repoOwner string, eventName string, action string) error {
-	h := sha256.New()
-	h.Write([]byte(repoOwner))
-	sha := h.Sum(nil)
-	shaStr := hex.EncodeToString(sha)
 	payload := UsageRecord{
-		UserId:    shaStr,
+		UserId:    repoOwner,
 		EventName: eventName,
 		Action:    action,
 		Token:     "diggerABC@@1998fE",
@@ -36,12 +31,8 @@ func SendUsageRecord(repoOwner string, eventName string, action string) error {
 }
 
 func SendLogRecord(repoOwner string, message string) error {
-	h := sha256.New()
-	h.Write([]byte(repoOwner))
-	sha := h.Sum(nil)
-	shaStr := hex.EncodeToString(sha)
 	payload := UsageRecord{
-		UserId:    shaStr,
+		UserId:    repoOwner,
 		EventName: "log from " + source,
 		Action:    message,
 		Token:     "diggerABC@@1998fE",