[GitHub] [incubator-apisix] xiangxiren opened a new issue #1207: why not merge parameters of the same type in route management

xiangxiren opened a new issue #1207: why not merge parameters of the same type in route management URL: https://github.com/apache/incubator-apisix/issues/1207 in routing management,why not merge these same types of parameters into one.Isn't this more concise and clear? ![image](https:/

[GitHub] [incubator-apisix] chnliyong commented on issue #1188: feature: Support for OAuth User Managed Access Protocol(UMA) for Authorization

chnliyong commented on issue #1188: feature: Support for OAuth User Managed Access Protocol(UMA) for Authorization URL: https://github.com/apache/incubator-apisix/issues/1188#issuecomment-595070533 Hi @sshniro , As I'm a little bit familiar with **Keycloak**, I've read the Keycloak

[incubator-apisix-dashboard] branch master updated: feat: 增加 API Key

This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-apisix-dashboard.git The following commit(s) were added to refs/heads/master by this push: new b3b3065 feat: 增加 API Key

[incubator-apisix] branch master updated: feature: supported key-based authentication to the dashboard. (#1169)

This is an automated email from the ASF dual-hosted git repository. wenming pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-apisix.git The following commit(s) were added to refs/heads/master by this push: new be2eec3 feature: supported key-based

[GitHub] [incubator-apisix] moonming merged pull request #1169: feature: supported key-based authentication to the dashboard.

moonming merged pull request #1169: feature: supported key-based authentication to the dashboard. URL: https://github.com/apache/incubator-apisix/pull/1169 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r388093075 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix-dashboard] jiezaizone opened a new issue #134: We're sorry but Dashboard for APISIX doesn't work properly without JavaScript enabled. Please enable it to continue.

jiezaizone opened a new issue #134: We're sorry but Dashboard for APISIX doesn't work properly without JavaScript enabled. Please enable it to continue. URL: https://github.com/apache/incubator-apisix-dashboard/issues/134 ![image](https://user-images.githubusercontent.com/2897/75942237

[GitHub] [incubator-apisix-dashboard] jiezaizone opened a new issue #133: The navigation bar is not visible

jiezaizone opened a new issue #133: The navigation bar is not visible URL: https://github.com/apache/incubator-apisix-dashboard/issues/133  导航栏不可见，也没有收缩，但可以点击 --

[GitHub] [incubator-apisix] moonming commented on issue #1206: CI: enabled all CI env.

moonming commented on issue #1206: CI: enabled all CI env. URL: https://github.com/apache/incubator-apisix/pull/1206#issuecomment-594989979 There are blocking bugs in both Mac and lualocks environments This is an automated mes

[GitHub] [incubator-apisix] moonming opened a new pull request #1206: CI: enabled all CI env.

moonming opened a new pull request #1206: CI: enabled all CI env. URL: https://github.com/apache/incubator-apisix/pull/1206 Previously, some tests were closed due to the long running time of Travis, but now they must be opened to avoid these CI becoming useless decorations

[GitHub] [incubator-apisix] moonming opened a new pull request #1205: Revert "feature: auto import nameservers into APISIX from system reso…

moonming opened a new pull request #1205: Revert "feature: auto import nameservers into APISIX from system reso… URL: https://github.com/apache/incubator-apisix/pull/1205 …lver. (#1167)" This reverts commit e1ac426443293c6140ddb257e482537edd43c36a. This PR break CI and no test cas

[GitHub] [incubator-apisix] moonming commented on issue #1167: feature: auto import nameservers into APISIX from system resolv file.…

moonming commented on issue #1167: feature: auto import nameservers into APISIX from system resolv file.… URL: https://github.com/apache/incubator-apisix/pull/1167#issuecomment-594984943 this PR break the CI: https://travis-ci.org/apache/incubator-apisix/jobs/658023224#L1420 -

[GitHub] [incubator-apisix] moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.…

moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.… URL: https://github.com/apache/incubator-apisix/pull/1167#discussion_r388035389 ## File path: bin/apisix ## @@ -603,6 +631,18 @@ local function init

[GitHub] [incubator-apisix] moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.…

moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.… URL: https://github.com/apache/incubator-apisix/pull/1167#discussion_r388035303 ## File path: bin/apisix ## @@ -603,6 +631,18 @@ local function init

[GitHub] [incubator-apisix] iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r388026003 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -77,6 +81,18 @@ local function crea

[GitHub] [incubator-apisix] iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r388029992 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -351,71 +359,154 @@ local function

[GitHub] [incubator-apisix] moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.…

moonming commented on a change in pull request #1167: feature: auto import nameservers into APISIX from system resolv file.… URL: https://github.com/apache/incubator-apisix/pull/1167#discussion_r388029122 ## File path: .travis/apisix_cli_test.sh ## @@ -52,3 +52,17 @@ if [

[GitHub] [incubator-apisix] moonming commented on issue #1167: feature: auto import nameservers into APISIX from system resolv file.…

moonming commented on issue #1167: feature: auto import nameservers into APISIX from system resolv file.… URL: https://github.com/apache/incubator-apisix/pull/1167#issuecomment-594976329 @qiujiayu @membphis two questions: 1. why not use the `resolver local=on`? 2. we need a switch to

[GitHub] [incubator-apisix] iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

iGeeky commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r388026003 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -77,6 +81,18 @@ local function crea

[GitHub] [incubator-apisix] moonming commented on issue #1200: Missing global rules usage

moonming commented on issue #1200: Missing global rules usage URL: https://github.com/apache/incubator-apisix/issues/1200#issuecomment-594970976 @ShiningRush Apache APISIX not has the CROS plugin yet, welcome PR for CORS. Thi

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path

membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r388000511 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' the

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path

membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r388000511 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' the

[GitHub] [incubator-apisix] membphis commented on issue #1199: fix: global ruls admin api should list resource without id

membphis commented on issue #1199: fix: global ruls admin api should list resource without id URL: https://github.com/apache/incubator-apisix/pull/1199#issuecomment-594931872 https://travis-ci.org/apache/incubator-apisix/jobs/658211556#L722 travis fail --

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r387994968 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -351,71 +359,154 @@ local functio

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r387994640 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -351,71 +359,154 @@ local functio

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

membphis commented on a change in pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204#discussion_r387992672 ## File path: lua/apisix/plugins/wolf-rbac.lua ## @@ -77,6 +81,18 @@ local function cr

[GitHub] [incubator-apisix] Akayeshmantha commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks

Akayeshmantha commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks URL: https://github.com/apache/incubator-apisix/issues/1184#issuecomment-594716236 Hi @membphis I was looking through this issue. And as I found lua_path is set on below

[GitHub] [incubator-apisix] moonming closed issue #1198: request help: How to rewrite/modify response in APISIX

moonming closed issue #1198: request help: How to rewrite/modify response in APISIX URL: https://github.com/apache/incubator-apisix/issues/1198 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-apisix] moonming commented on issue #1198: request help: How to rewrite/modify response in APISIX

moonming commented on issue #1198: request help: How to rewrite/modify response in APISIX URL: https://github.com/apache/incubator-apisix/issues/1198#issuecomment-594560837 @sshniro You can also send these questions to the dev mailing list:) --

[GitHub] [incubator-apisix] iGeeky opened a new pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`.

iGeeky opened a new pull request #1204: Wolf-Rbac: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1204 ### Add new api `change_pwd` and `user_info` * Add api `/apisix/plugin/wolf-rbac/change_pwd` used to change password * Add api `/a

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387690590 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387686415 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix] jie123108 closed pull request #1203: wolf-rbac: add new api `change_pwd` and `user_info`.

jie123108 closed pull request #1203: wolf-rbac: add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1203 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387686415 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387686415 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix] sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path

sshniro commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387686415 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' then

[GitHub] [incubator-apisix] sshniro commented on issue #1201: Adding warning message when running APISIX in root path

sshniro commented on issue #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#issuecomment-594542660 > BTW, if the current user is a root user, we need to add `user root;` in `conf/nginx.conf`, otherwise, the APISIX inst

[GitHub] [incubator-apisix] moonming commented on issue #1189: request help: How to implement thread sleep in Nginx Test

moonming commented on issue #1189: request help: How to implement thread sleep in Nginx Test URL: https://github.com/apache/incubator-apisix/issues/1189#issuecomment-594537957 @sshniro you can try `ngx.sleep(2)`. This is an

[GitHub] [incubator-apisix] membphis commented on issue #1203: wolf-rbac: add new api `change_pwd` and `user_info`.

membphis commented on issue #1203: wolf-rbac: add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1203#issuecomment-594523117 @iGeeky do you have time to review this PR? This is an a

[GitHub] [incubator-apisix] membphis commented on issue #1201: Adding warning message when running APISIX in root path

membphis commented on issue #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#issuecomment-594520712 BTW, if the current user is a root user, we need to add `user root;` in `conf/nginx.conf`, otherwise, the APISIX insta

[GitHub] [incubator-apisix] jie123108 opened a new pull request #1203: Add new api `change_pwd` and `user_info`.

jie123108 opened a new pull request #1203: Add new api `change_pwd` and `user_info`. URL: https://github.com/apache/incubator-apisix/pull/1203 ### Add new api `change_pwd` and `user_info` * Add api `/apisix/plugin/wolf-rbac/change_pwd` used to change password * Add api `/apisix/pl

[GitHub] [incubator-apisix] membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path

membphis commented on a change in pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#discussion_r387661302 ## File path: bin/apisix ## @@ -50,6 +50,12 @@ if script_path:sub(1, 2) == './' the

[GitHub] [incubator-apisix] membphis commented on issue #1201: Adding warning message when running APISIX in root path

membphis commented on issue #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201#issuecomment-594515809 You are so cool, submit a PR so soon This is an autom

[GitHub] [incubator-apisix] membphis commented on issue #1202: bug: prometheus and zipkin do not support global_rule

membphis commented on issue #1202: bug: prometheus and zipkin do not support global_rule URL: https://github.com/apache/incubator-apisix/issues/1202#issuecomment-594514708 We need to modify the Apache APISIX core to resolve this problem. ---

[GitHub] [incubator-apisix] membphis commented on issue #1200: Missing global rules usage

membphis commented on issue #1200: Missing global rules usage URL: https://github.com/apache/incubator-apisix/issues/1200#issuecomment-594513102 You are right, PR welcome. This is an automated message from the Apache Git Ser

[GitHub] [incubator-apisix] membphis closed issue #441: doc: how to write a plugin step by step?

membphis closed issue #441: doc: how to write a plugin step by step? URL: https://github.com/apache/incubator-apisix/issues/441 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-apisix] membphis commented on issue #441: doc: how to write a plugin step by step?

membphis commented on issue #441: doc: how to write a plugin step by step? URL: https://github.com/apache/incubator-apisix/issues/441#issuecomment-594511216 fixed already This is an automated message from the Apache Git Servi

[GitHub] [incubator-apisix] rhubard opened a new issue #1202: bug: prometheus and zipkin do not support global_rule

rhubard opened a new issue #1202: bug: prometheus and zipkin do not support global_rule URL: https://github.com/apache/incubator-apisix/issues/1202 ### Issue description The global_rule plugin is not executed during the log phase ### Environment * apisix version (cmd: `apis

[GitHub] [incubator-apisix] sshniro commented on issue #1198: request help: How to rewrite/modify response in APISIX

sshniro commented on issue #1198: request help: How to rewrite/modify response in APISIX URL: https://github.com/apache/incubator-apisix/issues/1198#issuecomment-594503649 Thanks, will try this. This is an automated message

[GitHub] [incubator-apisix] sshniro opened a new pull request #1201: Adding warning message when running APISIX in root path

sshniro opened a new pull request #1201: Adding warning message when running APISIX in root path URL: https://github.com/apache/incubator-apisix/pull/1201 Fix #1196 This is an automated message from the Apache Git Service

[GitHub] [incubator-apisix] ShiningRush opened a new issue #1200: Missing global rules usage

ShiningRush opened a new issue #1200: Missing global rules usage URL: https://github.com/apache/incubator-apisix/issues/1200 Our team need a global plugin to enable CORS of service, then i found `global_rules`, but i didnt see any docs about it. I think i could append its description to

[GitHub] [incubator-apisix] membphis commented on issue #1199: fix: global ruls admin api should list resource without id

membphis commented on issue #1199: fix: global ruls admin api should list resource without id URL: https://github.com/apache/incubator-apisix/pull/1199#issuecomment-594494745 @ShiningRush we'd better add some test cases to confirm it is right. --

[GitHub] [incubator-apisix] ShiningRush opened a new pull request #1199: fix: global ruls admin api should list resource without id

ShiningRush opened a new pull request #1199: fix: global ruls admin api should list resource without id URL: https://github.com/apache/incubator-apisix/pull/1199 # Summary apisix admin api will return error when access "/apisix/admin/global_rules", ### Issues resolved Fix:

[GitHub] [incubator-apisix] membphis commented on issue #1196: improvement: print a warning message when running APISIX in `/root` directory

membphis commented on issue #1196: improvement: print a warning message when running APISIX in `/root` directory URL: https://github.com/apache/incubator-apisix/issues/1196#issuecomment-594464734 Yes, you are absolutely right. We can check the APISIX working folder. https://github.c

[GitHub] [incubator-apisix] membphis commented on issue #1198: request help: How to rewrite/modify response in APISIX

membphis commented on issue #1198: request help: How to rewrite/modify response in APISIX URL: https://github.com/apache/incubator-apisix/issues/1198#issuecomment-594463261 You can take a look at this plugin: https://github.com/apache/incubator-apisix/blob/master/doc/plugins/response-rewr

[GitHub] [incubator-apisix] membphis closed issue #1152: request help: fail to mqtt-proxy request

membphis closed issue #1152: request help: fail to mqtt-proxy request URL: https://github.com/apache/incubator-apisix/issues/1152 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-apisix] membphis commented on issue #1152: request help: fail to mqtt-proxy request

membphis commented on issue #1152: request help: fail to mqtt-proxy request URL: https://github.com/apache/incubator-apisix/issues/1152#issuecomment-594459921 https://github.com/apache/incubator-apisix/blob/master/lua/apisix/stream/router/ip_port.lua#L43 I think the latest APISIX h

[GitHub] [incubator-apisix] membphis commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks

membphis commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks URL: https://github.com/apache/incubator-apisix/issues/1184#issuecomment-594434863 > @membphis I would like to work on this issue OK, I have assigned it to you. -

[GitHub] [incubator-apisix] Akayeshmantha commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks

Akayeshmantha commented on issue #1184: bug: failed to call `apisix init` after apisix was installed by luarocks URL: https://github.com/apache/incubator-apisix/issues/1184#issuecomment-594412238 @membphis I would like to work on this issue --

[GitHub] [incubator-apisix] JohannT9527 commented on issue #1152: request help: fail to mqtt-proxy request

JohannT9527 commented on issue #1152: request help: fail to mqtt-proxy request URL: https://github.com/apache/incubator-apisix/issues/1152#issuecomment-594411307 你好，这个问题解决了吗？我遇到了跟你一模一样的问题。 This is an automated message from t

[GitHub] [incubator-apisix] moonming commented on issue #1195: feature: support multiple ports like stream listen in http and https listen port.

moonming commented on issue #1195: feature: support multiple ports like stream listen in http and https listen port. URL: https://github.com/apache/incubator-apisix/issues/1195#issuecomment-594394781 @wwd5613 welcome PR This

[GitHub] [incubator-apisix] sshniro edited a comment on issue #1189: request help: How to implement thread sleep in Nginx Test

sshniro edited a comment on issue #1189: request help: How to implement thread sleep in Nginx Test URL: https://github.com/apache/incubator-apisix/issues/1189#issuecomment-594379479 I would like to write a test case to test the inactive timeout. The following are the configurations. -

[GitHub] [incubator-apisix] sshniro opened a new issue #1198: request help: How to rewrite/modify response in APISIX

sshniro opened a new issue #1198: request help: How to rewrite/modify response in APISIX URL: https://github.com/apache/incubator-apisix/issues/1198 ### Issue description I would like to remove some auth headers from the upstream response. Is this possible with APISIX? Is the proxy r

[GitHub] [incubator-apisix] sshniro commented on issue #1196: improvement: print a warning message when running APISIX in `/root` directory

sshniro commented on issue #1196: improvement: print a warning message when running APISIX in `/root` directory URL: https://github.com/apache/incubator-apisix/issues/1196#issuecomment-594387468 Hi, @membphis just to clarify this check should be done inside the `/bin/apisix file` rite? To

[GitHub] [incubator-apisix] sshniro commented on issue #1186: request help: How to test the log_utils?

sshniro commented on issue #1186: request help: How to test the log_utils? URL: https://github.com/apache/incubator-apisix/issues/1186#issuecomment-594381751 Thanks, will try this out. This is an automated message from the Ap

[GitHub] [incubator-apisix] sshniro commented on issue #1189: request help: How to implement thread sleep in Nginx Test

sshniro commented on issue #1189: request help: How to implement thread sleep in Nginx Test URL: https://github.com/apache/incubator-apisix/issues/1189#issuecomment-594379479 I would like to write a test case to test the inactive timeout. The following are the configurations. - Inactiv