SeasideSt / Parasol

Testing web apps in Smalltalk using Selenium WebDriver.
MIT License
31 stars 16 forks source link

BPPharoPlatform>>#handleRequest: contains commented-out code to strip NULL characters from the response #22

Open Rinzwind opened 5 years ago

Rinzwind commented 5 years ago

There's some code in the method BPPharoPlatform>>#handleRequest: to strip trailing NULL characters from the response:

https://github.com/SeasideSt/Parasol/blob/c0aa4c2fb48a1627d924cad48a13205c231d3574/repository/Parasol-Pharo.package/BPPharoPlatform.class/instance/handleRequest..st#L7-L8

Either that can be removed, or it should be re-enabled again. Based on the change done in https://github.com/SeasideSt/Parasol/pull/15/commits/83ad7f2a6ed14271254fcab326c6a1845992da2f, I would say it might still be necessary.