On Fri, Aug 14, 2020 at 9:01 PM vincixu <[email protected]> wrote: > Not every request, just the requests which enable the geo plugin. >
We can not get the plugin list before matching route. That is why I advise we should support to fetch `city` info from `ctx.var`. > > YuanSheng Wang <[email protected]> 于2020年8月14日周五 下午3:17写道： > > > > parse IP to city(or any geo results) and put it to header before route > > matching > > > > we can not do this before route matching for every request. For example, > > not all routes > > require user city information. > > > > we should only do the parse when we need it. and we only parse the info > we > > need. > > > > so the `ctx.var` is a good way for us. > > > > > > On Fri, Aug 14, 2020 at 9:47 AM vincixu <[email protected]> wrote: > > > > > I think APISIX has already supported it.All we need to do is just parse > > IP > > > to city(or any geo results) and put it to header before route matching, > > so > > > users could define route with parsed result. > > > Am I misunderstanding something? > > > > > > > > > -- > > > > *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
