Bladeren bron

Merge pull request #12173 from zhongwencool/authz-headers

feat: don't merge authz default headers if user already setting
zhongwencool 2 jaren geleden
bovenliggende
commit
4cab42bbbe

+ 1 - 1
apps/emqx_auth_http/src/emqx_auth_http.app.src

@@ -1,7 +1,7 @@
 %% -*- mode: erlang -*-
 {application, emqx_auth_http, [
     {description, "EMQX External HTTP API Authentication and Authorization"},
-    {vsn, "0.1.1"},
+    {vsn, "0.1.2"},
     {registered, []},
     {mod, {emqx_auth_http_app, []}},
     {applications, [

+ 2 - 2
apps/emqx_auth_http/src/emqx_authz_http_schema.erl

@@ -113,7 +113,7 @@ headers(desc) ->
     ?DESC(?FUNCTION_NAME);
 headers(converter) ->
     fun(Headers) ->
-        maps:to_list(maps:merge(default_headers(), transform_header_name(Headers)))
+        maps:to_list(transform_header_name(Headers))
     end;
 headers(default) ->
     default_headers();
@@ -129,7 +129,7 @@ headers_no_content_type(converter) ->
         maps:to_list(
             maps:without(
                 [<<"content-type">>],
-                maps:merge(default_headers_no_content_type(), transform_header_name(Headers))
+                transform_header_name(Headers)
             )
         )
     end;