Merge pull request #435 from CartoDB/change-jobs-wip-endpoint

Change WIP jobs endpoint
This commit is contained in:
Daniel 2017-09-20 18:58:56 +02:00 committed by GitHub
commit 2d75906d00
4 changed files with 6 additions and 5 deletions

View File

@ -1,3 +1,4 @@
dist: precise
addons: addons:
apt: apt:
sources: sources:

View File

@ -3,6 +3,9 @@
## 1.47.2 ## 1.47.2
Released 2017-mm-dd Released 2017-mm-dd
Announcements:
* Change work in progress jobs endpoint from `[..]/job/wip` to `[..]/jobs-wip`
## 1.47.1 ## 1.47.1
Released 2017-08-13 Released 2017-08-13

View File

@ -49,8 +49,7 @@ JobController.prototype.route = function (app) {
this.createJob.bind(this) this.createJob.bind(this)
); );
app.get( app.get(
global.settings.base_url + '/sql/job/wip', global.settings.base_url + '/jobs-wip',
userMiddleware, authenticatedMiddleware(this.userDatabaseService),
this.listWorkInProgressJobs.bind(this) this.listWorkInProgressJobs.bind(this)
); );
app.get( app.get(
@ -96,8 +95,6 @@ JobController.prototype.listWorkInProgressJobs = function (req, res) {
return handleException(err, res); return handleException(err, res);
} }
res.header('X-Served-By-DB-Host', req.context.userDatabase.host);
req.profiler.done('list'); req.profiler.done('list');
req.profiler.end(); req.profiler.end();
req.profiler.sendStats(); req.profiler.sendStats();

View File

@ -109,7 +109,7 @@ BatchTestClient.prototype.getWorkInProgressJobs = function(override, callback) {
assert.response( assert.response(
this.server, this.server,
{ {
url: this.getUrl(override, 'wip'), url: '/api/v1/jobs-wip',
headers: { headers: {
host: this.getHost(override) host: this.getHost(override)
}, },