lizeming commented on issue #2468:
URL: https://github.com/apache/apisix/issues/2468#issuecomment-718400583
We tried the proxy-rewrite plugin to add custom header fields, and then
overwrite them with custom fields in the business proxy. This is indeed
feasible.
But apisix-dashboard
idbeta opened a new pull request #2559:
URL: https://github.com/apache/apisix/pull/2559
### What this PR does / why we need it:
#2558
ping @membphis
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been added?
moonming commented on issue #2377:
URL: https://github.com/apache/apisix/issues/2377#issuecomment-718395097
> But if the plugin order support self define, the priority may be not work,
please confirm.
@nic-chen please confirm this question, thx
-
spacewander closed issue #2553:
URL: https://github.com/apache/apisix/issues/2553
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 th
This is an automated email from the ASF dual-hosted git repository.
spacewander 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 61bcb18 fix: jwt-auth add consumer_schema
spacewander merged pull request #2555:
URL: https://github.com/apache/apisix/pull/2555
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
moonming commented on pull request #2499:
URL: https://github.com/apache/apisix/pull/2499#issuecomment-718390987
I don't think it is a good idea to modify the priority. I will take a look
again
This is an automated message f
membphis commented on issue #2533:
URL: https://github.com/apache/apisix/issues/2533#issuecomment-718390939
> [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed while
this domain should be parsed into IP address.
1. Do we set the right DNS resolver?
2. We sh
moonming commented on issue #614:
URL:
https://github.com/apache/apisix-dashboard/issues/614#issuecomment-718389460
This proposal looks good to me.
If it is added to github actions, I suggest that each type of test use a
separate action, so that it can be processed in parallel
---
membphis commented on issue #2558:
URL: https://github.com/apache/apisix/issues/2558#issuecomment-718389155
good luck
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
juzhiyuan commented on a change in pull request #619:
URL: https://github.com/apache/apisix-dashboard/pull/619#discussion_r514013342
##
File path: src/e2e/Login.e2e.js
##
@@ -0,0 +1,39 @@
+/* eslint-disable import/no-extraneous-dependencies */
+const puppeteer = require('puppet
juzhiyuan commented on a change in pull request #619:
URL: https://github.com/apache/apisix-dashboard/pull/619#discussion_r514012072
##
File path: .github/workflows/frontend-e2e-test.yml
##
@@ -0,0 +1,31 @@
+name: Front-end e2e test
+
+on:
+ push:
+branches:
+ - maste
idbeta commented on issue #2558:
URL: https://github.com/apache/apisix/issues/2558#issuecomment-718388035
I will try to fix it.
This is an automated message from the Apache Git Service.
To respond to the message, please log o
Yangzcy423911 closed issue #2551:
URL: https://github.com/apache/apisix/issues/2551
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
moonming commented on issue #2533:
URL: https://github.com/apache/apisix/issues/2533#issuecomment-718386861
I copy the error msg:
```
kubectl --namespace apisix exec -it apisix-ctznq -- cat logs/error.log
# 2020/10/27 02:39:56 [warn] 1#1: low address bits of 172.17.0.0/12 are
mean
Yangzcy423911 commented on issue #2551:
URL: https://github.com/apache/apisix/issues/2551#issuecomment-718386663
明白了，谢谢您！
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
membphis opened a new issue #2558:
URL: https://github.com/apache/apisix/issues/2558
should remove the first space in doc for `" , "`
```shell
$ find ./doc -name "*.md" | xargs grep " , "
./doc/architecture-design.md:* `Absolute match `: Complete match for
the given `u
membphis commented on a change in pull request #2389:
URL: https://github.com/apache/apisix/pull/2389#discussion_r514005987
##
File path: apisix/plugins/skywalking.lua
##
@@ -55,26 +76,74 @@ end
function _M.rewrite(conf, ctx)
core.log.debug("rewrite phase of skywalking pl
membphis opened a new issue #2557:
URL: https://github.com/apache/apisix/issues/2557
https://github.com/apache/apisix/pull/2389/files#r513940449
we can move funtion `try_read_attr` to `apisix.core.table`, then we can
reuse this function between different plugin.
---
nic-chen opened a new pull request #625:
URL: https://github.com/apache/apisix-dashboard/pull/625
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bugfix
- [x] New feature provided
- [ ] Improve performance
___
###
juzhiyuan commented on issue #614:
URL:
https://github.com/apache/apisix-dashboard/issues/614#issuecomment-718382945
3. Route
a. Please adjust those cases' order, it should be: create, get, put, search,
delete, get
b. how to generate 200+ records? through Puppeteer or admin API?
juzhiyuan edited a comment on issue #614:
URL:
https://github.com/apache/apisix-dashboard/issues/614#issuecomment-718382945
3. Route
a. Please adjust those cases' order, it should be: create, get, put, search,
delete, get
b. how to generate 200+ records? through Puppeteer or admin AP
juzhiyuan commented on pull request #619:
URL: https://github.com/apache/apisix-dashboard/pull/619#issuecomment-718379418
Your CI failed because of License issues, fix them first.
This is an automated message from the Apache
juzhiyuan closed issue #622:
URL: https://github.com/apache/apisix-dashboard/issues/622
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 #622:
URL:
https://github.com/apache/apisix-dashboard/issues/622#issuecomment-718378718
I will close this issue due to the duplicate one #598
This is an automated message from the Apache Git Ser
juzhiyuan commented on a change in pull request #619:
URL: https://github.com/apache/apisix-dashboard/pull/619#discussion_r513919280
##
File path: .github/workflows/frontend-e2e-test.yml
##
@@ -0,0 +1,31 @@
+name: Front-end e2e test
+
+on:
+ push:
+branches:
+ - maste
juzhiyuan commented on pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81#issuecomment-718377923
> please confirm we run `alpine/Dockerfile` in CI
https://github.com/apache/apisix-docker/blob/feat-remove-dashboard/example/docker-compose-alpine.yml#L7
---
membphis commented on pull request #2549:
URL: https://github.com/apache/apisix/pull/2549#issuecomment-718377817
@htwdjqr many thx for your contribution ^_^
This is an automated message from the Apache Git Service.
To respond
membphis commented on pull request #2556:
URL: https://github.com/apache/apisix/pull/2556#issuecomment-718377287
@shoogoome please take a look at the output of CI:
https://github.com/apache/apisix/pull/2556/checks?check_run_id=1324835665#step:10:187
---
membphis commented on pull request #2556:
URL: https://github.com/apache/apisix/pull/2556#issuecomment-718377017
@qiujiayu do you have time to look at this PR?
This is an automated message from the Apache Git Service.
To resp
juzhiyuan closed issue #623:
URL: https://github.com/apache/apisix-dashboard/issues/623
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 #623:
URL:
https://github.com/apache/apisix-dashboard/issues/623#issuecomment-718376662
2.0 is totally different from 1.5, 2.0 doesn't have MySQL anymore, it's
lighter and clearer, once you find any question, feel free to open issues and
tell us.
This i
juzhiyuan commented on issue #623:
URL:
https://github.com/apache/apisix-dashboard/issues/623#issuecomment-718375523
Hi @wangxunfan, we are refactoring the new Dashboard 2.0, and it's under the
2.0 RC2 stage, would you like to have a try on the newer dashboard?
Here is the milestone
membphis commented on pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81#issuecomment-718375264
please confirm we run `alpine/Dockerfile` in CI
This is an automated message from the Apache Git Service.
To
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch feat-remove-dashboard
in repository https://gitbox.apache.org/repos/asf/apisix-docker.git.
from 2e575ff Update Dockerfile
add 7219ea2 featL remove dashboard build-stage
add 720
juzhiyuan commented on issue #614:
URL:
https://github.com/apache/apisix-dashboard/issues/614#issuecomment-718372805
Just hold on, give us no longer than 3 days to check this issue.
This is an automated message from the Apac
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch feat-remove-dashboard
in repository https://gitbox.apache.org/repos/asf/apisix-docker.git.
from eb2c14b feat: remove Dashboard
add 2e575ff Update Dockerfile
No new revisions were a
juzhiyuan closed issue #546:
URL: https://github.com/apache/apisix-dashboard/issues/546
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 commented on issue #546:
URL:
https://github.com/apache/apisix-dashboard/issues/546#issuecomment-718371164
We need to remove `api/internal/core/store/validate_mock.go` and
`api/internal/core/storage/storage_mock.go` manually before releasing.
-
juzhiyuan opened a new issue #624:
URL: https://github.com/apache/apisix-dashboard/issues/624
TODO
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
juzhiyuan commented on issue #624:
URL:
https://github.com/apache/apisix-dashboard/issues/624#issuecomment-718370682
cc @membphis @gxthrj
This is an automated message from the Apache Git Service.
To respond to the message,
LiteSun commented on issue #614:
URL:
https://github.com/apache/apisix-dashboard/issues/614#issuecomment-718369685
Any suggestions for this e2e test solution?
@liuxiran @bzp2010
This is an automated message from the Apac
Jaycean commented on a change in pull request #2555:
URL: https://github.com/apache/apisix/pull/2555#discussion_r513983451
##
File path: t/plugin/jwt-auth.t
##
@@ -519,3 +519,23 @@ GET /t
\{"error_msg":"failed to check the configuration of plugin jwt-auth err:
additional prop
shoogoome opened a new pull request #2556:
URL: https://github.com/apache/apisix/pull/2556
### What this PR does / why we need it:
support multi service discovery
### Pre-submission checklist:
* [ ] Did you explain what problem does this PR solve? Or what new features
have
wangxunfan opened a new issue #623:
URL: https://github.com/apache/apisix-dashboard/issues/623
Please answer these questions before submitting your issue.
- Why do you submit this issue?
- [√ ] Question or discussion
- [√] Bug
- [ ] Requirements
- [ ] Feature or performanc
moonming merged pull request #2549:
URL: https://github.com/apache/apisix/pull/2549
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 change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from d2aa1d8 feat: get schema by schema_type (#2509)
add 87ce6af update install-dependencies.md (#2549)
No new re
moonming commented on a change in pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81#discussion_r513959976
##
File path: alpine/Dockerfile
##
@@ -7,10 +7,6 @@ RUN set -x \
&& apk add --no-cache --virtual .build-deps git \
&& git clone -b ${APISIX_V
Jaycean commented on a change in pull request #2555:
URL: https://github.com/apache/apisix/pull/2555#discussion_r513949482
##
File path: t/plugin/jwt-auth.t
##
@@ -519,3 +519,23 @@ GET /t
\{"error_msg":"failed to check the configuration of plugin jwt-auth err:
additional prop
juzhiyuan commented on a change in pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81#discussion_r513947957
##
File path: alpine/Dockerfile
##
@@ -7,10 +7,6 @@ RUN set -x \
&& apk add --no-cache --virtual .build-deps git \
&& git clone -b ${APISIX_
juzhiyuan commented on a change in pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81#discussion_r513947957
##
File path: alpine/Dockerfile
##
@@ -7,10 +7,6 @@ RUN set -x \
&& apk add --no-cache --virtual .build-deps git \
&& git clone -b ${APISIX_
juzhiyuan closed issue #79:
URL: https://github.com/apache/apisix-website/issues/79
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
spacewander commented on a change in pull request #2555:
URL: https://github.com/apache/apisix/pull/2555#discussion_r513945321
##
File path: t/plugin/jwt-auth.t
##
@@ -519,3 +519,23 @@ GET /t
\{"error_msg":"failed to check the configuration of plugin jwt-auth err:
additional
juzhiyuan opened a new pull request #81:
URL: https://github.com/apache/apisix-docker/pull/81
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
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch feat-remove-dashboard
in repository https://gitbox.apache.org/repos/asf/apisix-docker.git
commit eb2c14b91cacd306fe10f99cc9a8c5824b066df2
Author: juzhiyuan
AuthorDate: Thu Oct 29 12:05:51 2020
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch feat-remove-dashboard
in repository https://gitbox.apache.org/repos/asf/apisix-docker.git.
at eb2c14b feat: remove Dashboard
This branch includes the following new commits:
new e
liuxiran commented on pull request #2555:
URL: https://github.com/apache/apisix/pull/2555#issuecomment-718345936
add a test case: get jwt-auth schema would be better @Jaycean :)
This is an automated message from the Apache G
juzhiyuan opened a new issue #622:
URL: https://github.com/apache/apisix-dashboard/issues/622
This one will track all issues related to 2.0 RC3
- [ ] https://github.com/apache/apisix-dashboard/issues/615
This is an aut
spacewander commented on a change in pull request #2389:
URL: https://github.com/apache/apisix/pull/2389#discussion_r513896783
##
File path: doc/plugins/skywalking.md
##
@@ -67,59 +64,97 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H
'X-API-KEY: edd1c9f034335f1
}
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/apisix-website.git
The following commit(s) were added to refs/heads/asf-site by this push:
new c6ed083 Deploy to GitHub pages
moonming merged pull request #80:
URL: https://github.com/apache/apisix-website/pull/80
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
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-website.git
The following commit(s) were added to refs/heads/master by this push:
new 4eae07f doc: add 2.0 release. (#80)
4e
juzhiyuan commented on issue #615:
URL:
https://github.com/apache/apisix-dashboard/issues/615#issuecomment-718343149
After discussions, we are going to fix this in 2.0 RC3, it will take no
longer than 3 days.
This is an aut
nic-chen opened a new issue #621:
URL: https://github.com/apache/apisix-dashboard/issues/621
Please answer these questions before submitting your issue.
- Why do you submit this issue?
- [ ] Question or discussion
- [ ] Bug
- [x] Requirements
- [ ] Feature or performance i
thekingofworld edited a comment on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718339899
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
thekingofworld commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718339899
@spacewander > The plugin doesn't set Content-Length correctly.
It seems to be the problem.
This is an auto
dabue commented on a change in pull request #2488:
URL: https://github.com/apache/apisix/pull/2488#discussion_r513924200
##
File path: apisix/plugins/error-log-logger.lua
##
@@ -0,0 +1,235 @@
+--
+-- Licensed to the Apache Software Foundation (ASF) under one or more
+-- contrib
spacewander commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718337888
It is a known issue.
> NOTE: echo plugin is written as an example. There are some unhandled cases
and you should not use it in the production!
https://github.com/apache/a
nic-chen opened a new issue #620:
URL: https://github.com/apache/apisix-dashboard/issues/620
After the manager api refactoring, there are still many problems, as follow:
1. Access log:
Save to a local file, and can customize the path
Support log rotate
Support custom log form
moonming opened a new pull request #80:
URL: https://github.com/apache/apisix-website/pull/80
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
This is an automated email from the ASF dual-hosted git repository.
wenming pushed a change to branch release-2.0
in repository https://gitbox.apache.org/repos/asf/apisix-website.git.
at 312cc68 doc: add 2.0 release.
This branch includes the following new commits:
new 312cc68 doc:
This is an automated email from the ASF dual-hosted git repository.
wenming pushed a commit to branch release-2.0
in repository https://gitbox.apache.org/repos/asf/apisix-website.git
commit 312cc68704bc161dc5e9a5a430164eb0a9b7b3fa
Author: WenMing
AuthorDate: Thu Oct 29 11:30:20 2020 +0800
d
thekingofworld commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718336780
the upstream `127.0.0.1:8080` is a simple http server, implemented through
`Go` language
![image](https://user-images.githubusercontent.com/8169665/97522169-1755dc80-19da-11
thekingofworld commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718334757
@moonming OK, here is text
```
curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY:
edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '
{
"plugins": {
moonming commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718332935
@thekingofworld Can you copy the text of the command line? The picture is
not easy to debug
This is an automated mes
moonming commented on issue #2554:
URL: https://github.com/apache/apisix/issues/2554#issuecomment-718332608
@Akayeshmantha do you have time to take a look? thx
This is an automated message from the Apache Git Service.
To resp
htwdjqr commented on a change in pull request #2549:
URL: https://github.com/apache/apisix/pull/2549#discussion_r513908283
##
File path: doc/zh-cn/install-dependencies.md
##
@@ -40,15 +40,21 @@ CentOS 7
wget http://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
Jaycean opened a new pull request #2555:
URL: https://github.com/apache/apisix/pull/2555
Signed-off-by: Jaycean
### What this PR does / why we need it:
fix: #2553
- jwt-auth add consumer_schema to the _M
### Pre-submission checklist:
* [x] Did you explain what pr
lizeming commented on issue #2468:
URL: https://github.com/apache/apisix/issues/2468#issuecomment-718328570
This is one of the business request chains.
This business has been around for a long time.
Request chain: client -> internet address(2xx.xxx.xxx.xx9:8096) ->
pre-agent
Caelebs commented on issue #615:
URL:
https://github.com/apache/apisix-dashboard/issues/615#issuecomment-718326560
Before the 2.1 version is released, how should I modify the source code to
build the manager-api docker image file?
@juzhiyuan
--
thekingofworld opened a new issue #2554:
URL: https://github.com/apache/apisix/issues/2554
### Issue description
After adding the official echo plugin, the output content did not meet
expectations.
add d2aa1d8 feat: get schema by schema_type (#2509)
No new
juzhiyuan merged pull request #2509:
URL: https://github.com/apache/apisix/pull/2509
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
Jaycean commented on pull request #2509:
URL: https://github.com/apache/apisix/pull/2509#issuecomment-718301182
> LGTM
>
> tips:
> after this pr merged, jwt-auth also needs to add `consumer_schema` to the
`_M` @Jaycean
>
https://github.com/apache/apisix/blob/1dfc4cdacd620e6d59
juzhiyuan closed issue #2441:
URL: https://github.com/apache/apisix/issues/2441
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
spacewander opened a new pull request #2552:
URL: https://github.com/apache/apisix/pull/2552
Close #2517.
### What this PR does / why we need it:
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been
Yangzcy423911 commented on issue #2551:
URL: https://github.com/apache/apisix/issues/2551#issuecomment-718300433
您好，没有找到关于TLS KEY的配置
This is an automated message from the Apache Git Service.
To respond to the message, please
liuxiran edited a comment on issue #2082:
URL: https://github.com/apache/apisix/issues/2082#issuecomment-698736589
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
spacewander commented on issue #2551:
URL: https://github.com/apache/apisix/issues/2551#issuecomment-718298247
See https://github.com/apache/apisix/pull/2548
This is an automated message from the Apache Git Service.
To respon
liuxiran commented on pull request #2509:
URL: https://github.com/apache/apisix/pull/2509#issuecomment-718296772
LGTM
tips:
after this pr merged, jwt-auth also needs to add `consumer_schema` to the
`_M` @Jaycean
https://github.com/apache/apisix/blob/1dfc4cdacd620e6d59c3d4c755
zhangguanzhang commented on issue #2530:
URL: https://github.com/apache/apisix/issues/2530#issuecomment-718296485
This part of the information is too little, a more complete entry process
document is needed
This is an automa
juzhiyuan commented on a change in pull request #2524:
URL: https://github.com/apache/apisix/pull/2524#discussion_r513843881
##
File path: .github/workflows/build.yml
##
@@ -12,7 +12,12 @@ jobs:
fail-fast: false
matrix:
platform: [ubuntu-18.04]
-o
1 - 100 of 162 matches
Mail list logo
