You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/11/13 04:11:39 UTC
[GitHub] [apisix] backnero opened a new issue #2738: bug: ? Is replaced with %3
backnero opened a new issue #2738:
URL: https://github.com/apache/apisix/issues/2738
### Issue description
The routing is configured as follows
```
{
"priority": 0,
"methods": [
"GET",
"OPTIONS"
],
"uris": [
"/conf/agent/config/raw/C/*"
],
"plugins": {
"proxy-rewrite": {
"regex_uri": [
"^/conf/agent/config/raw/C/(.*)/(.*)",
"/agent/api/v1/config/raw?appName=C&clusterName=$1&namespaceName=$2"
],
"scheme": "http"
}
}
}
```
? Is replaced with %3
```
127.0.0.1 - - [13/Nov/2020:04:07:58 +0000] "GET /agent/api/v1/config/raw%3FappName=C&clusterName=dev&namespaceName=router HTTP/1.1" 404
```
### 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:
users@infra.apache.org
[GitHub] [apisix] HelloBug0 commented on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
HelloBug0 commented on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-727134715
I run you test and get `/agent/api/v1/config/raw?appName=C&clusterName=dev&namespaceName=router` which works normal.
Actually, it is %3F. If we urlencoded '?', we will get %3F. With this clue, you can check it again. Or there is something I missed?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [apisix] idbeta edited a comment on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
idbeta edited a comment on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-726539771
@backnero I did not reproduce in my env, my upstream is built by python. Could you provide more information?
```shell
curl -X GET http://127.0.0.1:9080/conf/agent/config/raw/C/aa/aa
```
```shell
127.0.0.1 - - [13/Nov/2020 13:56:01] "GET /agent/api/v1/config/raw?appName=C&clusterName=aa&namespaceName=aa HTTP/1.1" 200 -
Host: 127.0.0.1
X-Real-IP: 127.0.0.1
X-Forwarded-For: 127.0.0.1
X-Forwarded-Proto: http
X-Forwarded-Host: 127.0.0.1
X-Forwarded-Port: 9080
User-Agent: curl/7.54.0
Accept: */*
```
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [apisix] membphis commented on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
membphis commented on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-729548152
please make a try with the lastest version or 2.0 . I remember we have fixed 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:
users@infra.apache.org
[GitHub] [apisix] membphis edited a comment on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
membphis edited a comment on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-729548152
please make a try with the lastest version or 2.0 . I remembered we have fixed 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:
users@infra.apache.org
[GitHub] [apisix] HelloBug0 edited a comment on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
HelloBug0 edited a comment on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-727134715
I run you test and get `/agent/api/v1/config/raw?appName=C&clusterName=dev&namespaceName=router` which works normal.
Actually, it is %3F. If we urlencoded '?', we will get %3F. With this clue, you can check it again. Or is there something I missed?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [apisix] backnero closed issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
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 specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] idbeta commented on issue #2738: bug: ? Is replaced with %3
Posted by GitBox <gi...@apache.org>.
idbeta commented on issue #2738:
URL: https://github.com/apache/apisix/issues/2738#issuecomment-726539771
@backnero I did not reproduce in my env, my upstream is built by python.
```shell
curl -X GET http://127.0.0.1:9080/conf/agent/config/raw/C/aa/aa
```
```shell
127.0.0.1 - - [13/Nov/2020 13:56:01] "GET /agent/api/v1/config/raw?appName=C&clusterName=aa&namespaceName=aa HTTP/1.1" 200 -
Host: 127.0.0.1
X-Real-IP: 127.0.0.1
X-Forwarded-For: 127.0.0.1
X-Forwarded-Proto: http
X-Forwarded-Host: 127.0.0.1
X-Forwarded-Port: 9080
User-Agent: curl/7.54.0
Accept: */*
```
----------------------------------------------------------------
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:
users@infra.apache.org