github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #3776 from overleaf/ho-post-clsi-cookie

Browse source 
convert _populateServerIdViaRequest to a post request

GitOrigin-RevId: cbbf926708804527ba3717a66e9f0192ecf70fbd
This commit is contained in:
Timothée Alby 2021-03-29 11:19:42 +02:00 committed by Copybot
parent 94896bca4e
commit d346172819
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
services/web/app/src/Features/Compile/ClsiCookieManager.js
View file

@ -58,7 +58,7 @@ module.exports = function(backendGroup) {
callback = function(err, serverId) {}
}
const url = `${Settings.apis.clsi.url}/project/${project_id}/status`
return request.get(url, (err, res, body) => {
return request.post(url, (err, res, body) => {
if (err != null) {
OError.tag(err, 'error getting initial server id for project', {
project_id

4
services/web/test/unit/src/Compile/ClsiCookieManagerTests.js
View file

@ -30,7 +30,7 @@ describe('ClsiCookieManager', function() {
}
this.project_id = '123423431321'
this.request = {
get: sinon.stub(),
post: sinon.stub(),
cookie: realRequst.cookie,
jar: realRequst.jar,
defaults: () => {
@ -123,7 +123,7 @@ describe('ClsiCookieManager', function() {
describe('_populateServerIdViaRequest', function() {
beforeEach(function() {
this.response = 'some data'
this.request.get.callsArgWith(1, null, this.response)
this.request.post.callsArgWith(1, null, this.response)
return (this.ClsiCookieManager.setServerId = sinon
.stub()
.callsArgWith(2, null, 'clsi-9'))