[GitHub] [incubator-apisix] asher86v opened a new issue #975: request help: 开启health健康检查,不能自动剔除有问题的节点

2019-12-15 Thread GitBox
asher86v opened a new issue #975: request help:  开启health健康检查,不能自动剔除有问题的节点
URL: https://github.com/apache/incubator-apisix/issues/975
 
 
   ### Issue description
   开启health健康检查,不能自动剔除有问题的节点
   
   ### Environment
   * apisix version (cmd: `apisix version`): 0.9-rc1
   * OS:  CentOS Linux release 7.4.1708 (Core)
   
   
   curl -i -X PUT http://10.110.2.120:9080/apisix/admin/routes/1112 -d '
   {
   "uri": "/*",
   "hosts": ["www.dd.com"],
   "methods": ["PUT", "GET", "POST", "DELETE"],
   "upstream": {
   "type": "roundrobin",
   "nodes": {
   "10.110.2.95:5000": 1,
   "10.110.2.95:5001": 1
   },
   "checks": {
   "passive": {
   "healthy": {
   "http_statuses": [200, 201, 204, 301],
   "successes": 1
   },
   "unhealthy": {
   "http_statuses": [500, 504,502, 403],
   "http_failures": 1,
   "tcp_failures": 1
   }
   }
   }
   }
   }'
   
   这时候关掉其中一个节点得服务,会出现502错误。
   
   ➜  ~ curl www.dd.com:9080
   5000%

   ➜  ~ curl www.dd.com:9080
   5001%

   ➜  ~ curl www.dd.com:9080
   5000%

   ➜  ~ curl www.dd.com:9080
   5001%

   ➜  ~ curl www.dd.com:9080
   
   502 Bad Gateway
   
   502 Bad Gateway
   openresty
   
   
   
   
   
   


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix-dashboard] jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入

2019-12-15 Thread GitBox
jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入
URL: 
https://github.com/apache/incubator-apisix-dashboard/issues/88#issuecomment-565890142
 
 
   
![image](https://user-images.githubusercontent.com/2897/70877024-bf4afd00-1ff6-11ea-983d-362e0c284472.png)
   测试了,hosts要输入域名才可以,ip则不行


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix-dashboard] jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入

2019-12-15 Thread GitBox
jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入
URL: 
https://github.com/apache/incubator-apisix-dashboard/issues/88#issuecomment-565889751
 
 
   
![image](https://user-images.githubusercontent.com/2897/70876903-67ac9180-1ff6-11ea-9f06-5aff67274891.png)
   urls可以通过录入回车(选中)的方式录入值,hosts则录入失败,会被清空


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix-dashboard] jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入

2019-12-15 Thread GitBox
jiezaizone commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入
URL: 
https://github.com/apache/incubator-apisix-dashboard/issues/88#issuecomment-565888958
 
 
   > You can input any value directly or select any existing options
   
   URLs可以这样录入,但Hosts录入失败,录入值选中后自动情况


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] moonming commented on issue #970: Update serverless.md

2019-12-15 Thread GitBox
moonming commented on issue #970: Update serverless.md
URL: https://github.com/apache/incubator-apisix/pull/970#issuecomment-565876959
 
 
   @chspy any update?


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] moonming merged pull request #974: Added the conf/nginx.conf to .gitignore

2019-12-15 Thread GitBox
moonming merged pull request #974: Added the conf/nginx.conf to .gitignore
URL: https://github.com/apache/incubator-apisix/pull/974
 
 
   


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] moonming commented on issue #974: Added the conf/nginx.conf to .gitignore

2019-12-15 Thread GitBox
moonming commented on issue #974: Added the conf/nginx.conf to .gitignore
URL: https://github.com/apache/incubator-apisix/pull/974#issuecomment-565862389
 
 
   nice work, thx


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


With regards,
Apache Git Services


[incubator-apisix] branch master updated: Added the conf/nginx.conf to .gitignore (#974)

2019-12-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 af1a29a  Added the conf/nginx.conf to .gitignore (#974)
af1a29a is described below

commit af1a29a38e6c7623fa697ea524832ece76a9cbf6
Author: Kirill 
AuthorDate: Mon Dec 16 03:12:53 2019 +0300

Added the conf/nginx.conf to .gitignore (#974)
---
 .gitignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitignore b/.gitignore
index 60d5294..f65e311 100644
--- a/.gitignore
+++ b/.gitignore
@@ -46,6 +46,7 @@ t/servroot
 go
 \.*
 conf/apisix.uid
+conf/nginx.conf
 deps
 scgi_temp
 uwsgi_temp



[GitHub] [incubator-apisix] moonming commented on issue #949: request help: How to replace uri?

2019-12-15 Thread GitBox
moonming commented on issue #949: request help: How to replace uri?
URL: 
https://github.com/apache/incubator-apisix/issues/949#issuecomment-565862321
 
 
   sure, welcome PR :) thx


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] mausspb opened a new pull request #974: Added the conf/nginx.conf to .gitignore

2019-12-15 Thread GitBox
mausspb opened a new pull request #974: Added the conf/nginx.conf to .gitignore
URL: https://github.com/apache/incubator-apisix/pull/974
 
 
   ### Summary
   
   Added `conf/nginx.conf` to `.gitignore`
   
   ### Issues resolved
   
   Resolves #968 
   


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] agile6v commented on issue #949: request help: How to replace uri?

2019-12-15 Thread GitBox
agile6v commented on issue #949: request help: How to replace uri?
URL: 
https://github.com/apache/incubator-apisix/issues/949#issuecomment-565814226
 
 
   @moonming I agree with your suggestion. Is it better to change sub_uri to 
regex_uri? If possible, I will submit a PR later.


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] moonming commented on issue #973: Add protocol header for upstream service

2019-12-15 Thread GitBox
moonming commented on issue #973: Add protocol header for upstream service
URL: 
https://github.com/apache/incubator-apisix/issues/973#issuecomment-565813358
 
 
   you can send email to mailing list to discuss with more developer.


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] moonming commented on issue #973: Add protocol header for upstream service

2019-12-15 Thread GitBox
moonming commented on issue #973: Add protocol header for upstream service
URL: 
https://github.com/apache/incubator-apisix/issues/973#issuecomment-565813149
 
 
   I got your point.
   I think we can add this to `/bin/apisix` directly.


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] chnliyong commented on issue #973: Add protocol header for upstream service

2019-12-15 Thread GitBox
chnliyong commented on issue #973: Add protocol header for upstream service
URL: 
https://github.com/apache/incubator-apisix/issues/973#issuecomment-565812530
 
 
   > I think you can use the proxy-rewrite plugin.
   
   I thought this is a very commonly requirement if `apisix` is used as a 
`reverse proxy` to offer ssl offloading, and configure the `proxy-rewrite`  for 
all routes is tedious.
   We may add a option such as `enable_protocol_forwarded` and 
`protocol_forwarded_header` in `config.yaml` if we don't  want this as default. 
 Actually, I would like this be default.
   
   ```nginx
 proxy_set_header 
 proxy_set_header X-Forwarded-Proto $scheme;
 proxy_set_header 
   ```


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] chnliyong opened a new issue #973: Add protocol header for upstream service

2019-12-15 Thread GitBox
chnliyong opened a new issue #973: Add protocol header for upstream service
URL: https://github.com/apache/incubator-apisix/issues/973
 
 
   The upstream service such as web rendering may sometimes need to know the 
incoming request's scheme(`http` or `https`) from client. Conventionally use 
the http header 
`X-Forwarded-Proto`[X-Forwarded-Proto](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Proto)


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix] chnliyong commented on issue #914: Make apisix's production deployment **FHS**able

2019-12-15 Thread GitBox
chnliyong commented on issue #914: Make apisix's production deployment 
**FHS**able
URL: 
https://github.com/apache/incubator-apisix/issues/914#issuecomment-565806586
 
 
   > @chnliyong Is this FHS way is good for docker?
   
   Actually, the docker way does not care this.
   
   As `apisix` run within `OpenResty`, it's hard to pass variable by command 
line arguments, the most common way is using the nginx `prefix` as a base 
directory. So the `convention` of apisix is not `FHS`, it's `Nginx`


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


With regards,
Apache Git Services


[GitHub] [incubator-apisix-dashboard] juzhiyuan commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入

2019-12-15 Thread GitBox
juzhiyuan commented on issue #88: 配置路由参数类型错误,urls、hosts选项不能录入
URL: 
https://github.com/apache/incubator-apisix-dashboard/issues/88#issuecomment-565806430
 
 
   You can input any value directly or select any existing options


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


With regards,
Apache Git Services