You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by GitBox <gi...@apache.org> on 2020/10/07 15:33:03 UTC
[GitHub] [qpid-dispatch] kgiusti opened a new pull request #868: Corrupt raw output testcase DO NOT MERGE
kgiusti opened a new pull request #868:
URL: https://github.com/apache/qpid-dispatch/pull/868
Using proton master (commit 341d30f8cff0ba98bac65604ad87e4d01062080c)
To reproduce:
$ mkdir BUILD; cd BUILD; cmake ..; make
$ python tests/run.py "-m" "unittest" "-v" "system_tests_http1_adaptor.Http1AdaptorEdge2EdgeTest"
You should see some output, then the test will hang until it times out. If you reproduce the problem there will be console output with something like this:
FIXME
SETUP DONE
test_01_streaming (system_tests_http1_adaptor.Http1AdaptorEdge2EdgeTest)
Test multiple clients sending streaming messages in parallel ... 2020-10-07 11:18:47.323080 TestServer listening on :9090
SERVERS UP
SPAWNING CLIENTS
WAITING CLIENTS
CHUNK READ HEADER...
CHUNK HEADER=b'1f9' (505)
CHUNK DATA (LEN)=507
CHUNK READ HEADER...
CHUNK HEADER=b'000000' (0)
LAST CHUNK
CHUNKED BODY=b'000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
00000000000000000000000000000000000000000000000000000000000000000000000000000000**\x00Su\xb0\x00\x00\x02\x00**000000000000000000000000000000000000000\
00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
SERVER SHUTDOWN 11
2020-10-07 11:18:47.752470 TestServer :9090 shutting down
CLIENT POST SHUTDOWN...
2020-10-07 11:18:47.772930 TestServer :9090 closed
<snip>
The test sends an http message with a chunked body encoding. Each chunk is 505 bytes long and is patterned with '0' for the first chunk, '1' for the second, ... up to '9' then the last chunk is filled with 'a'.
In the above console output you can see that some extra data appears in the chunk (highlighted).
The router will record the contents of raw buffers that have been passed to the raw connection, before and after the buffers are written. See the file
./system_test.dir/system_tests_http1_adaptor/Http1AdaptorEdge2EdgeTest/setUpClass/EA2-3.out
And look for statements like this:
[C1] **Raw Write**: Ptr=0x15dc8a0 len=128
value='1f9^M
000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
[C1] Raw Write: Ptr=0x17b9fe0 len=512
value='0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
[C1] Raw Write: Ptr=0x17ba0cf len=273
value='0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
000000000000000^M
1f9^M
<snip>
[C1] **Raw Written**: Ptr=0x15dc8a0 len=128
value='1f9^M
000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
[C1] Raw Written: Ptr=0x17b9fe0 len=512
value='0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
[C1] Raw Written: Ptr=0x17ba0cf len=273
value='0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000\
000000000000000^M
1f9^M
use tcpdump/wireshark to capture messages sent to port 8080 (client->router) and port 9090 (router->server) to see what is appearing on the wire
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-dispatch] kgiusti commented on pull request #868: Corrupt raw output testcase DO NOT MERGE
Posted by GitBox <gi...@apache.org>.
kgiusti commented on pull request #868:
URL: https://github.com/apache/qpid-dispatch/pull/868#issuecomment-705563834
This appears to be a problem somewhere in the adaptor's AMQP message construction logic. Still debugging...
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-dispatch] kgiusti commented on pull request #868: Corrupt raw output testcase DO NOT MERGE
Posted by GitBox <gi...@apache.org>.
kgiusti commented on pull request #868:
URL: https://github.com/apache/qpid-dispatch/pull/868#issuecomment-708558183
abandoned
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-dispatch] kgiusti closed pull request #868: Corrupt raw output testcase DO NOT MERGE
Posted by GitBox <gi...@apache.org>.
kgiusti closed pull request #868:
URL: https://github.com/apache/qpid-dispatch/pull/868
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] [qpid-dispatch] kgiusti commented on pull request #868: Corrupt raw output testcase DO NOT MERGE
Posted by GitBox <gi...@apache.org>.
kgiusti commented on pull request #868:
URL: https://github.com/apache/qpid-dispatch/pull/868#issuecomment-705563834
This appears to be a problem somewhere in the adaptor's AMQP message construction logic. Still debugging...
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org