Add reverse proxy test in travis

Chocobozzz · Chocobozzz · commit 09becad8e85c · 2018-03-29T11:39:53.000+02:00
diff --git a/client/src/app/core/auth/auth.service.ts b/client/src/app/core/auth/auth.service.ts
@@ -70,7 +70,8 @@ export class AuthService {
 
                  if (error.status === 403) {
                    errorMessage = `Cannot retrieve OAuth Client credentials: ${error.text}. \n`
-                   errorMessage += 'Ensure you have correctly configured PeerTube (config/ directory), in particular the "webserver" section.'
+                   errorMessage += 'Ensure you have correctly configured PeerTube (config/ directory), ' +
+                     'in particular the "webserver" section.'
                  }
 
                  // We put a bigger timeout
diff --git a/server/initializers/constants.ts b/server/initializers/constants.ts
@@ -238,7 +238,7 @@ const CONSTRAINTS_FIELDS = {
 const RATES_LIMIT = {
   LOGIN: {
     WINDOW_MS: 5 * 60 * 1000, // 5 minutes
-    MAX: 10 // 10 attempts
+    MAX: 15 // 15 attempts
   }
 }
 
diff --git a/server/tests/api/index-fast.ts b/server/tests/api/index-fast.ts
@@ -11,3 +11,4 @@ import './videos/video-privacy'
 import './videos/services'
 import './server/email'
 import './server/config'
+import './server/reverse-proxy'
diff --git a/server/tests/api/server/reverse-proxy.ts b/server/tests/api/server/reverse-proxy.ts
@@ -64,7 +64,7 @@ describe('Test application behind a reverse proxy', function () {
   it('Should rate limit logins', async function () {
     const user = { username: 'root', password: 'fail' }
 
-    for (let i = 0; i < 9; i++) {
+    for (let i = 0; i < 14; i++) {
       await userLogin(server, user, 400)
     }
 

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,8 @@ export class AuthService {`
`70`	`70`
`71`	`71`	`if (error.status === 403) {`
`72`	`72`	errorMessage = `Cannot retrieve OAuth Client credentials: ${error.text}. \n`
`73`		`- errorMessage += 'Ensure you have correctly configured PeerTube (config/ directory), in particular the "webserver" section.'`
	`73`	`+ errorMessage += 'Ensure you have correctly configured PeerTube (config/ directory), ' +`
	`74`	`+ 'in particular the "webserver" section.'`
`74`	`75`	`}`
`75`	`76`
`76`	`77`	`// We put a bigger timeout`
Original file line number	Diff line number	Diff line change
`@@ -238,7 +238,7 @@ const CONSTRAINTS_FIELDS = {`
`238`	`238`	`const RATES_LIMIT = {`
`239`	`239`	`LOGIN: {`
`240`	`240`	`WINDOW_MS: 5 * 60 * 1000, // 5 minutes`
`241`		`- MAX: 10 // 10 attempts`
	`241`	`+ MAX: 15 // 15 attempts`
`242`	`242`	`}`
`243`	`243`	`}`
`244`	`244`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ describe('Test application behind a reverse proxy', function () {`
`64`	`64`	`it('Should rate limit logins', async function () {`
`65`	`65`	`const user = { username: 'root', password: 'fail' }`
`66`	`66`
`67`		`- for (let i = 0; i < 9; i++) {`
	`67`	`+ for (let i = 0; i < 14; i++) {`
`68`	`68`	`await userLogin(server, user, 400)`
`69`	`69`	`}`
`70`	`70`