Skip to content

Commit d9418fe

Browse files
committed
Remove newlines from log messages
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>
1 parent b94918c commit d9418fe

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

main.go

+9-7
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ func main() {
4747
messageHandler := func(msg *stan.Msg) {
4848
i := atomic.AddUint64(&counter, 1)
4949

50-
log.Printf("[#%d] Received on [%s]: '%s'\n", i, msg.Subject, msg)
50+
log.Printf("[#%d] Received on [%s]: '%s'", i, msg.Subject, msg)
5151

5252
started := time.Now()
5353

@@ -60,7 +60,7 @@ func main() {
6060
xCallID := req.Header.Get("X-Call-Id")
6161

6262
functionURL := makeFunctionURL(&req, &config, req.Path, req.QueryString)
63-
fmt.Printf("[#%d] Invoking: %s with %d bytes, via: %s\n", i, req.Function, len(req.Body), functionURL)
63+
fmt.Printf("[#%d] Invoking: %s with %d bytes, via: %s", i, req.Function, len(req.Body), functionURL)
6464

6565
if config.DebugPrintBody {
6666
fmt.Println(string(req.Body))
@@ -73,6 +73,8 @@ func main() {
7373
return
7474
}
7575

76+
req.Header.Set("User-Agent", "openfaas-ce/nats-queue-worker")
77+
7678
defer request.Body.Close()
7779
copyHeaders(request.Header, &req.Header)
7880

@@ -110,7 +112,7 @@ func main() {
110112
timeTaken)
111113

112114
if err != nil {
113-
log.Printf("[#%d] Posted callback to: %s - status %d, error: %s\n", i, req.CallbackURL.String(), http.StatusServiceUnavailable, err.Error())
115+
log.Printf("[#%d] Posted callback to: %s - status %d, error: %s", i, req.CallbackURL.String(), http.StatusServiceUnavailable, err.Error())
114116
} else {
115117
log.Printf("[#%d] Posted result to %s - status: %d", i, req.CallbackURL.String(), resultStatusCode)
116118
}
@@ -132,14 +134,14 @@ func main() {
132134
if config.WriteDebug {
133135
fmt.Println(string(functionResult))
134136
} else {
135-
fmt.Printf("[#%d] %s returned %d bytes\n", i, req.Function, len(functionResult))
137+
fmt.Printf("[#%d] %s returned %d bytes", i, req.Function, len(functionResult))
136138
}
137139
}
138140

139141
timeTaken := time.Since(started).Seconds()
140142

141143
if req.CallbackURL != nil {
142-
log.Printf("[#%d] Callback to: %s\n", i, req.CallbackURL.String())
144+
log.Printf("[#%d] Callback to: %s", i, req.CallbackURL.String())
143145

144146
resultStatusCode, err := postResult(&client,
145147
res,
@@ -151,7 +153,7 @@ func main() {
151153
timeTaken)
152154

153155
if err != nil {
154-
log.Printf("[#%d] Error posting to callback-url: %s\n", i, err)
156+
log.Printf("[#%d] Error posting to callback-url: %s", i, err)
155157
} else {
156158
log.Printf("[#%d] Posted result for %s to callback-url: %s, status: %d", i, req.Function, req.CallbackURL.String(), resultStatusCode)
157159
}
@@ -190,7 +192,7 @@ func main() {
190192

191193
fmt.Printf("\nReceived an interrupt, unsubscribing and closing connection...\n\n")
192194
if err := natsQueue.closeConnection(); err != nil {
193-
log.Panicf("Cannot close connection to %s because of an error: %v\n", natsQueue.natsURL, err)
195+
log.Panicf("Cannot close connection to %s because of an error: %v", natsQueue.natsURL, err)
194196
}
195197
close(signalChan)
196198
}

0 commit comments

Comments
 (0)