[GitHub] [apisix] membphis commented on a change in pull request #2368: add uri fuzzing test
membphis commented on a change in pull request #2368: URL: https://github.com/apache/apisix/pull/2368#discussion_r510898494 ## File path: t/specialtest/cases/test_fuzziing_uri_of_route.py ## @@ -0,0 +1,142 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +# !/usr/bin/env python +# -*- coding: utf-8 -*- +import os +import time +import subprocess +import random +import urllib +import re +import requests +import psutil +import grequests + + +def get_pid_byname(): +name = "apisix" +cmd = "ps -ef | grep %s/conf | grep master | grep -v grep| awk '{print $2}'" % name +p = subprocess.Popen(cmd, stderr=subprocess.PIPE, + stdout=subprocess.PIPE, shell=True) +p.wait() +return p.stdout.read().strip() + + +def get_workerres(pid): +parent = psutil.Process(pid) +children = parent.children(recursive=True) +for p in children: +cp = psutil.Process(p.pid) +print(p.pid, cp.cpu_percent(interval=1.0), cp.memory_percent()) + + +def cur_file_dir(): +return os.path.split(os.path.realpath(__file__))[0] + + +def requesttest(url, times): +start = time.time() +tasks = [] +r = [] +while time.time() - start <= times: +tasks.append(grequests.get(url)) +res = grequests.map(tasks, size=50) +r.extend([i.status_code for i in res]) +return r + + +def setup_module(): +global headers, apisixhost, apisixpid, apisixpath +apisixpid = int(get_pid_byname()) +apisixpath = psutil.Process(apisixpid).cwd() +os.chdir(apisixpath) + +subprocess.call("./bin/apisix stop", shell=True, stdout=subprocess.PIPE) Review comment: I think this one should be right, please confirm. ``` subprocess.call("./bin/apisix stop", shell=True, stdout=subprocess.PIPE) time.sleep(1) subprocess.Popen("> logs/error.log", shell=True, stdout=subprocess.PIPE) subprocess.call("etcd &", shell=True, stdout=subprocess.PIPE) ## File path: t/specialtest/cases/test_fuzziing_uri_of_route.py ## @@ -0,0 +1,142 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +# !/usr/bin/env python +# -*- coding: utf-8 -*- +import os +import time +import subprocess +import random +import urllib +import re +import requests +import psutil +import grequests + + +def get_pid_byname(): +name = "apisix" +cmd = "ps -ef | grep %s/conf | grep master | grep -v grep| awk '{print $2}'" % name +p = subprocess.Popen(cmd, stderr=subprocess.PIPE, + stdout=subprocess.PIPE, shell=True) +p.wait() +return p.stdout.read().strip() + + +def get_workerres(pid): +parent = psutil.Process(pid) +children = parent.children(recursive=True) +for p in children: +cp = psutil.Process(p.pid) +print(p.pid, cp.cpu_percent(interval=1.0), cp.memory_percent()) + + +def cur_file_dir(): +return os.path.split(os.path.realpath(__file__))[0] + + +def requesttest(url, times): +start = time.time() +tasks = [] +r = [] +while time.time() - start <= times: +tasks.append(grequests.get(url)) +res = grequests.map(tasks, size=50) +r.extend([i.status_code for i in res]) +return r + + +def setup_module(): +global headers, apisixhost, apisixpid, apisixpath +apisixpid = int(get_pid_byname()) +apisixpath = psutil.Process(apisixpid).cwd() +os.chdir(apisixpath) + +subprocess.call("./bin/apisix stop", shell=True, stdout=subproc
[GitHub] [apisix] membphis commented on a change in pull request #2368: add uri fuzzing test
membphis commented on a change in pull request #2368: URL: https://github.com/apache/apisix/pull/2368#discussion_r510897373 ## File path: .travis/linux_specialtest_runner.sh ## @@ -0,0 +1,120 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +set -ex + +export_or_prefix() { +export OPENRESTY_PREFIX="/usr/local/openresty-debug" +} + +create_lua_deps() { +echo "Create lua deps cache" + +make deps +luarocks install luacov-coveralls --tree=deps --local > build.log 2>&1 || (cat build.log && exit 1) + +sudo rm -rf build-cache/deps +sudo cp -r deps build-cache/ +sudo cp rockspec/apisix-master-0.rockspec build-cache/ +} + +before_install() { +echo "Before install" +} + +do_install() { +export_or_prefix + +wget -qO - https://openresty.org/package/pubkey.gpg | sudo apt-key add - +sudo apt-get -y update --fix-missing +sudo apt-get -y install software-properties-common +sudo add-apt-repository -y "deb http://openresty.org/package/ubuntu $(lsb_release -sc) main" + +sudo apt-get update +sudo apt-get install openresty-debug lua5.1 liblua5.1-0-dev + +wget https://github.com/luarocks/luarocks/archive/v2.4.4.tar.gz +tar -xf v2.4.4.tar.gz +cd luarocks-2.4.4 +./configure --prefix=/usr > build.log 2>&1 || (cat build.log && exit 1) +make build > build.log 2>&1 || (cat build.log && exit 1) +sudo make install > build.log 2>&1 || (cat build.log && exit 1) +cd .. +rm -rf luarocks-2.4.4 + +sudo luarocks install luacheck > build.log 2>&1 || (cat build.log && exit 1) + +./utils/install-etcd.sh + +if [ ! -f "build-cache/apisix-master-0.rockspec" ]; then +create_lua_deps + +else +src=`md5sum rockspec/apisix-master-0.rockspec | awk '{print $1}'` +src_cp=`md5sum build-cache/apisix-master-0.rockspec | awk '{print $1}'` +if [ "$src" = "$src_cp" ]; then +echo "Use lua deps cache" +sudo cp -r build-cache/deps ./ +else +create_lua_deps +fi +fi +} + +script() { +export_or_prefix +export PATH=$OPENRESTY_PREFIX/nginx/sbin:$OPENRESTY_PREFIX/luajit/bin:$OPENRESTY_PREFIX/bin:$PATH +openresty -V +sudo service etcd stop +mkdir -p ~/etcd-data +etcd --listen-client-urls 'http://0.0.0.0:2379' --advertise-client-urls='http://0.0.0.0:2379' --data-dir ~/etcd-data > /dev/null 2>&1 & +etcdctl version +sleep 5 + +./bin/apisix start + +#start again --> fial +res=`./bin/apisix start` +if [ "$res" != "APISIX is running..." ]; then +echo "failed: APISIX runs repeatedly" +exit 1 +fi + +cd t/specialtest/ +sudo python runtest.py +pytest --force-flaky --max-runs=3 --no-flaky-report -v -s "cases" + +} + + +case_opt=$1 +shift + +case ${case_opt} in +before_install) +before_install "$@" +;; +do_install) +do_install "$@" +;; +script) +script "$@" +;; + + +esac Review comment: got it, thx 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: us...@infra.apache.org
[GitHub] [apisix] membphis commented on a change in pull request #2368: add uri fuzzing test
membphis commented on a change in pull request #2368: URL: https://github.com/apache/apisix/pull/2368#discussion_r510698277 ## File path: .travis/linux_specialtest_runner.sh ## @@ -0,0 +1,120 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +set -ex + +export_or_prefix() { +export OPENRESTY_PREFIX="/usr/local/openresty-debug" +} + +create_lua_deps() { +echo "Create lua deps cache" + +make deps +luarocks install luacov-coveralls --tree=deps --local > build.log 2>&1 || (cat build.log && exit 1) + +sudo rm -rf build-cache/deps +sudo cp -r deps build-cache/ +sudo cp rockspec/apisix-master-0.rockspec build-cache/ +} + +before_install() { +echo "Before install" +} + +do_install() { +export_or_prefix + +wget -qO - https://openresty.org/package/pubkey.gpg | sudo apt-key add - +sudo apt-get -y update --fix-missing +sudo apt-get -y install software-properties-common +sudo add-apt-repository -y "deb http://openresty.org/package/ubuntu $(lsb_release -sc) main" + +sudo apt-get update +sudo apt-get install openresty-debug lua5.1 liblua5.1-0-dev + +wget https://github.com/luarocks/luarocks/archive/v2.4.4.tar.gz +tar -xf v2.4.4.tar.gz +cd luarocks-2.4.4 +./configure --prefix=/usr > build.log 2>&1 || (cat build.log && exit 1) +make build > build.log 2>&1 || (cat build.log && exit 1) +sudo make install > build.log 2>&1 || (cat build.log && exit 1) +cd .. +rm -rf luarocks-2.4.4 + +sudo luarocks install luacheck > build.log 2>&1 || (cat build.log && exit 1) + +./utils/install-etcd.sh + +if [ ! -f "build-cache/apisix-master-0.rockspec" ]; then +create_lua_deps + +else +src=`md5sum rockspec/apisix-master-0.rockspec | awk '{print $1}'` +src_cp=`md5sum build-cache/apisix-master-0.rockspec | awk '{print $1}'` +if [ "$src" = "$src_cp" ]; then +echo "Use lua deps cache" +sudo cp -r build-cache/deps ./ +else +create_lua_deps +fi +fi +} + +script() { +export_or_prefix +export PATH=$OPENRESTY_PREFIX/nginx/sbin:$OPENRESTY_PREFIX/luajit/bin:$OPENRESTY_PREFIX/bin:$PATH +openresty -V +sudo service etcd stop +mkdir -p ~/etcd-data +etcd --listen-client-urls 'http://0.0.0.0:2379' --advertise-client-urls='http://0.0.0.0:2379' --data-dir ~/etcd-data > /dev/null 2>&1 & +etcdctl version +sleep 5 + +./bin/apisix start + +#start again --> fial +res=`./bin/apisix start` +if [ "$res" != "APISIX is running..." ]; then +echo "failed: APISIX runs repeatedly" +exit 1 +fi + +cd t/specialtest/ +sudo python runtest.py +pytest --force-flaky --max-runs=3 --no-flaky-report -v -s "cases" + Review comment: remove this blank line ## File path: .travis/linux_specialtest_runner.sh ## @@ -0,0 +1,120 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +set -ex + +export_or_prefix() { +export OPENRESTY_PREFIX="/usr/local/openresty-debug" +} + +create_lua_deps() { +echo "Create lua deps cache" + +make deps +luarocks install luacov-coveralls --tree=deps --local > build.log 2>&1 || (cat build.log && exit 1) + +sudo rm -rf build-cache/deps +sudo cp -r deps build-cache/ +sudo cp rockspec/apisix-master-0.rockspec build-cache/ +} + +before_install() { +echo "Before install" +} + +do_install() { +
[GitHub] [apisix] membphis commented on a change in pull request #2368: add uri fuzzing test
membphis commented on a change in pull request #2368: URL: https://github.com/apache/apisix/pull/2368#discussion_r505621566 ## File path: .travis/linux_specialtest_runner.sh ## @@ -34,22 +34,23 @@ create_lua_deps() { } before_install() { -sudo cpanm --notest Test::Nginx >build.log 2>&1 || (cat build.log && exit 1) -docker pull redis:3.0-alpine -docker run --rm -itd -p 6379:6379 --name apisix_redis redis:3.0-alpine -docker run --rm -itd -e HTTP_PORT= -e HTTPS_PORT= -p : -p : mendhak/http-https-echo -# Runs Keycloak version 10.0.2 with inbuilt policies for unit tests -docker run --rm -itd -e KEYCLOAK_USER=admin -e KEYCLOAK_PASSWORD=123456 -p 8090:8080 -p 8443:8443 sshniro/keycloak-apisix -# spin up kafka cluster for tests (1 zookeper and 1 kafka instance) -docker pull bitnami/zookeeper:3.6.0 -docker pull bitnami/kafka:latest -docker network create kafka-net --driver bridge -docker run --name zookeeper-server -d -p 2181:2181 --network kafka-net -e ALLOW_ANONYMOUS_LOGIN=yes bitnami/zookeeper:3.6.0 -docker run --name kafka-server1 -d --network kafka-net -e ALLOW_PLAINTEXT_LISTENER=yes -e KAFKA_CFG_ZOOKEEPER_CONNECT=zookeeper-server:2181 -e KAFKA_CFG_ADVERTISED_LISTENERS=PLAINTEXT://127.0.0.1:9092 -p 9092:9092 -e KAFKA_CFG_AUTO_CREATE_TOPICS_ENABLE=true bitnami/kafka:latest -docker pull bitinit/eureka -docker run --name eureka -d -p 8761:8761 --env ENVIRONMENT=apisix --env spring.application.name=apisix-eureka --env server.port=8761 --env eureka.instance.ip-address=127.0.0.1 --env eureka.client.registerWithEureka=true --env eureka.client.fetchRegistry=false --env eureka.client.serviceUrl.defaultZone=http://127.0.0.1:8761/eureka/ bitinit/eureka -sleep 5 -docker exec -i kafka-server1 /opt/bitnami/kafka/bin/kafka-topics.sh --create --zookeeper zookeeper-server:2181 --replication-factor 1 --partitions 1 --topic test2 +echo "Before install" +# sudo cpanm --notest Test::Nginx >build.log 2>&1 || (cat build.log && exit 1) Review comment: if we do not need them, we should delete them directly. ## File path: .travis/linux_specialtest_runner.sh ## @@ -93,34 +94,34 @@ do_install() { # sudo apt-get install tree -y # tree deps -git clone https://github.com/iresty/test-nginx.git test-nginx -make utils - -git clone https://github.com/apache/openwhisk-utilities.git .travis/openwhisk-utilities -cp .travis/ASF* .travis/openwhisk-utilities/scancode/ - -ls -l ./ -if [ ! -f "build-cache/grpc_server_example" ]; then -wget https://github.com/iresty/grpc_server_example/releases/download/20200901/grpc_server_example-amd64.tar.gz -tar -xvf grpc_server_example-amd64.tar.gz -mv grpc_server_example build-cache/ -fi - -if [ ! -f "build-cache/proto/helloworld.proto" ]; then -if [ ! -f "grpc_server_example/main.go" ]; then -git clone https://github.com/iresty/grpc_server_example.git grpc_server_example -fi - -cd grpc_server_example/ -mv proto/ ../build-cache/ -cd .. -fi - -if [ ! -f "build-cache/grpcurl" ]; then -wget https://github.com/api7/grpcurl/releases/download/20200314/grpcurl-amd64.tar.gz -tar -xvf grpcurl-amd64.tar.gz -mv grpcurl build-cache/ -fi +# git clone https://github.com/iresty/test-nginx.git test-nginx Review comment: dittoo 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: us...@infra.apache.org
[GitHub] [apisix] membphis commented on a change in pull request #2368: add uri fuzzing test
membphis commented on a change in pull request #2368: URL: https://github.com/apache/apisix/pull/2368#discussion_r503905280 ## File path: .travis/linux_specialtest_runner.sh ## @@ -0,0 +1,167 @@ +#!/usr/bin/env bash +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE file distributed with +# this work for additional information regarding copyright ownership. +# The ASF licenses this file to You under the Apache License, Version 2.0 +# (the "License"); you may not use this file except in compliance with +# the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# + +set -ex + +export_or_prefix() { +export OPENRESTY_PREFIX="/usr/local/openresty-debug" +} + +create_lua_deps() { +echo "Create lua deps cache" + +make deps +luarocks install luacov-coveralls --tree=deps --local > build.log 2>&1 || (cat build.log && exit 1) + +sudo rm -rf build-cache/deps +sudo cp -r deps build-cache/ +sudo cp rockspec/apisix-master-0.rockspec build-cache/ +} + +before_install() { +sudo cpanm --notest Test::Nginx >build.log 2>&1 || (cat build.log && exit 1) Review comment: I think we need to remove those useless install steps for this test case. Please submit the smallest available version. 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: us...@infra.apache.org