[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-15 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r488413639 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-12 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486965508 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-12 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486965508 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-12 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486965508 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-11 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486966337 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-11 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486965508 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-08 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r485311928 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-08 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r485309909 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-08 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r485103731 ## File path: t/lib/test_admin.lua ## @@ -128,6 +128,7 @@ function _M.test(uri, method, body, pattern, headers) end if not

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-08-30 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r479869859 ## File path: conf/config.yaml ## @@ -21,3 +21,13 @@ # host: # - "http://127.0.0.1:2379; # +# +# If you want specify the Admin API token,

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-08-30 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r479867959 ## File path: .travis/apisix_cli_test.sh ## @@ -166,7 +200,14 @@ echo "passed: admin https enabled" # rollback to the default -git checkout

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-08-25 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r477014633 ## File path: t/APISIX.pm ## @@ -81,7 +81,11 @@ apisix: stream_proxy: tcp: - 9100 - admin_key: null + admin_key: +- + name:

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-08-25 Thread GitBox
membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r477014540 ## File path: t/admin/token.t ## @@ -155,7 +164,7 @@ PUT /apisix/admin/plugins/reload === TEST 6: reload plugins with api key(arguments) --- request