[GitHub] [incubator-apisix] spacewander opened a new pull request #801: chore: fixed typo detected by misspell

2019-11-04 Thread GitBox
spacewander opened a new pull request #801: chore: fixed typo detected by misspell URL: https://github.com/apache/incubator-apisix/pull/801 To make `t/plugin/ip-restriction.t` pass, need to make a new release of https://github.com/iresty/jsonschema.

[GitHub] [incubator-apisix] Miss-you opened a new issue #802: request help: Is there any plan that the APISIX supports the Consul?

2019-11-04 Thread GitBox
Miss-you opened a new issue #802: request help: Is there any plan that the APISIX supports the Consul? URL: https://github.com/apache/incubator-apisix/issues/802 ### Issue description Consul is similar with etcd, and there are some company or individuals which use the consul instead of

[GitHub] [incubator-apisix] shuaijinchao commented on issue #733: 支持nginx的proxy_pass路由转发功能

2019-11-04 Thread GitBox
shuaijinchao commented on issue #733: 支持nginx的proxy_pass路由转发功能 URL: https://github.com/apache/incubator-apisix/issues/733#issuecomment-549396440 @jiagoumengxiang `proxy-rewrite` plugin document. please see: https://github.com/iresty/apisix/blob/master/doc/plugins/proxy-rewrite.md

[GitHub] [incubator-apisix] Miss-you opened a new issue #803: request help: Does the APISIX need to write the configuration to the local disk?

2019-11-04 Thread GitBox
Miss-you opened a new issue #803: request help: Does the APISIX need to write the configuration to the local disk? URL: https://github.com/apache/incubator-apisix/issues/803 ### Issue description Does the APISIX need to write the configuration to the local disk? The APISIX

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new issue #64: feature: it's better for users can choose chash keys from a dropdown list

2019-11-04 Thread GitBox
lilien1010 opened a new issue #64: feature: it's better for users can choose chash keys from a dropdown list URL: https://github.com/apache/incubator-apisix-dashboard/issues/64 This is an automated message from the Apache

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new pull request #65: enhance: can choose chash keys from drop down list

2019-11-04 Thread GitBox
lilien1010 opened a new pull request #65: enhance: can choose chash keys from drop down list URL: https://github.com/apache/incubator-apisix-dashboard/pull/65 fix: https://github.com/apache/incubator-apisix-dashboard/issues/64

[GitHub] [incubator-apisix] Miss-you opened a new issue #804: request help: Which one has a higher priority, Eureka or Consul?

2019-11-04 Thread GitBox
Miss-you opened a new issue #804: request help: Which one has a higher priority, Eureka or Consul? URL: https://github.com/apache/incubator-apisix/issues/804 ### Issue description If you want to integrate into the Spring Cloud community, you need to support the Spring Cloud service

[GitHub] [incubator-apisix-dashboard] moonming opened a new pull request #56: license: moved License information to LICENSE.

2019-11-01 Thread GitBox
moonming opened a new pull request #56: license: moved License information to LICENSE. URL: https://github.com/apache/incubator-apisix-dashboard/pull/56 fixed https://github.com/apache/incubator-apisix-dashboard/issues/54

[GitHub] [incubator-apisix] membphis closed issue #677: test: run test cases of APISIX on Tengine + lua-nginx-module

2019-11-01 Thread GitBox
membphis closed issue #677: test: run test cases of APISIX on Tengine + lua-nginx-module URL: https://github.com/apache/incubator-apisix/issues/677 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-apisix] membphis merged pull request #683: travis: supported to run apisix with tengine.

2019-11-01 Thread GitBox
membphis merged pull request #683: travis: supported to run apisix with tengine. URL: https://github.com/apache/incubator-apisix/pull/683 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix] moonming commented on issue #712: bug: Not send tracing data headers to upstream

2019-11-01 Thread GitBox
moonming commented on issue #712: bug: Not send tracing data headers to upstream URL: https://github.com/apache/incubator-apisix/issues/712#issuecomment-548991825 fixed by https://github.com/apache/incubator-apisix/pull/713

[GitHub] [incubator-apisix] moonming closed issue #712: bug: Not send tracing data headers to upstream

2019-11-01 Thread GitBox
moonming closed issue #712: bug: Not send tracing data headers to upstream URL: https://github.com/apache/incubator-apisix/issues/712 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix] moonming closed issue #533: add more configures of nginx to config.yaml

2019-11-01 Thread GitBox
moonming closed issue #533: add more configures of nginx to config.yaml URL: https://github.com/apache/incubator-apisix/issues/533 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-apisix] moonming closed issue #581: bug: the client suspended when APISIX benchmark for http KeepAlive using self-develop benchmark tool

2019-11-01 Thread GitBox
moonming closed issue #581: bug: the client suspended when APISIX benchmark for http KeepAlive using self-develop benchmark tool URL: https://github.com/apache/incubator-apisix/issues/581 This is an automated message from

[GitHub] [incubator-apisix] moonming closed issue #425: doc: add doc for hot reload plugins.

2019-11-01 Thread GitBox
moonming closed issue #425: doc: add doc for hot reload plugins. URL: https://github.com/apache/incubator-apisix/issues/425 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-apisix] moonming commented on issue #631: bug: SSL_do_handshake() failed (SSL: error:1417A179:SSL routines:tls_post_process_client_hello:cert cb error) while SSL handshaking, clien

2019-11-01 Thread GitBox
moonming commented on issue #631: bug: SSL_do_handshake() failed (SSL: error:1417A179:SSL routines:tls_post_process_client_hello:cert cb error) while SSL handshaking, clien URL: https://github.com/apache/incubator-apisix/issues/631#issuecomment-548991756 @a610569731 any update?

[GitHub] [incubator-apisix] membphis commented on issue #218: Support Centos6

2019-11-01 Thread GitBox
membphis commented on issue #218: Support Centos6 URL: https://github.com/apache/incubator-apisix/issues/218#issuecomment-548995895 The current APISIX has supported CentOS 6 already. This is an automated message from the

[GitHub] [incubator-apisix] membphis opened a new issue #790: new plugin: data-validator

2019-11-01 Thread GitBox
membphis opened a new issue #790: new plugin: data-validator URL: https://github.com/apache/incubator-apisix/issues/790 Apache APISIX currently has a very high-performance jsonschema implementation that can be used to validate request parameters or request bodies. When the

[GitHub] [incubator-apisix] membphis opened a new pull request #791: travis: removed the temporary folder `openresty-1.15.8.2`.

2019-11-01 Thread GitBox
membphis opened a new pull request #791: travis: removed the temporary folder `openresty-1.15.8.2`. URL: https://github.com/apache/incubator-apisix/pull/791 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-apisix] membphis merged pull request #791: travis: removed the temporary folder `openresty-1.15.8.2`.

2019-11-01 Thread GitBox
membphis merged pull request #791: travis: removed the temporary folder `openresty-1.15.8.2`. URL: https://github.com/apache/incubator-apisix/pull/791 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-apisix] moonming commented on issue #415: doc: add doc for ip blacklist and whitelist plugin.

2019-11-01 Thread GitBox
moonming commented on issue #415: doc: add doc for ip blacklist and whitelist plugin. URL: https://github.com/apache/incubator-apisix/issues/415#issuecomment-548992188 fixed: https://github.com/apache/incubator-apisix/blob/master/doc/plugins/ip-restriction.md

[GitHub] [incubator-apisix] moonming closed issue #415: doc: add doc for ip blacklist and whitelist plugin.

2019-11-01 Thread GitBox
moonming closed issue #415: doc: add doc for ip blacklist and whitelist plugin. URL: https://github.com/apache/incubator-apisix/issues/415 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix] moonming closed issue #421: doc: Add screenshots of dashboard operations for docs of plugins

2019-11-01 Thread GitBox
moonming closed issue #421: doc: Add screenshots of dashboard operations for docs of plugins URL: https://github.com/apache/incubator-apisix/issues/421 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-apisix] moonming commented on issue #218: Support Centos6

2019-11-01 Thread GitBox
moonming commented on issue #218: Support Centos6 URL: https://github.com/apache/incubator-apisix/issues/218#issuecomment-548991301 we removed lib3 and rapidjson from APISIX, so I think we can close this issue. This is an

[GitHub] [incubator-apisix] moonming closed issue #218: Support Centos6

2019-11-01 Thread GitBox
moonming closed issue #218: Support Centos6 URL: https://github.com/apache/incubator-apisix/issues/218 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-apisix] moonming closed issue #504: how to reload the local yaml file

2019-11-01 Thread GitBox
moonming closed issue #504: how to reload the local yaml file URL: https://github.com/apache/incubator-apisix/issues/504 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-apisix] moonming commented on issue #317: bug: json schema in rapidjson not support default value

2019-11-01 Thread GitBox
moonming commented on issue #317: bug: json schema in rapidjson not support default value URL: https://github.com/apache/incubator-apisix/issues/317#issuecomment-548991473 we removed rapidjson from APISIX, so we can close this issue.

[GitHub] [incubator-apisix] moonming closed issue #317: bug: json schema in rapidjson not support default value

2019-11-01 Thread GitBox
moonming closed issue #317: bug: json schema in rapidjson not support default value URL: https://github.com/apache/incubator-apisix/issues/317 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-apisix] moonming closed issue #515: Error: module 'apisix.core.yaml' not found

2019-11-01 Thread GitBox
moonming closed issue #515: Error: module 'apisix.core.yaml' not found URL: https://github.com/apache/incubator-apisix/issues/515 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-apisix] moonming commented on issue #515: Error: module 'apisix.core.yaml' not found

2019-11-01 Thread GitBox
moonming commented on issue #515: Error: module 'apisix.core.yaml' not found URL: https://github.com/apache/incubator-apisix/issues/515#issuecomment-548991553 consider resolved This is an automated message from the Apache

[GitHub] [incubator-apisix] moonming merged pull request #737: bugfix: skipped to init etcd if use local file as config center.

2019-11-01 Thread GitBox
moonming merged pull request #737: bugfix: skipped to init etcd if use local file as config center. URL: https://github.com/apache/incubator-apisix/pull/737 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-apisix] moonming closed issue #716: bug: yaml方式配置,还需要配置etcd

2019-11-01 Thread GitBox
moonming closed issue #716: bug: yaml方式配置,还需要配置etcd URL: https://github.com/apache/incubator-apisix/issues/716 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-apisix] moonming closed issue #702: ping contributors

2019-11-01 Thread GitBox
moonming closed issue #702: ping contributors URL: https://github.com/apache/incubator-apisix/issues/702 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-apisix] moonming opened a new pull request #789: doc: add mail list in readme.

2019-11-01 Thread GitBox
moonming opened a new pull request #789: doc: add mail list in readme. URL: https://github.com/apache/incubator-apisix/pull/789 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-apisix] moonming closed issue #666: bug: wrong value of ssl_ciphers

2019-11-01 Thread GitBox
moonming closed issue #666: bug: wrong value of ssl_ciphers URL: https://github.com/apache/incubator-apisix/issues/666 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-apisix] membphis commented on issue #788: Luarocks install: we can not direct users to use unreleased source code.

2019-11-01 Thread GitBox
membphis commented on issue #788: Luarocks install: we can not direct users to use unreleased source code. URL: https://github.com/apache/incubator-apisix/issues/788#issuecomment-548993028 the line of source code: https://github.com/apache/incubator-apisix/blob/master/Makefile#L124

[GitHub] [incubator-apisix] membphis merged pull request #789: doc: add mail list in readme.

2019-11-01 Thread GitBox
membphis merged pull request #789: doc: add mail list in readme. URL: https://github.com/apache/incubator-apisix/pull/789 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-apisix] membphis commented on issue #677: test: run test cases of APISIX on Tengine + lua-nginx-module

2019-11-01 Thread GitBox
membphis commented on issue #677: test: run test cases of APISIX on Tengine + lua-nginx-module URL: https://github.com/apache/incubator-apisix/issues/677#issuecomment-548994164 We need to patch the Nginx core, the OpenResty core need it. We need to find a fully patch list.

[GitHub] [incubator-apisix] membphis commented on issue #794: Feature prometheus

2019-11-02 Thread GitBox
membphis commented on issue #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#issuecomment-549018147 Cod style, you can install luacheck and check it on your local machine: ```shell [./lua/apisix/plugins/prometheus/exporter.lua]: 64: line

[GitHub] [incubator-apisix] lyyoung closed pull request #793: Feature update prometheus

2019-11-02 Thread GitBox
lyyoung closed pull request #793: Feature update prometheus URL: https://github.com/apache/incubator-apisix/pull/793 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-apisix] lyyoung opened a new pull request #793: Feature update prometheus

2019-11-02 Thread GitBox
lyyoung opened a new pull request #793: Feature update prometheus URL: https://github.com/apache/incubator-apisix/pull/793 feature: [#792 ](https://github.com/apache/incubator-apisix/issues/792) This is an automated message

[GitHub] [incubator-apisix] lyyoung opened a new issue #792: Feature: Update prometheus plugin

2019-11-02 Thread GitBox
lyyoung opened a new issue #792: Feature: Update prometheus plugin URL: https://github.com/apache/incubator-apisix/issues/792 The current Prometheus plug-in is relatively simple. According to the common business requirements, it is required to calculate the service-machine-level delay,

[GitHub] [incubator-apisix] lyyoung opened a new pull request #794: Feature prometheus

2019-11-02 Thread GitBox
lyyoung opened a new pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794 [#792 ](https://github.com/apache/incubator-apisix/issues/792) This is an automated message from the Apache

[GitHub] [incubator-apisix] lyyoung commented on issue #794: Feature prometheus

2019-11-02 Thread GitBox
lyyoung commented on issue #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#issuecomment-549022787 > Cod style, you can install luacheck and check it on your local machine: > > ```shell > [./lua/apisix/plugins/prometheus/exporter.lua]: >

[GitHub] [incubator-apisix] membphis opened a new pull request #805: updated jsonschema to v0.4.

2019-11-04 Thread GitBox
membphis opened a new pull request #805: updated jsonschema to v0.4. URL: https://github.com/apache/incubator-apisix/pull/805 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-apisix] membphis commented on issue #802: request help: Is there any plan that the APISIX supports the Consul?

2019-11-04 Thread GitBox
membphis commented on issue #802: request help: Is there any plan that the APISIX supports the Consul? URL: https://github.com/apache/incubator-apisix/issues/802#issuecomment-549623496 PR welcome ^_^ This is an automated

[GitHub] [incubator-apisix-dashboard] juzhiyuan opened a new pull request #67: Feature remove useless files

2019-11-04 Thread GitBox
juzhiyuan opened a new pull request #67: Feature remove useless files URL: https://github.com/apache/incubator-apisix-dashboard/pull/67 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix] membphis commented on issue #803: request help: Does the APISIX need to write the configuration to the local disk?

2019-11-04 Thread GitBox
membphis commented on issue #803: request help: Does the APISIX need to write the configuration to the local disk? URL: https://github.com/apache/incubator-apisix/issues/803#issuecomment-549622253 Currently OpenResty has no worker exit phase, we can use the timer to synchronize the

[GitHub] [incubator-apisix-dashboard] juzhiyuan merged pull request #67: Feature remove useless files

2019-11-04 Thread GitBox
juzhiyuan merged pull request #67: Feature remove useless files URL: https://github.com/apache/incubator-apisix-dashboard/pull/67 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-apisix] membphis opened a new pull request #806: change: used `lua-resty-prometheus` which is simpler.

2019-11-04 Thread GitBox
membphis opened a new pull request #806: change: used `lua-resty-prometheus` which is simpler. URL: https://github.com/apache/incubator-apisix/pull/806 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-apisix] membphis commented on issue #801: chore: fixed typo detected by misspell

2019-11-04 Thread GitBox
membphis commented on issue #801: chore: fixed typo detected by misspell URL: https://github.com/apache/incubator-apisix/pull/801#issuecomment-549611769 https://github.com/apache/incubator-apisix/pull/805 You can do a `rebase` after this PR was merged.

[GitHub] [incubator-apisix-dashboard] juzhiyuan opened a new issue #66: LICENSE: remove ALv2 header from original files

2019-11-04 Thread GitBox
juzhiyuan opened a new issue #66: LICENSE: remove ALv2 header from original files URL: https://github.com/apache/incubator-apisix-dashboard/issues/66 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-apisix] membphis merged pull request #801: chore: fixed typo detected by misspell

2019-11-04 Thread GitBox
membphis merged pull request #801: chore: fixed typo detected by misspell URL: https://github.com/apache/incubator-apisix/pull/801 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-apisix] membphis commented on issue #801: chore: fixed typo detected by misspell

2019-11-04 Thread GitBox
membphis commented on issue #801: chore: fixed typo detected by misspell URL: https://github.com/apache/incubator-apisix/pull/801#issuecomment-549641867 Many thanks @spacewander This is an automated message from the Apache

[GitHub] [incubator-apisix] soulsoul commented on issue #804: request help: Which one has a higher priority, Eureka or Consul?

2019-11-04 Thread GitBox
soulsoul commented on issue #804: request help: Which one has a higher priority, Eureka or Consul? URL: https://github.com/apache/incubator-apisix/issues/804#issuecomment-549618769 eureka +1 This is an automated message

[GitHub] [incubator-apisix] membphis closed issue #733: 支持nginx的proxy_pass路由转发功能

2019-11-04 Thread GitBox
membphis closed issue #733: 支持nginx的proxy_pass路由转发功能 URL: https://github.com/apache/incubator-apisix/issues/733 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-apisix] lilien1010 commented on a change in pull request #775: bugfix: support more built-in parameters when set chash balancer

2019-11-02 Thread GitBox
lilien1010 commented on a change in pull request #775: bugfix: support more built-in parameters when set chash balancer URL: https://github.com/apache/incubator-apisix/pull/775#discussion_r341833882 ## File path: doc/architecture-design-cn.md ## @@ -237,7 +237,7 @@ APISIX

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new issue #57: bug: when enter the router list , the Action column out of views

2019-11-03 Thread GitBox
lilien1010 opened a new issue #57: bug: when enter the router list ,the Action column out of views URL: https://github.com/apache/incubator-apisix-dashboard/issues/57 when enter the router list ,the Action column out of views,we always need to scroll to the right, which is inconvenient

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new pull request #62: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 opened a new pull request #62: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/62 1、when enter the router list ,the Action column fixed to the right and always inside view. 2、keep router upstream service list ordered

[GitHub] [incubator-apisix] lyyoung commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
lyyoung commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341842059 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +48,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix-dashboard] lilien1010 closed pull request #61: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 closed pull request #61: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/61 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-apisix] moonming commented on a change in pull request #794: Feature prometheus

2019-11-02 Thread GitBox
moonming commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341835727 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -21,6 +21,9 @@ local ngx_capture = ngx.location.capture

[GitHub] [incubator-apisix] moonming commented on a change in pull request #794: Feature prometheus

2019-11-02 Thread GitBox
moonming commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341835737 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +47,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] lyyoung commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
lyyoung commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341839612 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +47,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] membphis commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
membphis commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341840766 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +48,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] lyyoung commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
lyyoung commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341840808 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +47,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] membphis commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
membphis commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341840534 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +47,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] membphis commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
membphis commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341841144 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -44,26 +48,35 @@ function _M.init() -- per service

[GitHub] [incubator-apisix] membphis merged pull request #794: Feature prometheus

2019-11-03 Thread GitBox
membphis merged pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-apisix] moonming commented on issue #790: new plugin: data-validator

2019-11-02 Thread GitBox
moonming commented on issue #790: new plugin: data-validator URL: https://github.com/apache/incubator-apisix/issues/790#issuecomment-549107224 ``` succ_code = 200 ``` I think we don't need `succ_code` This is an

[GitHub] [incubator-apisix] moonming merged pull request #775: bugfix: support more built-in parameters when set chash balancer

2019-11-03 Thread GitBox
moonming merged pull request #775: bugfix: support more built-in parameters when set chash balancer URL: https://github.com/apache/incubator-apisix/pull/775 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-apisix] moonming closed issue #717: bug: Can't set other Nginx built-in parameters when config chash balancer

2019-11-03 Thread GitBox
moonming closed issue #717: bug: Can't set other Nginx built-in parameters when config chash balancer URL: https://github.com/apache/incubator-apisix/issues/717 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-apisix] lyyoung commented on a change in pull request #794: Feature prometheus

2019-11-03 Thread GitBox
lyyoung commented on a change in pull request #794: Feature prometheus URL: https://github.com/apache/incubator-apisix/pull/794#discussion_r341839491 ## File path: lua/apisix/plugins/prometheus/exporter.lua ## @@ -21,6 +21,9 @@ local ngx_capture = ngx.location.capture

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new issue #59: bug: not support multi hosts and multi uris when create a router

2019-11-03 Thread GitBox
lilien1010 opened a new issue #59: bug: not support multi hosts and multi uris when create a router URL: https://github.com/apache/incubator-apisix-dashboard/issues/59 This is an automated message from the Apache Git

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new issue #58: bug: each time I input a new upstream or router, the list is always in different order

2019-11-03 Thread GitBox
lilien1010 opened a new issue #58: bug: each time I input a new upstream or router, the list is always in different order URL: https://github.com/apache/incubator-apisix-dashboard/issues/58 each time I input a new upstream or router, the list is always in different order, which I

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new pull request #61: Dev for multi hosts uris

2019-11-03 Thread GitBox
lilien1010 opened a new pull request #61: Dev for multi hosts uris URL: https://github.com/apache/incubator-apisix-dashboard/pull/61 1、when enter the router list ,the Action column fixed to the right and always inside view. 2、keep router upstream service list ordered by ID, which I

[GitHub] [incubator-apisix-dashboard] lilien1010 closed pull request #60: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 closed pull request #60: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/60 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-apisix-dashboard] lilien1010 opened a new pull request #60: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 opened a new pull request #60: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/60 1、when enter the router list ,the Action column fixed to the right and always inside view. 2、keep router upstream service list ordered

[GitHub] [incubator-apisix] moonming commented on issue #790: new plugin: data-validator

2019-11-02 Thread GitBox
moonming commented on issue #790: new plugin: data-validator URL: https://github.com/apache/incubator-apisix/issues/790#issuecomment-549107199 Is it support json schema v4 or v5? This is an automated message from the Apache

[GitHub] [incubator-apisix-dashboard] juzhiyuan opened a new pull request #63: feat: remove useless files

2019-11-03 Thread GitBox
juzhiyuan opened a new pull request #63: feat: remove useless files URL: https://github.com/apache/incubator-apisix-dashboard/pull/63 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix-dashboard] juzhiyuan commented on issue #62: fix: add new features and function enhance

2019-11-03 Thread GitBox
juzhiyuan commented on issue #62: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/62#issuecomment-549154803 Thanks for your contribution, PR will be merged once I build & test the codes successfully today.

[GitHub] [incubator-apisix] soulsoul commented on issue #472: [WIP] doc: added comment for code.

2019-11-03 Thread GitBox
soulsoul commented on issue #472: [WIP] doc: added comment for code. URL: https://github.com/apache/incubator-apisix/pull/472#issuecomment-549199557 ok This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-apisix] membphis opened a new pull request #798: test: reformatted by `reindex` tools.

2019-11-03 Thread GitBox
membphis opened a new pull request #798: test: reformatted by `reindex` tools. URL: https://github.com/apache/incubator-apisix/pull/798 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-apisix] BradyBromley opened a new pull request #795: Changed wording for clarity in README.md

2019-11-03 Thread GitBox
BradyBromley opened a new pull request #795: Changed wording for clarity in README.md URL: https://github.com/apache/incubator-apisix/pull/795 NOTE: Please read the Contributing.md guidelines before submitting your patch:

[GitHub] [incubator-apisix-dashboard] lilien1010 edited a comment on issue #62: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 edited a comment on issue #62: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/62#issuecomment-549204748 https://github.com/apache/incubator-apisix/pull/797 this PR need to be merged after this PR is merged, since the

[GitHub] [incubator-apisix-dashboard] lilien1010 commented on issue #62: fix: add new features and function enhance

2019-11-03 Thread GitBox
lilien1010 commented on issue #62: fix: add new features and function enhance URL: https://github.com/apache/incubator-apisix-dashboard/pull/62#issuecomment-549204748 https://github.com/apache/incubator-apisix/pull/797 this PR need to be merged after this PR is merged, since the code in

[GitHub] [incubator-apisix] lilien1010 opened a new pull request #797: Fix:

2019-11-03 Thread GitBox
lilien1010 opened a new pull request #797: Fix: URL: https://github.com/apache/incubator-apisix/pull/797 Allow set multi uris in one route config, this PR work with my last PR in apisix-dashboard so I think yous need to merge this PR first then merge the one below

[GitHub] [incubator-apisix] membphis merged pull request #795: Changed wording for clarity in README.md

2019-11-03 Thread GitBox
membphis merged pull request #795: Changed wording for clarity in README.md URL: https://github.com/apache/incubator-apisix/pull/795 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-apisix] membphis commented on issue #795: Changed wording for clarity in README.md

2019-11-03 Thread GitBox
membphis commented on issue #795: Changed wording for clarity in README.md URL: https://github.com/apache/incubator-apisix/pull/795#issuecomment-549207183 Many thanks, merged. This is an automated message from the Apache Git

[GitHub] [incubator-apisix] membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config

2019-11-03 Thread GitBox
membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config URL: https://github.com/apache/incubator-apisix/pull/797#discussion_r341889993 ## File path: t/admin/routes.t ## @@ -1330,3 +1330,34 @@ GET /t {"error_msg":"failed to load

[GitHub] [incubator-apisix] lilien1010 opened a new issue #796: bug: set uris to

2019-11-03 Thread GitBox
lilien1010 opened a new issue #796: bug: set uris to URL: https://github.com/apache/incubator-apisix/issues/796 ### Issue description ### Environment * apisix version (cmd: `apisix version`): * OS: ### Minimal test code / Steps to reproduce the issue 1. 2.

[GitHub] [incubator-apisix] membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config

2019-11-03 Thread GitBox
membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config URL: https://github.com/apache/incubator-apisix/pull/797#discussion_r341889932 ## File path: lua/apisix/schema_def.lua ## @@ -267,6 +267,14 @@ _M.route = { type =

[GitHub] [incubator-apisix] membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config

2019-11-03 Thread GitBox
membphis commented on a change in pull request #797: Fix: Allow set multi uris in one route config URL: https://github.com/apache/incubator-apisix/pull/797#discussion_r341890108 ## File path: t/node/route-host.t ## @@ -42,7 +42,7 @@ __DATA__

[GitHub] [incubator-apisix] moonming commented on a change in pull request #831: Add doc for "How to compile the Openresty" and fix the doc about how to install apisix.

2019-11-10 Thread GitBox
moonming commented on a change in pull request #831: Add doc for "How to compile the Openresty" and fix the doc about how to install apisix. URL: https://github.com/apache/incubator-apisix/pull/831#discussion_r344573336 ## File path: doc/install-dependencies.md ## @@

[GitHub] [incubator-apisix] Miss-you opened a new issue #830: request help: Add document about how to compile the Openresty.

2019-11-10 Thread GitBox
Miss-you opened a new issue #830: request help: Add document about how to compile the Openresty. URL: https://github.com/apache/incubator-apisix/issues/830 ### Issue description There is no doc in "install-dependencies.md" about how to compile the Openresty ### Environment

[GitHub] [incubator-apisix] membphis commented on issue #830: request help: Add document about how to compile the Openresty.

2019-11-10 Thread GitBox
membphis commented on issue #830: request help: Add document about how to compile the Openresty. URL: https://github.com/apache/incubator-apisix/issues/830#issuecomment-552278988 http://openresty.org/en/linux-packages.html We recommend using the official package.

[GitHub] [incubator-apisix] Miss-you commented on issue #830: request help: Add document about how to compile the Openresty.

2019-11-10 Thread GitBox
Miss-you commented on issue #830: request help: Add document about how to compile the Openresty. URL: https://github.com/apache/incubator-apisix/issues/830#issuecomment-552283733 https://github.com/apache/incubator-apisix/pull/831

[GitHub] [incubator-apisix] Miss-you opened a new pull request #831: Add doc for "How to compile the Openresty" and fix the doc about how to install apisix.

2019-11-10 Thread GitBox
Miss-you opened a new pull request #831: Add doc for "How to compile the Openresty" and fix the doc about how to install apisix. URL: https://github.com/apache/incubator-apisix/pull/831 NOTE: Please read the Contributing.md guidelines before submitting your patch:

[GitHub] [incubator-apisix-dashboard] juzhiyuan commented on issue #70: fix:add regexp for uris and host filter when create route

2019-11-08 Thread GitBox
juzhiyuan commented on issue #70: fix:add regexp for uris and host filter when create route URL: https://github.com/apache/incubator-apisix-dashboard/pull/70#issuecomment-551916328 > @juzhiyuan how to show the error case clicking F12 to open Developer Tools if you use Chrome or

[GitHub] [incubator-apisix] membphis commented on a change in pull request #810: bugfix: wildcard certificates cannot match multi-level subdomains in …

2019-11-05 Thread GitBox
membphis commented on a change in pull request #810: bugfix: wildcard certificates cannot match multi-level subdomains in … URL: https://github.com/apache/incubator-apisix/pull/810#discussion_r342896149 ## File path: t/router/radixtree-sni.t ## @@ -426,3 +426,188 @@ lua

  1   2   3   4   5   6   7   8   9   10   >