istio / old_issues_repo

Deprecated issue-tracking repo, please post new issues or feature requests to istio/istio instead.
37 stars 9 forks source link

istio incorrect handle of ingress backend path #86

Open eigokor opened 7 years ago

eigokor commented 7 years ago

Is this a BUG or FEATURE REQUEST?:

Did you review existing epics or issues to identify if this already being worked on? (please try to add the correct labels and epics):

Bug: Y

What Version of Istio and Kubernetes are you using, where did you get Istio from, Installation details

istioctl version
istioctl version:

Version: 0.1.6
GitRevision: dab2033
GitBranch: release-0.1
User: jenkins@ubuntu-16-04-build-12ac793f80be71
GolangVersion: go1.8
KubeInjectHub: docker.io/istio
KubeInjectTag: 0.1

apiserver version:

Version: 0.1.6
GitRevision: dab2033
GitBranch: release-0.1
User: jenkins@ubuntu-16-04-build-12ac793f80be71
GolangVersion: go1.8.1

kubectl version
Client Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.6", GitCommit:"4bc5e7f9a6c25dc4c03d4d656f2cefd21540e28c", GitTreeState:"clean", BuildDate:"2017-09-14T06:55:55Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.5", GitCommit:"17d7182a7ccbb167074be7a87f0a68bd00d58d97", GitTreeState:"clean", BuildDate:"2017-08-31T08:56:23Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Is Istio Auth enabled or not ? Did you install istio.yaml, istio-auth.yaml.... istio.yaml

What happened: ingress rule path: /internal/v1.0/service is handled as regex

What you expected to happen: pattern must be regex only if ".*" used

How to reproduce it: https://stackoverflow.com/questions/46399248/istio-incorrect-handle-of-ingress-backend-path/46528125#46528125

Feature Request: Y/N

Describe the feature:

linsun commented 7 years ago

Issue moved to istio/pilot #1389 via ZenHub

rshriram commented 7 years ago

Lets keep this open

rshriram commented 7 years ago

also cc @louiscryan @smawson