Skip to content

Commit

Permalink
fix(nginx): add group property support (#163)
Browse files Browse the repository at this point in the history
The property was already parsed by the config but was not used to create the request
  • Loading branch information
acouvreur committed Jul 20, 2023
1 parent 4fd8a24 commit 3a93180
Showing 1 changed file with 22 additions and 7 deletions.
29 changes: 22 additions & 7 deletions plugins/nginx/njs/sablier.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,16 +75,23 @@ function buildRequest(c) {
*/
function createDynamicUrl(config) {
const url = `${config.sablierUrl}/api/strategies/dynamic`
const query = querystring.stringify({
names: config.names.split(",").map(name => name.trim()),
const query = {
session_duration: config.sessionDuration,
display_name:config.displayName,
theme: config.theme,
refresh_frequency: config.refreshFrequency,
show_details: config.showDetails
});
};

if(config.names) {
query.names = config.names.split(",").map(name => name.trim())
} else if(config.group) {
query.group = config.group
} else {
throw new Error('you must specify names or group');
}

return {url, query}
return {url, query: querystring.stringify(query)}
}

/**
Expand All @@ -94,13 +101,21 @@ function buildRequest(c) {
*/
function createBlockingUrl(config) {
const url = `${config.sablierUrl}/api/strategies/blocking`
const query = querystring.stringify({
const query = {
names: config.names.split(",").map(name => name.trim()),
session_duration: config.sessionDuration,
timeout:config.timeout,
});
};

if(config.names) {
query.names = config.names.split(",").map(name => name.trim())
} else if(config.group) {
query.group = config.group
} else {
throw new Error('you must specify names or group');
}

return {url, query}
return {url, query: querystring.stringify(query)}
}

export default { call };

0 comments on commit 3a93180

Please sign in to comment.