Fix a bug that ManagementFrontend adds a garbage to http response#582
Fix a bug that ManagementFrontend adds a garbage to http response#582simon-mo merged 3 commits intoucbrise:developfrom
Conversation
|
Can one of the admins verify this patch? |
|
jenkins test this please |
|
Jenkins ok to test |
|
Jenkins test this please |
|
Test FAILed. |
|
Jenkins test this please |
|
Test FAILed. |
|
Jenkins test this please |
|
Test PASSed. |
|
@simon-mo : |
|
@withsmilo thanks, i just rebased it, I will merge this once the test passes again. I understand the tests are still flaky. I will take care of it. |
|
Test FAILed. |
|
Jenkins test this please |
|
Test FAILed. |
Currently ManagementFrontend sends ['http header' + 'content' + '\n'] as a http response to the outer side. Our team found some error cases occurred by wrong value(content.length()) of Content-Length in the 'http header'. To fix it, '\n' must be removed from the http response. I found the same bug in QueryFrontend and pull-requested it already. (ucbrise#415)
withsmilo
force-pushed
the
fix_mgnt_frontend_bug
branch
from
aace07f to
524fb0b
Compare
|
Jenkins test this please |
|
Test PASSed. |
|
Test PASSed. |
…brise#582) * Fix a bug that ManagementFrontend adds a garbage to http response Currently ManagementFrontend sends ['http header' + 'content' + '\n'] as a http response to the outer side. Our team found some error cases occurred by wrong value(content.length()) of Content-Length in the 'http header'. To fix it, '\n' must be removed from the http response. I found the same bug in QueryFrontend and pull-requested it already. (ucbrise#415) * Add a timer to container_test
…brise#582) * Fix a bug that ManagementFrontend adds a garbage to http response Currently ManagementFrontend sends ['http header' + 'content' + '\n'] as a http response to the outer side. Our team found some error cases occurred by wrong value(content.length()) of Content-Length in the 'http header'. To fix it, '\n' must be removed from the http response. I found the same bug in QueryFrontend and pull-requested it already. (ucbrise#415) * Add a timer to container_test
…brise#582) * Fix a bug that ManagementFrontend adds a garbage to http response Currently ManagementFrontend sends ['http header' + 'content' + '\n'] as a http response to the outer side. Our team found some error cases occurred by wrong value(content.length()) of Content-Length in the 'http header'. To fix it, '\n' must be removed from the http response. I found the same bug in QueryFrontend and pull-requested it already. (ucbrise#415) * Add a timer to container_test
4 participants
Currently ManagementFrontend sends
['http header' + 'content' + '\n']as a http response to the outer side. Our team found some error cases occurred by wrong value(content.length()) of Content-Length in the 'http header'. To fix it,\nmust be removed from the http response.I found the same bug in QueryFrontend and pull-requested it already. (#415)