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/06/08 01:45:04 UTC

[GitHub] [incubator-apisix] moonming opened a new issue #1666: add test cases in `t/` for admin api https access

moonming opened a new issue #1666:
URL: https://github.com/apache/incubator-apisix/issues/1666


   I think the test coverage of add test case in shell is too low, e.g. I don't know what happens if I open https but send an http request, and it not test that the host is incorrect, it's basically the same as not testing at all.
   Maybe we can merge this PR first, but test cases must added later.
   
   _Originally posted by @moonming in https://github.com/apache/incubator-apisix/pull/1648_


----------------------------------------------------------------
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