Fix options handling (#396)

Author: bryannaegele

instrumentation/opentelemetry_cowboy/src/opentelemetry_cowboy.erl

@@ -124,11 +124,7 @@ setup(Opts) when is_list(Opts) ->
     setup(maps:from_list(Opts));
 setup(Opts) ->
     InitialConfig = maps:merge(default_opts(), Opts),
-    OptInAttrs = lists:filtermap(fun({Name, OptedIn}) ->
-        case OptedIn of
-            true -> {true, Name};
-            false -> false end
-        end, maps:get(opt_in_attrs, InitialConfig)),
+    OptInAttrs = maps:get(opt_in_attrs, InitialConfig),
     ReversedClientAddressHeaders = lists:reverse(maps:get(client_address_headers, InitialConfig)),
     ReversedSchemeHeaders = lists:reverse(maps:get(scheme_headers, InitialConfig)),
     FinalOpts = maps:merge(InitialConfig, #{

instrumentation/opentelemetry_cowboy/test/opentelemetry_cowboy_SUITE.erl

@@ -153,12 +153,12 @@ public_endpoint_true_fn(_Req, _Opts) ->
 with_all_optins(_Config) ->
     Opts = #{
       opt_in_attrs => [
-        {?CLIENT_PORT, true},
-        {?HTTP_REQUEST_BODY_SIZE, true},
-        {?HTTP_RESPONSE_BODY_SIZE, true},
-        {?NETWORK_LOCAL_ADDRESS, true},
-        {?NETWORK_LOCAL_PORT, true},
-        {?NETWORK_TRANSPORT, true}
+        ?CLIENT_PORT,
+        ?HTTP_REQUEST_BODY_SIZE,
+        ?HTTP_RESPONSE_BODY_SIZE,
+        ?NETWORK_LOCAL_ADDRESS,
+        ?NETWORK_LOCAL_PORT,
+        ?NETWORK_TRANSPORT
       ],
       request_headers => [<<"test-header">>],
       response_headers => [<<"content-type">>]