[GitHub] [incubator-apisix] membphis commented on issue #1604: bug: stand alone mode proxy-rewrite headers has bug

2020-07-15 Thread GitBox


membphis commented on issue #1604:
URL: 
https://github.com/apache/incubator-apisix/issues/1604#issuecomment-659190881


   and we need to update this file: 
https://github.com/apache/incubator-apisix/blob/master/apisix/core/config_yaml.lua
   
   @nic-chen do you have time to fix this bug?



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] membphis commented on issue #1604: bug: stand alone mode proxy-rewrite headers has bug

2020-07-15 Thread GitBox


membphis commented on issue #1604:
URL: 
https://github.com/apache/incubator-apisix/issues/1604#issuecomment-659190647


   we need to check schema after we got the conf, here is an example:
   

   ```lua
   if data_valid and self.item_schema then
   data_valid, err = check_schema(self.item_schema, item.value)
   if not data_valid then
   log.error("failed to check item data of [", self.key,
   "] err:", err, " ,val: ", json.encode(item.value))
   end
   end
   ```
   
   Here is the code for config_etcd:
   
https://github.com/apache/incubator-apisix/blob/master/apisix/core/config_etcd.lua#L169



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you opened a new issue #1853: bug: The recommended value for worker_shutdown_timeout is 240 seconds.

2020-07-15 Thread GitBox


Miss-you opened a new issue #1853:
URL: https://github.com/apache/incubator-apisix/issues/1853


   ### Issue description
   
   current
   ```
 worker_shutdown_timeout: 3s # timeout for a graceful shutdown of 
worker processes
   ```
   https://github.com/apache/incubator-apisix/blob/master/conf/config.yaml#L104
   
   recommendation
   ```
 worker_shutdown_timeout: 240s # timeout for a graceful shutdown of 
worker processes
   ```
   
   reasons:
   1. 3 seconds is too short, once a graceful restart will quickly disconnect 
the long connection
   2. The default value for nginx ingress is 240 seconds.
   
   ### Environment
   
   * apisix version (cmd: `apisix version`):
   * OS:
   
   ### Minimal test code / Steps to reproduce the issue
   1.
   2.
   3.
   
   ### What's the actual result? (including assertion message & call stack if 
applicable)
   
   
   
   ### What's the expected result?
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] moonming commented on a change in pull request #1852: doc: update README.md for proxy-mirror

2020-07-15 Thread GitBox


moonming commented on a change in pull request #1852:
URL: https://github.com/apache/incubator-apisix/pull/1852#discussion_r455526398



##
File path: README.md
##
@@ -69,7 +69,8 @@ A/B testing, canary release, blue-green deployment, limit 
rate, defense against
 - Hash-based Load Balancing: Load balance with consistent hashing sessions.
 - [Health Checks](doc/health-check.md): Enable health check on the 
upstream node, and will automatically filter unhealthy nodes during load 
balancing to ensure system stability.
 - Circuit-Breaker: Intelligent tracking of unhealthy upstream services.
-
+- [Proxy Mirror](doc/plugins/proxy-mirror.md): Provides the ability to mirror 
client requests.

Review comment:
   need add sapces before `-`, keep line with above item.

##
File path: README_CN.md
##
@@ -68,7 +68,8 @@ A/B 测试、金丝雀发布(灰度发布)、蓝绿部署、限流限速、抵
 - 支持一致性 hash 的负载均衡:动态支持一致性 hash 的负载均衡。
 - [健康检查](doc/health-check.md):启用上游节点的健康检查,将在负载均衡期间自动过滤不健康的节点,以确保系统稳定性。
 - 熔断器: 智能跟踪不健康上游服务。
-
+- [代理镜像](doc/zh-cn/plugins/proxy-mirror.md): 提供镜像客户端请求的能力。

Review comment:
   ditto





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] UnderTreeTech commented on issue #1804: performance: spend more time on ssl handshake

2020-07-15 Thread GitBox


UnderTreeTech commented on issue #1804:
URL: 
https://github.com/apache/incubator-apisix/issues/1804#issuecomment-659175368


   Any updates? @membphis @moonming 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] scaat opened a new pull request #1852: doc: update README.md for proxy-mirror

2020-07-15 Thread GitBox


scaat opened a new pull request #1852:
URL: https://github.com/apache/incubator-apisix/pull/1852


   ### What this PR does / why we need it:
   
   
   Fix #1847
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Beckjiang opened a new issue #1850: request help: How to grayscale update the api configuration

2020-07-15 Thread GitBox


Beckjiang opened a new issue #1850:
URL: https://github.com/apache/incubator-apisix/issues/1850


   ### Issue description
   Does it now support grayscale update API configuration?
   
   If it's not supported, what are the ideas to implement it?
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] wanzhenhan opened a new issue #1851: health check can't use

2020-07-15 Thread GitBox


wanzhenhan opened a new issue #1851:
URL: https://github.com/apache/incubator-apisix/issues/1851


   环境:
   version: v1.4.0
   路由和upstream配置:
   curl -i --connect-timeout 1 http://127.0.0.1:9080/apisix/admin/routes/ 
-H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '
   {
   "methods": ["POST", "GET","DELETE"],
   "uri": "//v1/readiness",
   "plugins": {
   },
   "upstream": {
   "type": "roundrobin",
   "nodes": {
   "10.82.4.112:10188": 1,
   "10.82.4.112:10190": 1
   },
   "checks": {
   "active": {
   "http_path": "//v1/liveness",
   "host": "health.com",
   "healthy": {
   "interval": 4,
   "successes": 1
   },
   "unhealthy": {
   "interval": 5,
   "http_failures": 1
   }
   }
   }
   }
   }'
   当客户端发起访问时,报如下错误:
   failed to run balancer_by_lua*: 
/usr/local/openresty/lualib/resty/lock.lua:153: API disabled in the context of 
balancer_by_lua*



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-659119097


   Hi @membphis, 
   Do we think this is PR good to move forward :D? 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] ko-han commented on a change in pull request #1795: feature: expose API for dns resolving

2020-07-15 Thread GitBox


ko-han commented on a change in pull request #1795:
URL: https://github.com/apache/incubator-apisix/pull/1795#discussion_r455467851



##
File path: apisix/init.lua
##
@@ -630,5 +630,18 @@ function _M.stream_log_phase()
 run_plugin("log")
 end
 
+do
+local dns_resolver_cache = core.lrucache.new({
+ttl = 30, count = 512, invalid_stale = true,
+})
+
+function _M.dns_resolve(hostname)
+if ipmatcher.parse_ipv4(hostname) or
+ipmatcher.parse_ipv6(hostname) then
+return hostname
+end

Review comment:
   It's added.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] RocFang opened a new pull request #1849: fix hyper links in readme

2020-07-15 Thread GitBox


RocFang opened a new pull request #1849:
URL: https://github.com/apache/incubator-apisix/pull/1849


   ### What this PR does / why we need it:
   
   
   Found some link errors when start reading the docs.
   
   ### Pre-submission checklist:
   
   * [ x ] Did you explain what problem does this PR solve? Or what new 
features have been added?
   * [ ] Have you added corresponding test cases?
   * [ ] Have you modified the corresponding document?
   * [ x ] Is this PR backward compatible?
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-658897291


   According to this 
[doc](https://github.com/travis-ci/travis-ci/issues/576#issuecomment-65317915), 
we can close and reopen, which will trigger a new CI run without making 
changes. 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d closed pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d closed pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-658895818


   The CI fails due to CI server network issue: 
https://github.com/apache/incubator-apisix/pull/1845/checks?check_run_id=874200614#step:3:68
   
   Will make a small change and trigger another CI run.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-658885499


   > @moonming @membphis , is it possible to force trigger a CI?
   > I made new commit to my local repo mtx2d/incubato-apisix, but the CI still 
looks at the old states.(my master should be 11 commits ahead but the CI still 
believes I am only 5 commits ahead).
   > 
   > How can I force trigger a CI?
   
   Seems like the CI picks it up now.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-658874317


   @moonming @membphis , is it possible to force trigger a CI? 
   I made new commit to my local repo mtx2d/incubato-apisix, but the CI still 
looks at the old states.(my master should be 11 commits ahead but the CI still 
believes I am only 5 commits ahead). 
   
   
   How can I force trigger a CI? 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#issuecomment-658863570


   Trying to figure out how to trigger CI.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on a change in pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


mtx2d commented on a change in pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#discussion_r455175520



##
File path: t/lib/server.lua
##
@@ -112,10 +112,6 @@ function _M.mock_skywalking_v2_instance_register()
 ngx.say(json_encode(ret))
 end
 
-function _M.mock_skywalking_v2_instance_heartbeat()
-ngx.say('skywalking heartbeat ok')
-end
-

Review comment:
   reverted in latest commit. 

##
File path: Makefile
##
@@ -176,7 +176,7 @@ install: default
 
 ### test: Run the test case
 test:
-   prove -I../test-nginx/lib -I./ -r -s t/
+   TEST_NGINX_BINARY=`which openresty` prove -I../test-nginx/lib 
-I../test-nginx/inc -I./ -r -s t/

Review comment:
   Reverted in new commit.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you edited a comment on issue #1842: request help: log phase occurs nil pointer error log.

2020-07-15 Thread GitBox


Miss-you edited a comment on issue #1842:
URL: 
https://github.com/apache/incubator-apisix/issues/1842#issuecomment-658845583


   If you are using luarocks to install, you can try `luarocks install apisix` 
to upgrade the APISIX



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you commented on issue #1842: request help: log phase occurs nil pointer error log.

2020-07-15 Thread GitBox


Miss-you commented on issue #1842:
URL: 
https://github.com/apache/incubator-apisix/issues/1842#issuecomment-658845583


   If you are using luarocks to install, you can uninstall and reinstall.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you commented on issue #1842: request help: log phase occurs nil pointer error log.

2020-07-15 Thread GitBox


Miss-you commented on issue #1842:
URL: 
https://github.com/apache/incubator-apisix/issues/1842#issuecomment-658844948


   I think you should upgrade the apache apisix .



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you commented on issue #1846: bug: Missing the CODE_STYLE.md file

2020-07-15 Thread GitBox


Miss-you commented on issue #1846:
URL: 
https://github.com/apache/incubator-apisix/issues/1846#issuecomment-658844218


   https://github.com/apache/incubator-apisix/pull/1848



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you edited a comment on issue #1846: bug: Missing the CODE_STYLE.md file

2020-07-15 Thread GitBox


Miss-you edited a comment on issue #1846:
URL: 
https://github.com/apache/incubator-apisix/issues/1846#issuecomment-658843329


   I think it would be more appropriate to use this document as a code style 
guide.
   https://github.com/luarocks/lua-style-guide
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you commented on issue #1846: bug: Missing the CODE_STYLE.md file

2020-07-15 Thread GitBox


Miss-you commented on issue #1846:
URL: 
https://github.com/apache/incubator-apisix/issues/1846#issuecomment-658843329


   I think it would be more appropriate to use this document as a code style 
guide.
   https://github.com/luarocks/lua-style-guide



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] Miss-you opened a new pull request #1848: [doc] remove the CODE_STYLE.md ref

2020-07-15 Thread GitBox


Miss-you opened a new pull request #1848:
URL: https://github.com/apache/incubator-apisix/pull/1848


   ### What this PR does / why we need it:
   
   
   
   Remove the CODE_STYLE.md ref which would cause invalid reference.
   
   ### Pre-submission checklist:
   
   * [ ] Did you explain what problem does this PR solve? Or what new features 
have been added?
   * [ ] Have you added corresponding test cases?
   * [ ] Have you modified the corresponding document?
   * [ ] Is this PR backward compatible?
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[incubator-apisix-website] branch master updated: feat(graduate): remove incubating text (#52)

2020-07-15 Thread juzhiyuan
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-website.git


The following commit(s) were added to refs/heads/master by this push:
 new 61b2372  feat(graduate): remove incubating text (#52)
61b2372 is described below

commit 61b237271a3eb90c2933086db97d782078eb62be
Author: 琚致远 
AuthorDate: Wed Jul 15 21:11:20 2020 +0800

feat(graduate): remove incubating text (#52)
---
 README.md  |  4 ++--
 config.yaml| 22 --
 content/contribute/committer/index.en.md   |  2 +-
 content/contribute/committer/index.zh.md   |  2 +-
 content/contribute/contributor/index.en.md |  2 +-
 content/contribute/contributor/index.zh.md |  2 +-
 content/contribute/security/index.en.md|  4 ++--
 content/contribute/security/index.zh.md|  4 ++--
 content/contribute/subscribe/index.en.md   |  4 ++--
 content/contribute/subscribe/index.zh.md   |  4 ++--
 content/downloads/index.en.md  | 12 +---
 content/downloads/index.zh.md  | 12 +---
 content/team/index.en.md   |  8 
 content/team/index.zh.md   |  6 +++---
 layouts/partials/footer.html   |  4 ++--
 15 files changed, 33 insertions(+), 59 deletions(-)

diff --git a/README.md b/README.md
index 28c277d..31f9715 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
-# Apache APISIX (incubating)
+# Apache APISIX
 
-> The website of [Apache APISIX 
(incubating)](https://github.com/apache/incubator-apisix), a cloud-native 
microservices api gateway.
+> The website of [Apache APISIX](https://github.com/apache/incubator-apisix), 
a cloud-native microservices api gateway.
 
 ## Development
 ```sh
diff --git a/config.yaml b/config.yaml
index 9b913a6..2a641e3 100644
--- a/config.yaml
+++ b/config.yaml
@@ -1,5 +1,5 @@
-baseURL: https://apisix.incubator.apache.org/
-title: Apache APISIX (incubating) 
+baseURL: https://apisix.apache.org/
+title: Apache APISIX
 theme: hugo-fresh
 googleAnalytics: #Put in your tracking code without quotes like this: 
UA-XX...
 #Disables warningss
@@ -13,10 +13,10 @@ languages:
 params:
   hero:
 # Main hero title
-title: Apache APISIX (incubating)
+title: Apache APISIX
 # Hero subtitle (optional)
 subtitle1: Cloud-native microservices API gateway, delivering the 
ultimate performance, security, open source and scalable platform for all your 
APIs and microservices.
-subtitle2: Apache APISIX (incubating) is based on Nginx and etcd. 
Compared with traditional API gateways, APISIX has dynamic routing and plug-in 
hot loading, which is especially suitable for API management under 
micro-service system.
+subtitle2: Apache APISIX is based on Nginx and etcd. Compared with 
traditional API gateways, APISIX has dynamic routing and plug-in hot loading, 
which is especially suitable for API management under micro-service system.
 subtitle3: Apache releases are beginning from version 0.9. 
 # Button text
 buttontext: View on GitHub
@@ -77,7 +77,7 @@ languages:
 name: 中
 link: /zh
   section1:
-title: Why Apache APISIX (incubating)?
+title: Why Apache APISIX?
 tiles:
 - title: High availability, safety first
   icon: mouse-globe
@@ -94,9 +94,6 @@ languages:
   section5: true
   footer:
 quicklinks:
-  column1:
-title: "Disclaimer"
-text: Apache APISIX (incubating) is an effort undergoing 
incubation at The Apache Software Foundation (ASF), sponsored by Incubator. 
Incubation is required of all newly accepted projects until a further review 
indicates that the infrastructure, communications, and decision making process 
have stabilized in a manner consistent with other successful ASF projects. 
While incubation status is not necessarily a reflection of the completeness or 
stability of the code, it does indica [...]
   column2:
 title: "Documentation"
 links:
@@ -110,7 +107,7 @@ languages:
 params:
 hero:
   # Main hero title
-  title: Apache APISIX (incubating)
+  title: Apache APISIX
   # Hero subtitle (optional)
   subtitle1: 云原生、高性能、可扩展的微服务 API 网关
   subtitle2: 基于 Nginx 和 etcd 来实现,具备动态路由、插件热加载和 gRPC 代理和协议转换功能,更适合微服务体系下的 
API 管理
@@ -174,7 +171,7 @@ languages:
 name: EN
 link: /
 section1:
-  title: 为什么选择 Apache APISIX (incubating)?
+  title: 为什么选择 Apache APISIX?
   tiles:
   - title: 高可用,安全第一
 icon: mouse-globe
@@ -189,14 +186,11 @@ languages:
 icon: doc-sync
 text: 支持 http 到 gRPC 的协议转换,支持 websocket、gRPC、Dubbo、MQTT 的代理,支持包括 ARM64 
在内的多种平台
 section3:
-  title: 谁在使用 Apache APISIX (incubating)?
+  title: 谁在使用 Apache APISIX?
   image: 

[GitHub] [incubator-apisix-website] juzhiyuan merged pull request #52: feat(graduate): remove incubating text

2020-07-15 Thread GitBox


juzhiyuan merged pull request #52:
URL: https://github.com/apache/incubator-apisix-website/pull/52


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix-website] juzhiyuan opened a new pull request #52: feat(graduate): remove incubating text

2020-07-15 Thread GitBox


juzhiyuan opened a new pull request #52:
URL: https://github.com/apache/incubator-apisix-website/pull/52


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[incubator-apisix-website] branch asf-site updated: Deploy to GitHub pages

2020-07-15 Thread github-bot
This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/incubator-apisix-website.git


The following commit(s) were added to refs/heads/asf-site by this push:
 new 87a6842  Deploy to GitHub pages
87a6842 is described below

commit 87a6842e525f6589c4b2933e00da88eedf8d4656
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Wed Jul 15 13:11:57 2020 +

Deploy to GitHub pages
---
 README.md|  4 +-
 categories/index.xml |  8 ++--
 contribute/2fa/index.html| 20 ++
 contribute/committer/index.html  | 22 +++
 contribute/contributor/index.html| 22 +++
 contribute/index.xml | 38 +--
 contribute/release/index.html| 20 ++
 contribute/security/index.html   | 28 --
 contribute/subscribe/index.html  | 26 -
 downloads/index.html | 34 +++--
 en/index.html|  2 +-
 en/sitemap.xml   | 72 ++--
 index.html   | 28 --
 index.xml| 48 
 sitemap.xml  |  4 +-
 tags/index.xml   |  8 ++--
 team/index.html  | 28 --
 zh/categories/index.xml  |  8 ++--
 zh/contribute/2fa/index.html | 27 +++---
 zh/contribute/committer/index.html   | 31 
 zh/contribute/contributor/index.html | 29 +++
 zh/contribute/index.xml  | 42 ++---
 zh/contribute/release/index.html | 27 +++---
 zh/contribute/security/index.html| 35 +-
 zh/contribute/subscribe/index.html   | 35 +-
 zh/downloads/index.html  | 41 +++-
 zh/index.html| 35 +-
 zh/index.xml | 52 +-
 zh/sitemap.xml   | 72 ++--
 zh/tags/index.xml|  8 ++--
 zh/team/index.html   | 33 -
 31 files changed, 447 insertions(+), 440 deletions(-)

diff --git a/README.md b/README.md
index 28c277d..31f9715 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
-# Apache APISIX (incubating)
+# Apache APISIX
 
-> The website of [Apache APISIX 
(incubating)](https://github.com/apache/incubator-apisix), a cloud-native 
microservices api gateway.
+> The website of [Apache APISIX](https://github.com/apache/incubator-apisix), 
a cloud-native microservices api gateway.
 
 ## Development
 ```sh
diff --git a/categories/index.xml b/categories/index.xml
index f1dfdf1..683311c 100644
--- a/categories/index.xml
+++ b/categories/index.xml
@@ -1,12 +1,12 @@
 
 http://www.w3.org/2005/Atom";>
   
-Categories on Apache APISIX (incubating)
-https://apisix.incubator.apache.org/categories/
-Recent content in Categories on Apache APISIX 
(incubating)
+Categories on Apache APISIX
+https://apisix.apache.org/categories/
+Recent content in Categories on Apache APISIX
 Hugo -- gohugo.io
 
-   https://apisix.incubator.apache.org/categories/index.xml"; rel="self" 
type="application/rss+xml" />
+   https://apisix.apache.org/categories/index.xml"; 
rel="self" type="application/rss+xml" />
 
 
   
diff --git a/contribute/2fa/index.html b/contribute/2fa/index.html
index ec1f4ef..a6faa02 100644
--- a/contribute/2fa/index.html
+++ b/contribute/2fa/index.html
@@ -7,7 +7,7 @@
 
 
-https://apisix.incubator.apache.org/contribute/2fa/"; />
+https://apisix.apache.org/contribute/2fa/"; />
 
 
 
@@ -16,7 +16,7 @@ Note:If you do not enable 2FA, you will be removed from the 
project and unable t
 
 
 
-Apache APISIX (incubating)
+Apache APISIX
 
 
 https://fonts.googleapis.com/css?family=Open+Sans:400,600"; 
rel="stylesheet">
@@ -253,18 +253,6 @@ Note:If you do not enable 2FA, you will be removed from 
the project and unable t
 
   
 
-Disclaimer
-
-
-  Apache APISIX (incubating) is an effort undergoing incubation at 
The Apache Software Foundation (ASF), sponsored by Incubator. Incubation is 
required of all newly accepted projects until a further review indicates that 
the infrastructure, communications, and decision making process have stabilized 
in a manner consistent with other successful ASF projects. While incubation 
status is not necessarily a reflection of the completeness or stability of the 
code, it does indicate t [...]
-
-
-
-  
-
-
-  
-
 Documentation
 
 
@@

[incubator-apisix-website] 01/01: feat(graduate): remove incubating text

2020-07-15 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository.

juzhiyuan pushed a commit to branch feat-graduate
in repository https://gitbox.apache.org/repos/asf/incubator-apisix-website.git

commit 20a877024ba14e4fdec061146b2d4e9fda5a8157
Author: juzhiyuan 
AuthorDate: Wed Jul 15 21:08:01 2020 +0800

feat(graduate): remove incubating text
---
 README.md  |  4 ++--
 config.yaml| 22 --
 content/contribute/committer/index.en.md   |  2 +-
 content/contribute/committer/index.zh.md   |  2 +-
 content/contribute/contributor/index.en.md |  2 +-
 content/contribute/contributor/index.zh.md |  2 +-
 content/contribute/security/index.en.md|  4 ++--
 content/contribute/security/index.zh.md|  4 ++--
 content/contribute/subscribe/index.en.md   |  4 ++--
 content/contribute/subscribe/index.zh.md   |  4 ++--
 content/downloads/index.en.md  | 12 +---
 content/downloads/index.zh.md  | 12 +---
 content/team/index.en.md   |  8 
 content/team/index.zh.md   |  6 +++---
 layouts/partials/footer.html   |  4 ++--
 15 files changed, 33 insertions(+), 59 deletions(-)

diff --git a/README.md b/README.md
index 28c277d..31f9715 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
-# Apache APISIX (incubating)
+# Apache APISIX
 
-> The website of [Apache APISIX 
(incubating)](https://github.com/apache/incubator-apisix), a cloud-native 
microservices api gateway.
+> The website of [Apache APISIX](https://github.com/apache/incubator-apisix), 
a cloud-native microservices api gateway.
 
 ## Development
 ```sh
diff --git a/config.yaml b/config.yaml
index 9b913a6..2a641e3 100644
--- a/config.yaml
+++ b/config.yaml
@@ -1,5 +1,5 @@
-baseURL: https://apisix.incubator.apache.org/
-title: Apache APISIX (incubating) 
+baseURL: https://apisix.apache.org/
+title: Apache APISIX
 theme: hugo-fresh
 googleAnalytics: #Put in your tracking code without quotes like this: 
UA-XX...
 #Disables warningss
@@ -13,10 +13,10 @@ languages:
 params:
   hero:
 # Main hero title
-title: Apache APISIX (incubating)
+title: Apache APISIX
 # Hero subtitle (optional)
 subtitle1: Cloud-native microservices API gateway, delivering the 
ultimate performance, security, open source and scalable platform for all your 
APIs and microservices.
-subtitle2: Apache APISIX (incubating) is based on Nginx and etcd. 
Compared with traditional API gateways, APISIX has dynamic routing and plug-in 
hot loading, which is especially suitable for API management under 
micro-service system.
+subtitle2: Apache APISIX is based on Nginx and etcd. Compared with 
traditional API gateways, APISIX has dynamic routing and plug-in hot loading, 
which is especially suitable for API management under micro-service system.
 subtitle3: Apache releases are beginning from version 0.9. 
 # Button text
 buttontext: View on GitHub
@@ -77,7 +77,7 @@ languages:
 name: 中
 link: /zh
   section1:
-title: Why Apache APISIX (incubating)?
+title: Why Apache APISIX?
 tiles:
 - title: High availability, safety first
   icon: mouse-globe
@@ -94,9 +94,6 @@ languages:
   section5: true
   footer:
 quicklinks:
-  column1:
-title: "Disclaimer"
-text: Apache APISIX (incubating) is an effort undergoing 
incubation at The Apache Software Foundation (ASF), sponsored by Incubator. 
Incubation is required of all newly accepted projects until a further review 
indicates that the infrastructure, communications, and decision making process 
have stabilized in a manner consistent with other successful ASF projects. 
While incubation status is not necessarily a reflection of the completeness or 
stability of the code, it does indica [...]
   column2:
 title: "Documentation"
 links:
@@ -110,7 +107,7 @@ languages:
 params:
 hero:
   # Main hero title
-  title: Apache APISIX (incubating)
+  title: Apache APISIX
   # Hero subtitle (optional)
   subtitle1: 云原生、高性能、可扩展的微服务 API 网关
   subtitle2: 基于 Nginx 和 etcd 来实现,具备动态路由、插件热加载和 gRPC 代理和协议转换功能,更适合微服务体系下的 
API 管理
@@ -174,7 +171,7 @@ languages:
 name: EN
 link: /
 section1:
-  title: 为什么选择 Apache APISIX (incubating)?
+  title: 为什么选择 Apache APISIX?
   tiles:
   - title: 高可用,安全第一
 icon: mouse-globe
@@ -189,14 +186,11 @@ languages:
 icon: doc-sync
 text: 支持 http 到 gRPC 的协议转换,支持 websocket、gRPC、Dubbo、MQTT 的代理,支持包括 ARM64 
在内的多种平台
 section3:
-  title: 谁在使用 Apache APISIX (incubating)?
+  title: 谁在使用 Apache APISIX?
   image: 
https://raw.githubusercontent.com/iresty/iresty.com/master/user-wall.jpg
 section5: true
 footer:
   quicklinks:
-column1:
-  t

[incubator-apisix-website] branch feat-graduate created (now 20a8770)

2020-07-15 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository.

juzhiyuan pushed a change to branch feat-graduate
in repository https://gitbox.apache.org/repos/asf/incubator-apisix-website.git.


  at 20a8770  feat(graduate): remove incubating text

This branch includes the following new commits:

 new 20a8770  feat(graduate): remove incubating text

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.




[GitHub] [incubator-apisix-website] juzhiyuan opened a new issue #51: Refactor Website by React or Vue.js

2020-07-15 Thread GitBox


juzhiyuan opened a new issue #51:
URL: https://github.com/apache/incubator-apisix-website/issues/51


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] membphis commented on a change in pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


membphis commented on a change in pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#discussion_r454943520



##
File path: t/lib/server.lua
##
@@ -112,10 +112,6 @@ function _M.mock_skywalking_v2_instance_register()
 ngx.say(json_encode(ret))
 end
 
-function _M.mock_skywalking_v2_instance_heartbeat()
-ngx.say('skywalking heartbeat ok')
-end
-

Review comment:
   Need to revert these codes, then the test case will run successfully.
   
   ```
   Test Summary Report
   ---
   t/plugin/skywalking.t(Wstat: 256 Tests: 30 Failed: 1)
 Failed test:  10
 Non-zero exit status: 1
   Files=120, Tests=4005, 1041 wallclock secs ( 1.02 usr  0.16 sys + 152.60 
cusr 29.88 csys = 183.66 CPU)
   Result: FAIL
   ##[error]Process completed with exit code 1.
   ```





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[incubator-apisix] branch master updated: doc: add more info for pr template. (#1840)

2020-07-15 Thread wenming
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 9a6a86c  doc: add more info for pr template. (#1840)
9a6a86c is described below

commit 9a6a86ce4f44b05e7437818c563b8f012ea0c247
Author: Wen Ming 
AuthorDate: Wed Jul 15 17:32:39 2020 +0800

doc: add more info for pr template. (#1840)
---
 .github/PULL_REQUEST_TEMPLATE.md | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index 0209e72..c5cd79d 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,6 +1,8 @@
-### Pre-submission checklist:
+### What this PR does / why we need it:
+
+
 
-*Please answer each of these after submitting your pull request:*
+### Pre-submission checklist:
 
 * [ ] Did you explain what problem does this PR solve? Or what new features 
have been added?
 * [ ] Have you added corresponding test cases?



[GitHub] [incubator-apisix] moonming merged pull request #1840: doc: add more info for pr template.

2020-07-15 Thread GitBox


moonming merged pull request #1840:
URL: https://github.com/apache/incubator-apisix/pull/1840


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] moonming commented on a change in pull request #1845: remove not used plugin heartbeat.lua

2020-07-15 Thread GitBox


moonming commented on a change in pull request #1845:
URL: https://github.com/apache/incubator-apisix/pull/1845#discussion_r454917841



##
File path: Makefile
##
@@ -176,7 +176,7 @@ install: default
 
 ### test: Run the test case
 test:
-   prove -I../test-nginx/lib -I./ -r -s t/
+   TEST_NGINX_BINARY=`which openresty` prove -I../test-nginx/lib 
-I../test-nginx/inc -I./ -r -s t/

Review comment:
   why change this line? which is non-related with heartbeat plugin.

##
File path: t/lib/server.lua
##
@@ -112,10 +112,6 @@ function _M.mock_skywalking_v2_instance_register()
 ngx.say(json_encode(ret))
 end
 
-function _M.mock_skywalking_v2_instance_heartbeat()
-ngx.say('skywalking heartbeat ok')
-end
-

Review comment:
   these codes should not remove.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] moonming commented on a change in pull request #1795: feature: expose API for dns resolving

2020-07-15 Thread GitBox


moonming commented on a change in pull request #1795:
URL: https://github.com/apache/incubator-apisix/pull/1795#discussion_r454892966



##
File path: apisix/init.lua
##
@@ -630,5 +630,18 @@ function _M.stream_log_phase()
 run_plugin("log")
 end
 
+do
+local dns_resolver_cache = core.lrucache.new({
+ttl = 30, count = 512, invalid_stale = true,
+})
+
+function _M.dns_resolve(hostname)
+if ipmatcher.parse_ipv4(hostname) or
+ipmatcher.parse_ipv6(hostname) then
+return hostname
+end

Review comment:
   need test cases for those codes.





This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] nanwenbo commented on issue #1831: request help: Mac os can't start apisix

2020-07-15 Thread GitBox


nanwenbo commented on issue #1831:
URL: 
https://github.com/apache/incubator-apisix/issues/1831#issuecomment-658608258


   > BTW, is APISIX still can wroks well? I think warn level will not block 
start.
   
   apisix  is not work, i reinstall openresty 1.15.8.3  ,apisix is working now.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d closed issue #1844: request help: Local Test for Master Barnch Failed 26% Cases

2020-07-15 Thread GitBox


mtx2d closed issue #1844:
URL: https://github.com/apache/incubator-apisix/issues/1844


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [incubator-apisix] mtx2d commented on issue #1844: request help: Local Test for Master Barnch Failed 26% Cases

2020-07-15 Thread GitBox


mtx2d commented on issue #1844:
URL: 
https://github.com/apache/incubator-apisix/issues/1844#issuecomment-658596137


   I will close this. The recommended way to test is to file a PR and trigger 
CI to run full set of tests.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org