[GitHub] [apisix] spacewander commented on issue #2954: ctx.var.upstream_addr may be separated by commas.

2020-12-06 Thread GitBox
spacewander commented on issue #2954: URL: https://github.com/apache/apisix/issues/2954#issuecomment-739605847 I look through the code and find that we don't use `ctx.var.upstream_xxx` in balancer phase. As there is nothing to purge, we can simply leave a comment.

[GitHub] [apisix] spacewander commented on issue #2954: ctx.var.upstream_addr may be separated by commas.

2020-12-04 Thread GitBox
spacewander commented on issue #2954: URL: https://github.com/apache/apisix/issues/2954#issuecomment-738700318 I think there are two solution 1. use `ngx.var` instead 2. explicitly clear the cache before retrying What about your opinion? @membphis

[GitHub] [apisix] spacewander commented on issue #2954: ctx.var.upstream_addr may be separated by commas.

2020-12-03 Thread GitBox
spacewander commented on issue #2954: URL: https://github.com/apache/apisix/issues/2954#issuecomment-738500790 PR is welcome! The value is 0 because `tonumber("xxx")` is nil. This is an automated message from the Apache Git S