Skip to content

Commit

Permalink
ci: remove whitelist of whitespace check (apache#4203)
Browse files Browse the repository at this point in the history
  • Loading branch information
spacewander authored May 10, 2021
1 parent cc577d2 commit 6a7db72
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
steps:
- uses: actions/checkout@v2
- name: 🧹 Check for trailing whitespace
run: "! git grep -EIn $'[ \t]+$' | grep -v 'received: '"
run: "! git grep -EIn $'[ \t]+$'"
misc:
name: misc checker
runs-on: ubuntu-latest
Expand Down
6 changes: 5 additions & 1 deletion t/lib/test_admin.lua
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,11 @@ function _M.test_ipv6(uri)
while true do
local line, err, part = sock:receive()
if line then
ngx.say("received: ", line)
if line ~= "" then
ngx.say("received: ", line)
else
ngx.say("received:")
end

else
ngx.say("failed to receive a line: ", err, " [", part, "]")
Expand Down
2 changes: 1 addition & 1 deletion t/node/remote-addr-ipv6.t
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ received: Content-Type: text/plain
received: Content-Length: 12
received: Connection: close
received: Server: APISIX/\d\.\d+(\.\d+)?
received:
received:
received: hello world
failed to receive a line: closed \[\]
close: 1 nil}
Expand Down

0 comments on commit 6a7db72

Please sign in to comment.