artilleryio / artillery-core

artillery-core - deprecated
Mozilla Public License 2.0
29 stars 104 forks source link

Allow configuration of socket.io transports #143

Closed pavelkucera closed 7 years ago

pavelkucera commented 7 years ago

Hi. I would like to use Artillery to test a socket.io application which doesn't support long polling, which is not possible with current implementation of Artillery. Without specifying that long polling isn't available socket.io tries connecting via long polling and fails.

This pull request allows using following structure in config:

config:
    socketio:
        transports: ["websocket"]

To tell socket.io-client that it should connect only using websockets. When the section is not defined, default values are used.

I'll be happy to adjust the code if necessary.

hassy commented 7 years ago

Looks good 👍

pavelkucera commented 7 years ago

Thank you!

pavelkucera commented 7 years ago