fix(oonimkall): make sure SoftwareName is set

While there, make sure we log the check-in request and response with `-v`, because otherwise 🤌🤌🤌🤌. Both part of ooni/probe#2644.
ooni · Dec 12, 2023 · 645cecb · 645cecb
1 parent d84729d
commit 645cecb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/internal/ooapi/checkin.go b/internal/ooapi/checkin.go
@@ -25,7 +25,7 @@ func NewDescriptorCheckIn(
 		AcceptEncodingGzip: true, // we want a small response
 		Authorization:      "",
 		ContentType:        httpapi.ApplicationJSON,
-		LogBody:            false, // we don't want to log psiphon config
+		LogBody:            true,
 		MaxBodySize:        0,
 		Method:             http.MethodPost,
 		Request: &httpapi.RequestDescriptor[*model.OOAPICheckInConfig]{

diff --git a/pkg/oonimkall/session.go b/pkg/oonimkall/session.go
@@ -467,6 +467,7 @@ func (sess *Session) CheckIn(ctx *Context, config *CheckInConfig) (*CheckInInfo,
 		ProbeASN:        info.ASNString(),
 		ProbeCC:         info.CountryCode,
 		RunType:         model.RunType(config.RunType),
+		SoftwareName:    config.SoftwareName,
 		SoftwareVersion: config.SoftwareVersion,
 		WebConnectivity: config.WebConnectivity.toModel(),
 	}