kallaballa / janosh Goto Github PK

I am getting a bus error, without a stack strace or something else while trying to send strings which are 20 characters or longer. The bug is reproduceable by running the following minimal lua script...

#!/lounge/bin/janosh -f

function receive(handle, message)
   print(message)
end

Janosh:wsOpen(8080)
Janosh:wsOnReceive(receive)

while true do
   Janosh:sleep(1000)
end

...and pasting the following snippet in your browsers javascript console.

var socket = new WebSocket('ws://10.20.30.40:8080');
socket.onopen = function() { socket.send('a'.repeat(20)); }

...which kills the server process. It works if one sends less than 20 characters.

Any ideas?

Some tests fail on Darwin - add/remove. Needs to be looked into.

seclorum@Darwin-machine:~/Documents/metalab/kallaballa/Janosh.seclorum {(janosh-on-darwin)}
$ ./test.sh

truncate: ok

truncate:46082543134820304

mkarray: ok

mkarray:5095506346131095679

mkobject: ok

mkobject:6033625201148905563

append: ok

append:9510324768122757536

set: ok

set:17106756998089531113

add: failed

add:772402077030743180

remove: failed

remove:772402077030743180

replace: ok

replace:1926185854176296875

copy: ok

copy:7217090941002447847

shift: ok

shift:3752346509794287921

the json the server returns will be broken if the values posted include a backslash.

need to investigate if this is caused by users posting backslash values or if converting to json has a bug.

I'm posting this issue to this repo, because Janosh should never return malformed json. I'm going to look into this later on, looks like a problem with with "url". Maybe it's related to #3 ?

at the moment duplicate subscriptions to the same key prefix are not allow and result in a failing assertion.

now that we have the exception wrapper for luajit replace all asserts in lua related code with exceptions.

implement c++ code that guards transactions in lua.

At the moment it is possible to (try to) open a transaction inside an open transaction. The results in a deadlock. find a mechanism to report a nested transaction as error.