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/apisix-dashboard.git
The following commit(s) were added to refs/heads/master by this push:
new 56d39ba fix README typo (#389)
juzhiyuan merged pull request #389:
URL: https://github.com/apache/apisix-dashboard/pull/389
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
juzhiyuan commented on issue #1781:
URL: https://github.com/apache/apisix/issues/1781#issuecomment-678725258
Closed because the dashboard related docs are point to its repo
This is an automated message from the Apache Git
juzhiyuan closed issue #1781:
URL: https://github.com/apache/apisix/issues/1781
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
membphis edited a comment on pull request #2001:
URL: https://github.com/apache/apisix/pull/2001#issuecomment-678724605
@redynasc it seems that the new test case is unstable. we need to fix this
before we merge this PR
```
# Failed test 'TEST 3: about frequency - response_body
membphis commented on pull request #2001:
URL: https://github.com/apache/apisix/pull/2001#issuecomment-678724605
it seems that the new test case is unstable. we need to fix this before we
merge this PR
```
# Failed test 'TEST 3: about frequency - response_body - response is
membphis merged pull request #2103:
URL: https://github.com/apache/apisix/pull/2103
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
membphis commented on a change in pull request #2103:
URL: https://github.com/apache/apisix/pull/2103#discussion_r475163915
##
File path: doc/plugins/request-id.md
##
@@ -71,7 +71,7 @@ HTTP/1.1 200 OK
## Disable Plugin
-Remove the corresponding json configuration in the
membphis closed issue #2027:
URL: https://github.com/apache/apisix/issues/2027
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
This is an automated email from the ASF dual-hosted git repository.
membphis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new 6ffd8b9 doc: Chinese version of plugins
membphis commented on a change in pull request #2103:
URL: https://github.com/apache/apisix/pull/2103#discussion_r475163731
##
File path: doc/zh-cn/plugins/request-id.md
##
@@ -0,0 +1,89 @@
+
+
+[English](../plugins/request-id.md)
+
+# 目录
+
+- [**名称**](#名称)
+- [**属性**](#属性)
+-
Yiyiyimu commented on pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#issuecomment-678719772
> @Yiyiyimu not just for `apisix start`, we need to handle here too:
>
>
https://github.com/apache/apisix/blob/master/apisix/core/config_local.lua#L30
>
Yiyiyimu commented on issue #1781:
URL: https://github.com/apache/apisix/issues/1781#issuecomment-678718798
Hi @juzhiyuan which part is outdated? I seem failed to find related doc in
dashboard repo
This is an automated
Yiyiyimu opened a new pull request #389:
URL: https://github.com/apache/apisix-dashboard/pull/389
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bug fix
- [ ] New feature provided
- [ ] Improve performance
- [x] Doc
Yiyiyimu commented on pull request #2103:
URL: https://github.com/apache/apisix/pull/2103#issuecomment-678717624
Currently, doc in both languages is missing `examples` and `test plugin`,
maybe we need to create another issue on these.
Yiyiyimu opened a new pull request #2103:
URL: https://github.com/apache/apisix/pull/2103
### What this PR does / why we need it:
fix #2027
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been added?
* [x]
nic-chen edited a comment on pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#issuecomment-678714864
@Yiyiyimu not just for `apisix start`, we need to handle here too:
https://github.com/apache/apisix/blob/master/apisix/core/config_local.lua#L30
nic-chen commented on issue #1991:
URL: https://github.com/apache/apisix/issues/1991#issuecomment-678715063
> @nic-chen Hello, When will this issue be fixed?
>
> 1. The grpc-transcode plugin, after the protocol conversion, the
parameters obtained by the back-end service are
nic-chen commented on pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#issuecomment-678714864
@Yiyiyimu not just for `apisix start`, we need to handle here:
https://github.com/apache/apisix/blob/master/apisix/core/profile.lua#L26
nic-chen commented on pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#issuecomment-678712944
> I think we can merge this PR at next week.
I think we should create a new branch that using etcd v2 before the
merge,and maintain it for a while,for the users who
Yiyiyimu commented on pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#issuecomment-678662273
> @Yiyiyimu and we need to update the document about this new feature
Sure I just added the relevant document.
Yiyiyimu commented on a change in pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#discussion_r475107379
##
File path: .travis/apisix_cli_test.sh
##
@@ -204,3 +204,26 @@ fi
sed -i 's/worker_processes: 2/worker_processes: auto/' conf/config.yaml
echo
Yiyiyimu commented on pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#issuecomment-678654583
> I think we can merge this PR at next week.
That would be great!
One more question, which place in doc do you suggest to put the etcd
migration link? Like we
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475100953
##
File path: doc/install-dependencies.md
##
@@ -28,12 +28,12 @@
Note
-- Apache APISIX currently only supports the v2 protocol storage to etcd,
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475100877
##
File path: bin/apisix
##
@@ -870,35 +870,30 @@ local function init_etcd(show_output)
local host_count = #(yaml_conf.etcd.host)
--- check
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475100649
##
File path: apisix/core/etcd.lua
##
@@ -15,11 +15,36 @@
-- limitations under the License.
--
local fetch_local_conf =
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475100669
##
File path: apisix/core/etcd.lua
##
@@ -43,25 +70,140 @@ local function new()
end
_M.new = new
+local function kvs2node(kvs)
+local node = {}
+
membphis opened a new pull request #2102:
URL: https://github.com/apache/apisix/pull/2102
### What this PR does / why we need it:
fix #2086
### Pre-submission checklist:
* [ ] Did you explain what problem does this PR solve? Or what new features
have been
membphis commented on pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#issuecomment-678647597
I think we can merge this PR at next week.
This is an automated message from the Apache Git Service.
To respond
membphis commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475095648
##
File path: apisix/core/etcd.lua
##
@@ -15,11 +15,36 @@
-- limitations under the License.
--
local fetch_local_conf =
membphis commented on a change in pull request #2091:
URL: https://github.com/apache/apisix/pull/2091#discussion_r475094944
##
File path: apisix/plugins/ip-restriction.lua
##
@@ -27,21 +27,31 @@ local lrucache = core.lrucache.new({
local schema = {
type = "object",
-
membphis commented on pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#issuecomment-678643282
@Yiyiyimu and we need to update the document about this new feature
This is an automated message from the
membphis commented on a change in pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#discussion_r475092789
##
File path: .travis/apisix_cli_test.sh
##
@@ -204,3 +204,26 @@ fi
sed -i 's/worker_processes: 2/worker_processes: auto/' conf/config.yaml
echo
membphis merged pull request #2100:
URL: https://github.com/apache/apisix/pull/2100
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
This is an automated email from the ASF dual-hosted git repository.
membphis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git
The following commit(s) were added to refs/heads/master by this push:
new d974c18 change: use draft7 style of JSON
membphis closed issue #2083:
URL: https://github.com/apache/apisix/issues/2083
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
Yiyiyimu commented on a change in pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#discussion_r475091197
##
File path: .travis/apisix_cli_test.sh
##
@@ -204,3 +204,26 @@ fi
sed -i 's/worker_processes: 2/worker_processes: auto/' conf/config.yaml
echo
Yiyiyimu opened a new pull request #2101:
URL: https://github.com/apache/apisix/pull/2101
### What this PR does / why we need it:
fix #2034 : allow user to specify the `conf/config.yaml` when starting the
APISIX project
Copy customer config.yaml to default location for
This is an automated email from the ASF dual-hosted git repository.
kvn pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-docker.git
The following commit(s) were added to refs/heads/master by this push:
new 6ac41f4 fix: mv config.yml to
gxthrj merged pull request #72:
URL: https://github.com/apache/apisix-docker/pull/72
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
Yiyiyimu commented on issue #2034:
URL: https://github.com/apache/apisix/issues/2034#issuecomment-678626404
@membphis Hi I'm working on this feature. I wonder if it could be a good
idea to overwrite the specified config.yaml to the default location
conf/config.yaml, since customized
membphis opened a new pull request #2100:
URL: https://github.com/apache/apisix/pull/2100
fix #2083.
### What this PR does / why we need it:
fix #2083, as title.
### Pre-submission checklist:
* [ ] Did you explain what problem does this PR solve? Or what new
membphis opened a new pull request #2099:
URL: https://github.com/apache/apisix/pull/2099
…rification
with other parameters.
### What this PR does / why we need it:
fix #2084
```
=== TEST 4: get plugin node-status schema
--- request
GET
membphis commented on issue #2098:
URL: https://github.com/apache/apisix/issues/2098#issuecomment-678617440
welcome PR to fix this issue
This is an automated message from the Apache Git Service.
To respond to the message,
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475052806
##
File path: apisix/core/etcd.lua
##
@@ -15,11 +15,36 @@
-- limitations under the License.
--
local fetch_local_conf =
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r475051880
##
File path: bin/apisix
##
@@ -870,35 +870,30 @@ local function init_etcd(show_output)
local host_count = #(yaml_conf.etcd.host)
--- check
juzhiyuan closed issue #28:
URL: https://github.com/apache/apisix-website/issues/28
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
juzhiyuan closed issue #61:
URL: https://github.com/apache/apisix-website/issues/61
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
juzhiyuan closed issue #62:
URL: https://github.com/apache/apisix-website/issues/62
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
juzhiyuan closed issue #51:
URL: https://github.com/apache/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
50 matches
Mail list logo