Hi, I want to ask if this feature is currently under development or has
been merged into the trunk branch,  "Extract the host from the
proxy-rewrite plugin into the upstream object” .

In addition, the routes object also has a host configuration, but in some
specific scenarios, there will be multiple domain names corresponding to
this route or a group of routes. I think it is more appropriate for the
host to be placed in the service object? Or is there any other better way
to use routes?

Thanks,
yousa

YuanSheng Wang <[email protected]> 于2020年1月4日周六 上午8:47写道:

> On Fri, Jan 3, 2020 at 5:54 PM Li Ling(Lien) <[email protected]> wrote:
>
> >  I think  a websocket enable feature here can be also good.
> >
>
> Of course, we can support it.
>
>
> >
> > junxu chen <[email protected]> 于2020年1月3日周五 下午5:40写道:
> > >
> > > >
> > > > curl -i http://127.0.0.1:9080/apisix/admin/services/1 -X PUT -d '
> > > > {
> > > >     "upstream": {
> > > >         "path": "/index.html",
> > > >         "host": "foo.com",
> > > >         "scheme": "http",
> > > >         "type": "roundrobin",
> > > >         "nodes": {
> > > >             "39.97.63.215:80 <http://39.97.63.215/>": 1
> > > >         }
> > > >     }
> > > > }'
> > > >
> > >
> > > is better
> > >
> > > On Thu, Jan 2, 2020 at 6:04 PM Ming Wen <[email protected]> wrote:
> > >
> > > > looks good to me.
> > > >
> > > > Thanks,
> > > > Ming Wen, Apache APISIX
> > > > Twitter: _WenMing
> > > >
> > > >
> > > > YuanSheng Wang <[email protected]> 于2020年1月2日周四 下午5:58写道:
> > > >
> > > > > curl -i http://127.0.0.1:9080/apisix/admin/services/1 -X PUT -d '
> > > > > {
> > > > >     "upstream": {
> > > > >         "path": "/index.html",
> > > > >         "host": "foo.com",
> > > > >         "scheme": "http",
> > > > >         "type": "roundrobin",
> > > > >         "nodes": {
> > > > >             "39.97.63.215:80": 1
> > > > >         }
> > > > >     }
> > > > > }'
> > > > >
> > > > > How about we put them in the `upstream` object?
> > > > >
> > > > > On Thu, Jan 2, 2020 at 4:06 PM YuanSheng Wang <[email protected]>
> > > > wrote:
> > > > >
> > > > > > > This implementation will replace some features of the
> > `proxy-rewrite`
> > > > > > plugin,
> > > > > >
> > > > > > Yes
> > > > > >
> > > > > > > The `up_*` naming is hard to understand.
> > > > > >
> > > > > > Any good name style?
> > > > > >
> > > > > >
> > > > > > On Thu, Jan 2, 2020 at 3:27 PM Ming Wen <[email protected]>
> > wrote:
> > > > > >
> > > > > >> This implementation will replace some features of the
> > `proxy-rewrite`
> > > > > >> plugin,
> > > > > >> right?
> > > > > >> BTW, the `up_*` naming is hard to understand.
> > > > > >>
> > > > > >> Thanks,
> > > > > >> Ming Wen, Apache APISIX
> > > > > >> Twitter: _WenMing
> > > > > >>
> > > > > >>
> > > > > >> YuanSheng Wang <[email protected]> 于2020年1月2日周四 下午2:53写道:
> > > > > >>
> > > > > >> > Hi:
> > > > > >> >
> > > > > >> > Supports more upstream parameters in `service`, this will make
> > > > APISIX
> > > > > >> > easier to use.
> > > > > >> >
> > > > > >> > What do you think?
> > > > > >> >
> > > > > >> > For example: up_path, up_host, up_port or up_scheme.
> > > > > >> >
> > > > > >> > curl -i http://127.0.0.1:9080/apisix/admin/services/1 -X PUT
> > -d '
> > > > > >> > {
> > > > > >> >     "upstream": {
> > > > > >> >         "type": "roundrobin",
> > > > > >> >         "nodes": {
> > > > > >> >             "39.97.63.215:80": 1
> > > > > >> >         }
> > > > > >> >     },
> > > > > >> >     "up_path": "/index.html",
> > > > > >> >     "up_host": "foo.com",
> > > > > >> >     "up_scheme": "http"
> > > > > >> > }'
> > > > > >> >
> > > > > >> > curl -i http://127.0.0.1:9080/apisix/admin/services/1 -X PUT
> > -d '
> > > > > >> > {
> > > > > >> >     "upstream": {
> > > > > >> >         "type": "roundrobin",
> > > > > >> >         "nodes": {
> > > > > >> >             "39.97.63.215:80": 1
> > > > > >> >         }
> > > > > >> >     },
> > > > > >> >     "up_host": "foo.com",
> > > > > >> >     "up_scheme": "https"
> > > > > >> > }'
> > > > > >> >
> > > > > >> > --
> > > > > >> >
> > > > > >> > *MembPhis*
> > > > > >> > My github: https://github.com/membphis
> > > > > >> > Apache APISIX: https://github.com/apache/incubator-apisix
> > > > > >> >
> > > > > >>
> > > > > >
> > > > > >
> > > > > > --
> > > > > >
> > > > > > *MembPhis*
> > > > > > My github: https://github.com/membphis
> > > > > > Apache APISIX: https://github.com/apache/incubator-apisix
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > >
> > > > > *MembPhis*
> > > > > My github: https://github.com/membphis
> > > > > Apache APISIX: https://github.com/apache/incubator-apisix
> > > > >
> > > >
> >
>
>
> --
> *MembPhis*
> My github: https://github.com/membphis
> Apache APISIX: https://github.com/apache/incubator-apisix
>

Reply via email to