consider sharing structured error log as function

[ldemailly]

From Wiard

"It took me (too) long to realise that you are setting err here for line 383 if I'm honest. Same goes for the logic at line 394.

I do understand you want to reuse the contex of the logger... but then maybe make a small func as in "return err with context" and then do something like"

return nil, returnErrorWithContext("invalid url")

where returnErrorWithContext func is the contents of:

		log.S(log.Error, "Unable to make request",
			log.Attr("method", method), log.Attr("url", o.URL), log.Attr("err", err),
			log.Attr("thread", o.ID), log.Attr("run", o.UniqueID))

Originally posted by @wiardvanrij in #785 (comment)

see thread