JanLi-air opened a new issue #2833:
URL: https://github.com/apache/apisix/issues/2833
### Issue description
config.yaml is used to override those configurations in config-default.yaml.
The problem is that some configurations are too niche to be included in
config-default.yaml. Suc
membphis commented on pull request #2832:
URL: https://github.com/apache/apisix/pull/2832#issuecomment-732717149
same problem, close this issue
![image](https://user-images.githubusercontent.com/6814606/100063220-d384b480-2e6b-11eb-92db-64fe792de4e7.png)
---
membphis closed pull request #2832:
URL: https://github.com/apache/apisix/pull/2832
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 opened a new pull request #2832:
URL: https://github.com/apache/apisix/pull/2832
same as https://github.com/apache/apisix/pull/2768
make a test
This is an automated message from the Apache Git Service.
To resp
juzhiyuan commented on issue #861:
URL:
https://github.com/apache/apisix-dashboard/issues/861#issuecomment-732711893
ok then it looks good to me 👏
This is an automated message from the Apache Git Service.
To respond to the m
liuxiran commented on issue #861:
URL:
https://github.com/apache/apisix-dashboard/issues/861#issuecomment-732710689
> @liuxiran @membphis Just to make sure, if there will have 1-to-n
requirement? (1 route with multiple group)
I think one route only belongs to one group
---
moonming merged pull request #2831:
URL: https://github.com/apache/apisix/pull/2831
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.
wenming 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 82628f0 fix: min_etcd_version is a nil value w
idbeta opened a new issue #863:
URL: https://github.com/apache/apisix-dashboard/issues/863
# Bug report
## Describe the bug
2.1 version create route with `cors` plugin and `cors` used the default
setting `{}`, as below:
```
{"name":"test","uris":["/name"],"methods":["GET","
liuxiran commented on issue #2737:
URL: https://github.com/apache/apisix/issues/2737#issuecomment-732689191
> > as for the second step I didn't find the relevant code , so do we have
any reference codes or documentations?
>
> Just to make sure, @membphis please take a look at here.
tokers commented on a change in pull request #2830:
URL: https://github.com/apache/apisix/pull/2830#discussion_r529226269
##
File path: apisix/plugins/hmac-auth.lua
##
@@ -224,6 +224,13 @@ local function generate_signature(ctx, secret_key, params)
for _, key in pairs
DHB-liuhong opened a new pull request #2831:
URL: https://github.com/apache/apisix/pull/2831
Signed-off-by: liuhong
### What this PR does / why we need it:
Executing apisix init_etcd,it will report min_etcd_version is a nil value
when etcd version is less than min_etcd_version
juzhiyuan commented on issue #2737:
URL: https://github.com/apache/apisix/issues/2737#issuecomment-732671886
> as for the second step I didn't find the relevant code , so do we have any
reference codes or documentations?
Just to make sure, @membphis please take a look at here.
juzhiyuan edited a comment on issue #2737:
URL: https://github.com/apache/apisix/issues/2737#issuecomment-732671450
@liuxiran If this is ok, kindly remove the `discuss` label and add
`enhancement`.
This is an automated mess
juzhiyuan commented on issue #2737:
URL: https://github.com/apache/apisix/issues/2737#issuecomment-732671450
If this is ok, kindly remove the `discuss` label and add `enhancement`.
This is an automated message from the Apache
juzhiyuan commented on issue #853:
URL:
https://github.com/apache/apisix-dashboard/issues/853#issuecomment-732670889
oh yes!
Please add some needed info in the log :)
This is an automated message from the Apache Git
juzhiyuan edited a comment on issue #861:
URL:
https://github.com/apache/apisix-dashboard/issues/861#issuecomment-732670507
@liuxiran @membphis Just to make sure, if there will have 1-to-n
requirement? (1 route with multiple group)
---
juzhiyuan commented on issue #861:
URL:
https://github.com/apache/apisix-dashboard/issues/861#issuecomment-732670507
Just to make sure, if there will have 1-to-n requirement? (1 route with
multiple group)
This is an automat
Firstsawyou commented on a change in pull request #2817:
URL: https://github.com/apache/apisix/pull/2817#discussion_r529218125
##
File path: apisix/init.lua
##
@@ -629,6 +630,12 @@ end
function _M.http_header_filter_phase()
core.response.set_header("Server", ver_header)
Firstsawyou commented on a change in pull request #2830:
URL: https://github.com/apache/apisix/pull/2830#discussion_r529213525
##
File path: apisix/plugins/hmac-auth.lua
##
@@ -224,6 +224,13 @@ local function generate_signature(ctx, secret_key, params)
for _, key in
idbeta opened a new issue #862:
URL: https://github.com/apache/apisix-dashboard/issues/862
# Bug report
## Describe the bug
2.1 version create route with jwt-auth will display an error
![image](https://user-images.githubusercontent.com/7396835/100048866-e50c9300-2e50-11eb-9
idbeta edited a comment on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732639178
> @idbeta pls confirm this issue, it seems to be a bug
~My 2.1 version environment also has this problem.~
sorry, 2.1 version environment did not reproduce
idbeta commented on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732639178
> @idbeta pls confirm this issue, it seems to be a bug
My 2.1 version environment also has this problem.
---
starsz commented on issue #852:
URL:
https://github.com/apache/apisix-dashboard/issues/852#issuecomment-732637307
> > So I think apisix-dashboard should forbid `POST` and merge `Create and
Update` into a function which was called by`PUT` method.
>
> agree with this. @starsz would li
This is an automated email from the ASF dual-hosted git repository.
wenming pushed a change to branch v2.1
in repository https://gitbox.apache.org/repos/asf/apisix.git.
at e8c9a61 fix
No new revisions were added by this update.
membphis commented on a change in pull request #2830:
URL: https://github.com/apache/apisix/pull/2830#discussion_r529184449
##
File path: apisix/plugins/hmac-auth.lua
##
@@ -224,6 +224,13 @@ local function generate_signature(ctx, secret_key, params)
for _, key in pai
juzhiyuan edited a comment on issue #609:
URL:
https://github.com/apache/apisix-dashboard/issues/609#issuecomment-732636310
Hi, @totemofwolf would you have time for this issue?
This is an automated message from the Apache Gi
juzhiyuan commented on issue #609:
URL:
https://github.com/apache/apisix-dashboard/issues/609#issuecomment-732636310
Hi @totemofwolf would you have time to this issue?
This is an automated message from the Apache Git Service
juzhiyuan closed issue #804:
URL: https://github.com/apache/apisix-dashboard/issues/804
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
LiteSun opened a new issue #861:
URL: https://github.com/apache/apisix-dashboard/issues/861
# why
In order to allow users to filter out routes with a specified group from a
large number of routes, I propose to add the route groups feature.
# how
## front-end
In order to fini
juzhiyuan commented on issue #756:
URL:
https://github.com/apache/apisix-dashboard/issues/756#issuecomment-732635397
duplicate with #850
This is an automated message from the Apache Git Service.
To respond to the message, p
juzhiyuan closed issue #756:
URL: https://github.com/apache/apisix-dashboard/issues/756
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
juzhiyuan closed issue #814:
URL: https://github.com/apache/apisix-dashboard/issues/814
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
juzhiyuan closed issue #802:
URL: https://github.com/apache/apisix-dashboard/issues/802
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
juzhiyuan closed issue #762:
URL: https://github.com/apache/apisix-dashboard/issues/762
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
juzhiyuan closed issue #764:
URL: https://github.com/apache/apisix-dashboard/issues/764
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
juzhiyuan closed issue #790:
URL: https://github.com/apache/apisix-dashboard/issues/790
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
juzhiyuan closed issue #812:
URL: https://github.com/apache/apisix-dashboard/issues/812
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
juzhiyuan closed issue #788:
URL: https://github.com/apache/apisix-dashboard/issues/788
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
juzhiyuan commented on issue #815:
URL:
https://github.com/apache/apisix-dashboard/issues/815#issuecomment-732635180
This issue is related to those issues.
#764
#802
#812
#814
#790
#788
#762
T
juzhiyuan closed issue #587:
URL: https://github.com/apache/apisix-dashboard/issues/587
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
starsz commented on issue #853:
URL:
https://github.com/apache/apisix-dashboard/issues/853#issuecomment-732634376
Hi,@juzhiyuan. I think the version info is needed.
And it's difficult to get `On your network` info accurately if the machine
has multiple network cards.
juzhiyuan closed issue #675:
URL: https://github.com/apache/apisix-dashboard/issues/675
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
juzhiyuan commented on issue #578:
URL:
https://github.com/apache/apisix-dashboard/issues/578#issuecomment-732632562
cc @membphis
This is an automated message from the Apache Git Service.
To respond to the message, please l
moonming commented on a change in pull request #2801:
URL: https://github.com/apache/apisix/pull/2801#discussion_r529179748
##
File path: CHANGELOG.md
##
@@ -32,6 +33,24 @@
- [0.7.0](#070)
- [0.6.0](#060)
+## 2.1.0
+
+### Core
+- :sunrise: **support ENV variable in configur
juzhiyuan closed issue #610:
URL: https://github.com/apache/apisix-dashboard/issues/610
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
juzhiyuan closed issue #624:
URL: https://github.com/apache/apisix-dashboard/issues/624
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
Firstsawyou opened a new pull request #2830:
URL: https://github.com/apache/apisix/pull/2830
### What this PR does / why we need it:
close #2823
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been a
juzhiyuan closed issue #673:
URL: https://github.com/apache/apisix-dashboard/issues/673
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
membphis commented on issue #852:
URL:
https://github.com/apache/apisix-dashboard/issues/852#issuecomment-732628199
> So I think apisix-dashboard should forbid `POST` and merge `Create and
Update` into a function which was called by`PUT` method.
agree with this. @starsz would like t
HelloBug0 commented on a change in pull request #2769:
URL: https://github.com/apache/apisix/pull/2769#discussion_r529177674
##
File path: .travis/apisix_cli_test.sh
##
@@ -389,12 +389,86 @@ if [ ! $? -eq 0 ]; then
exit 1
fi
-git checkout conf/config.yaml
+echo "passed:
starsz commented on issue #2818:
URL: https://github.com/apache/apisix/issues/2818#issuecomment-732625779
Hi, @moonming, I think there is no need to use `lua-resty-http`.
In CLI, we don't care about non-blocking, and luasocket is also easy to use
now.
What's your opinion about usi
juzhiyuan closed issue #717:
URL: https://github.com/apache/apisix-dashboard/issues/717
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
membphis commented on issue #2800:
URL: https://github.com/apache/apisix/issues/2800#issuecomment-732623644
> I'm going through the whole project background. I'm not so sure what
'init_kv=(' and 'apisix_cli_test.sh ' for, yet. Where can I figure it out
quickly. Would you give me some clues
juzhiyuan closed issue #722:
URL: https://github.com/apache/apisix-dashboard/issues/722
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
backnero commented on issue #2828:
URL: https://github.com/apache/apisix/issues/2828#issuecomment-732620943
when i enable config 'enable_debug'
```
020/11/24 03:19:08 [error] 17#17: *6274 failed to run log_by_lua*:
/usr/local/apisix/apisix/core/response.lua:128: headers have alrea
juzhiyuan commented on issue #784:
URL:
https://github.com/apache/apisix-dashboard/issues/784#issuecomment-732613042
current solution: use multiple select[1] to resolve this issue
[1] https://ant.design/components/select-cn/
-
spacewander commented on a change in pull request #2817:
URL: https://github.com/apache/apisix/pull/2817#discussion_r529174101
##
File path: apisix/init.lua
##
@@ -629,6 +630,12 @@ end
function _M.http_header_filter_phase()
core.response.set_header("Server", ver_header)
starsz edited a comment on issue #849:
URL:
https://github.com/apache/apisix-dashboard/issues/849#issuecomment-732029069
After discussed with @tokers @juzhiyuan .There are some things that need to
do in the backend.
Todo:
~~1. manager-api needs to start a goroutine to clean the n
spacewander commented on a change in pull request #2817:
URL: https://github.com/apache/apisix/pull/2817#discussion_r529097971
##
File path: t/node/upstream-status-5xx.t
##
@@ -0,0 +1,373 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
starsz commented on issue #849:
URL:
https://github.com/apache/apisix-dashboard/issues/849#issuecomment-732606230
@tokers. Agree with you. I think this can be done in DP.
So, CP doesn't need to care about it.
This is an
membphis commented on issue #2737:
URL: https://github.com/apache/apisix/issues/2737#issuecomment-732605433
I write an example, follow @liuxiran design: add new field `status`. I think
this way is good.
```
{
"uri": "/hello",# uri
"status": 0,# integer va
spacewander commented on issue #2820:
URL: https://github.com/apache/apisix/issues/2820#issuecomment-732604124
Because Nginx doesn't provide a mechanism to limit ingress traffic.
This is an automated message from the Apache G
spacewander commented on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732600680
Yes
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
juzhiyuan commented on issue #821:
URL:
https://github.com/apache/apisix-dashboard/issues/821#issuecomment-732599951
cc @liuxiran @membphis @moonming @starsz
This is an automated message from the Apache Git Service.
To resp
juzhiyuan commented on issue #822:
URL:
https://github.com/apache/apisix-dashboard/issues/822#issuecomment-732599253
cc @nic-chen @membphis @tokers @liuxiran
This is an automated message from the Apache Git Service.
To resp
wfgydbu edited a comment on issue #2820:
URL: https://github.com/apache/apisix/issues/2820#issuecomment-732546797
> There is not such a feature. But you can write your plugin works on
body_filter and do the egress limitation, then apply it to global rule. There
is not way to archive an ing
juzhiyuan commented on issue #831:
URL:
https://github.com/apache/apisix-dashboard/issues/831#issuecomment-732595015
according to current tasks, this enhancement feature will resolve in 2.3.
This is an automated message from
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 c8823d0 fix(consumer): keep the id field for
membphis merged pull request #2829:
URL: https://github.com/apache/apisix/pull/2829
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 commented on issue #839:
URL:
https://github.com/apache/apisix-dashboard/issues/839#issuecomment-732589128
Will this be resolved in 2.2?
This is an automated message from the Apache Git Service.
To respond to the m
juzhiyuan commented on issue #853:
URL:
https://github.com/apache/apisix-dashboard/issues/853#issuecomment-732584085
Just to make sure, what will log when starting the manager-api? How about
this one?
```
The manager-api is running successfully!
You can now access it by us
juzhiyuan commented on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732572672
https://github.com/apache/apisix/pull/2829
@spacewander Just to make sure, this PR will fix this issue?
juzhiyuan commented on issue #96:
URL: https://github.com/apache/apisix-website/issues/96#issuecomment-732558875
old release are removed from
https://dist.apache.org/repos/dist/release/apisix/
and https://downloads.apache.org/apisix/dashboard-2.0/ will take effect for
a while
juzhiyuan closed issue #96:
URL: https://github.com/apache/apisix-website/issues/96
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 commented on issue #96:
URL: https://github.com/apache/apisix-website/issues/96#issuecomment-732557279
DONE
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
liuxiran commented on issue #851:
URL:
https://github.com/apache/apisix-dashboard/issues/851#issuecomment-732556546
> I don't think it is a bug, audit field should maintain by server not
client.So we should not accept client audit field such as `CreatedAt`,
`CreatedBy` etc.
agree +
spacewander opened a new pull request #2829:
URL: https://github.com/apache/apisix/pull/2829
### What this PR does / why we need it:
### Pre-submission checklist:
* [ ] Did you explain what problem does this PR solve? Or what new features
have been added?
* [ ] Have
wfgydbu commented on issue #2820:
URL: https://github.com/apache/apisix/issues/2820#issuecomment-732546797
> There is not such a feature. But you can write your plugin works on
body_filter and do the egress limitation, then apply it to global rule. There
is not way to archive an ingress ba
backnero closed issue #2738:
URL: https://github.com/apache/apisix/issues/2738
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 s
juzhiyuan commented on issue #851:
URL:
https://github.com/apache/apisix-dashboard/issues/851#issuecomment-732546208
Not sure if it's a bug indeed, what's your opinion? I would be more prefer
with @ShiningRush's opinion 🤔
cc @membphis @nic-chen @liuxiran @tokers
-
spacewander edited a comment on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732537568
@cleverpig
~~Could you show your dashboard commit so that we can reproduce the issue?
It look like the latest dashboard (1a38fa6cf6540bc966098f6467a160312fb6b0f
spacewander edited a comment on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732537568
@cleverpig
~~Could you show your dashboard commit so that we can reproduce the issue?
It look like the latest dashboard (1a38fa6cf6540bc966098f6467a160312fb6b0f
Firstsawyou commented on a change in pull request #2817:
URL: https://github.com/apache/apisix/pull/2817#discussion_r529131022
##
File path: apisix/init.lua
##
@@ -629,6 +630,12 @@ end
function _M.http_header_filter_phase()
core.response.set_header("Server", ver_header)
spacewander commented on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732537568
@cleverpig
Could you show your dashboard commit so that we can reproduce the issue? It
look like the latest dashboard (1a38fa6cf6540bc966098f6467a160312fb6b0fd)
doesn
backnero edited a comment on issue #2828:
URL: https://github.com/apache/apisix/issues/2828#issuecomment-732536625
In addition, in version 2.0, when the global plug-in settings return to the
head, two servers will appear. One of the apisix, one is my custom. Is it to
show that apisix provi
Firstsawyou commented on a change in pull request #2817:
URL: https://github.com/apache/apisix/pull/2817#discussion_r529126522
##
File path: apisix/init.lua
##
@@ -629,6 +630,12 @@ end
function _M.http_header_filter_phase()
core.response.set_header("Server", ver_header)
backnero commented on issue #2828:
URL: https://github.com/apache/apisix/issues/2828#issuecomment-732536625
In addition, in version 2.0, when the global plug-in settings return to the
head, two servers will appear. One of the apisix, one is my custom. Is it to
show that apisix provides ser
backnero opened a new issue #2828:
URL: https://github.com/apache/apisix/issues/2828
### Issue description
When two routes of the same upstream-service are requested, the first route
responds successfully, and the second route shows path loss, or the response
value is the same as the fi
ShiningRush commented on issue #2006:
URL: https://github.com/apache/apisix/issues/2006#issuecomment-732533463
I will try to produce the bug in master, We have not been able to produce
this bug stably before : (
This is an a
ShiningRush edited a comment on pull request #860:
URL: https://github.com/apache/apisix-dashboard/pull/860#issuecomment-732531024
I have leave my opinion on the issue(#851 ) , we should talk more before
merge.
This is an au
ShiningRush edited a comment on issue #851:
URL:
https://github.com/apache/apisix-dashboard/issues/851#issuecomment-732531288
I don't think it is a bug, audit field should maintain by server not
client.So we should not accept client audit field such as `CreatedAt`,
`CreatedBy` etc.
ShiningRush commented on issue #851:
URL:
https://github.com/apache/apisix-dashboard/issues/851#issuecomment-732531288
I don't think it is a bug, audit field should maintain by server not
client.So we should not accept client audit field such as CreatedAt, CreatedBy
etc.
---
ShiningRush commented on pull request #860:
URL: https://github.com/apache/apisix-dashboard/pull/860#issuecomment-732531024
I don't think it is a bug, audit field should maintain by server not
client.So we should not accept client audit field such as `CreatedAt`,
`CreatedBy` etc.
---
spacewander commented on a change in pull request #2803:
URL: https://github.com/apache/apisix/pull/2803#discussion_r529111901
##
File path: .travis/apisix_cli_test.sh
##
@@ -458,6 +458,64 @@ fi
echo "passed: found 'my_dict' in nginx.conf"
+# allow injecting configuration
spacewander commented on a change in pull request #2769:
URL: https://github.com/apache/apisix/pull/2769#discussion_r529109459
##
File path: .travis/apisix_cli_test.sh
##
@@ -389,12 +389,86 @@ if [ ! $? -eq 0 ]; then
exit 1
fi
-git checkout conf/config.yaml
+echo "passe
GiftLee opened a new issue #2827:
URL: https://github.com/apache/apisix/issues/2827
### Issue description
abort.body
{"status":"success","time":"$time_local"}
$time_local is nginx variables
or Custom variable
ok english is poor
i want this
spacewander commented on a change in pull request #2801:
URL: https://github.com/apache/apisix/pull/2801#discussion_r529097117
##
File path: CHANGELOG.md
##
@@ -32,6 +33,24 @@
- [0.7.0](#070)
- [0.6.0](#060)
+## 2.1.0
+
+### Core
+- :sunrise: **support ENV variable in confi
spacewander edited a comment on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732514806
~~I guess it is because consumer id is removed in the master branch. We need
to update the stored etcd data.
https://github.com/apache/apisix/pull/2679~~
Sorr
spacewander edited a comment on issue #856:
URL:
https://github.com/apache/apisix-dashboard/issues/856#issuecomment-732514806
I guess it is because consumer id is removed in the master branch. We need
to update the stored etcd data.
https://github.com/apache/apisix/pull/2679
-
1 - 100 of 257 matches
Mail list logo