fix: don't attempt to propagate a null span. (#120)

Strictly this should not happen, we should always get a spanContext,
however, this is not currently the case.

Author: tcolgate

packages/opencensus-instrumentation-http/src/http.ts

@@ -256,6 +256,11 @@ export class HttpPlugin extends BasePlugin {
     return (span: Span): httpModule.ClientRequest => {
       plugin.logger.debug('makeRequestTrace');
 
+      if (!span) {
+        plugin.logger.debug('makeRequestTrace span is null');
+        return request;
+      }
+
       const setter: HeaderSetter = {
         setHeader(name: string, value: string) {
           request.setHeader(name, value);
@@ -267,11 +272,6 @@ export class HttpPlugin extends BasePlugin {
         propagation.inject(setter, span.spanContext);
       }
 
-      if (!span) {
-        plugin.logger.debug('makeRequestTrace span is null');
-        return request;
-      }
-
       request.on('response', (response: httpModule.ClientResponse) => {
         plugin.tracer.wrapEmitter(response);
         plugin.logger.debug('outgoingRequest on response()');