Remember cookies for following requests in scenario spec about artillery-core HOT 9 CLOSED

Thanks @gajjargaurav! 🌟 Perhaps the jar should be enabled by default, with the option to disable it on per-scenario basis?

from artillery-core.

@hassy could do that. However request by default doesn't set it. If we set up that as default we are making an assumption that users would want that most of the time. I don't see that to be the case.

from artillery-core.

@gajjargaurav Yes, perhaps, although if the server doesn't send any cookies, there would be no performance overhead - and if it does, then the user probably wants these cookies to be sent on subsequent requests. Either way, I think cookies should be a global option (set in config.defaults), and it should be possible to set it on individual scenarios. The implementation should also make sure that there's a different cookie jar for every running scenario, rather than a global request one.

from artillery-core.

Yes, that sounds like a plan. Thanks

from artillery-core.

@hassy , I've done a quick fix for now if that is acceptable. I would like to extend that to actually pass in cookies to set to in the cookie jar.

from artillery-core.

@gajjargaurav It does not seem to work as described in the issue, i.e. cookie support should be enabled by default.

I added a basic test case in this commit: e7ac291 - this makes sure that cookies are enabled by default - will you have a chance to rewrite the PR so that this test passes? If not, I'm happy to do it.

from artillery-core.

@hassy , I will look into it during the weekend and update the PR.

from artillery-core.

@gajjargaurav Awesome, thanks! :)

from artillery-core.

@hassy I did get it working locally for global request object, however I am having issues doing with a separate jar for each scenario

from artillery-core.