How can I add query params in plugin

[mofazhe]

I based on this request definition of parameters:

Here is my code of plugin:

function handleRequest() {
    // console.log(context.request.getBody())

    var timestamp=Date.now();
    console.log(timestamp)
    context.request.setEnvironmentVariable('timestamp', timestamp)
    var params=context.request.getQueryParams();
    console.log(params)
    params.push({"name":"time","value":"1232"});
    console.log(params)
    context.request.setQueryParams(params);
}

if('request' in context) {
    handleRequest()
}

But, here is the result:

Changes to environment variables in handleRequest also do not take effect in the request parameters, see the environment variable timestamp above.

Editing is only because the picture may leak privacy, please ignore it. Sorry for disturbing you.

[flawiddsouza]

Changes to environment variables in handleRequest also do not take effect in the request parameters, see the environment variable timestamp above.

This is now fixed.

This should work:

const queryParams = context.request.getQueryParams()
context.request.setQueryParams([...queryParams, { name: 'cat', value: '3' }])

Your code should work but looks like there's a limitation in the underlying library that I'm using for running plugins. I'll look into fixing that.

[mofazhe]

Changes to environment variables in handleRequest also do not take effect in the request parameters, see the environment variable timestamp above.

This is now fixed.

I will try it in next version.

This should work:

const queryParams = context.request.getQueryParams()
context.request.setQueryParams([...queryParams, { name: 'cat', value: '3' }])

I tried the code and it worked fine.

Your code should work but looks like there's a limitation in the underlying library that I'm using for running plugins. I'll look into fixing that.

That would be great.

[flawiddsouza]

Closing this for now, as there's a workaround. I'll try to fix the query params direct usage issue in the next release. A new version has been released. It has the fix for the environment variable issue you reported.