Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 9760566

Browse files
authored
Merge pull request #120 from ready-research/master
Security fix for ReDoS
2 parents 5869ff4 + 9fa822a commit 9760566

12 files changed

+11
-37
lines changed

lib/parsers/api.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
var trim = require('../utils/trim');
2-
31
function parse(content) {
4-
content = trim(content);
2+
content = content.trim();
53

64
// Search: type, url and title
75
// Example: {get} /user/:id Get User by ID.

lib/parsers/api_define.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
var trim = require('../utils/trim');
21
var unindent = require('../utils/unindent');
32

43
var ParameterError = require('../errors/parameter_error');
@@ -15,7 +14,7 @@ var _messages = {
1514
function parse(content, source, messages) {
1615
messages = messages || _messages;
1716

18-
content = trim(content);
17+
content = content.trim();
1918

2019
var parseRegExp = /^(\w*)(.*?)(?:\s+|$)(.*)$/gm;
2120
var matches = parseRegExp.exec(content);

lib/parsers/api_deprecated.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
var trim = require('../utils/trim');
21
var unindent = require('../utils/unindent');
32

43
function parse(content) {
5-
var deprecated = trim(content);
4+
var deprecated = content.trim();
65

76
if (deprecated.length > 0) {
87
return {

lib/parsers/api_description.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
var trim = require('../utils/trim');
21
var unindent = require('../utils/unindent');
32

43
function parse(content) {
5-
var description = trim(content);
4+
var description = content.trim();
65

76
if (description.length === 0)
87
return null;

lib/parsers/api_example.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
var trim = require('../utils/trim');
21
var unindent = require('../utils/unindent');
32

43
function parse(content, source) {
5-
source = trim(source);
4+
source = source.trim();
65

76
var title = '';
87
var text = '';

lib/parsers/api_group.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
var trim = require('../utils/trim');
2-
31
function parse(content) {
4-
var group = trim(content);
2+
var group = content.trim();
53

64
if (group.length === 0)
75
return null;

lib/parsers/api_name.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
var trim = require('../utils/trim');
2-
31
function parse(content) {
4-
var name = trim(content);
2+
var name = content.trim();
53

64
if(name.length === 0)
75
return null;

lib/parsers/api_param.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
var trim = require('../utils/trim');
21
var unindent = require('../utils/unindent');
32

43
var group = '';
@@ -69,7 +68,7 @@ var allowedValuesWithQuoteRegExp = new RegExp(/\'[^\']*[^\']\'/g);
6968
var allowedValuesRegExp = new RegExp(/[^,\s]+/g);
7069

7170
function parse(content, source, defaultGroup) {
72-
content = trim(content);
71+
content = content.trim();
7372

7473
// replace Linebreak with Unicode
7574
content = content.replace(/\n/g, '\uffff');

lib/parsers/api_sample_request.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
var trim = require('../utils/trim');
2-
31
function parse(content) {
4-
var url = trim(content);
2+
var url = content.trim();
53

64
if(url.length === 0)
75
return null;

lib/parsers/api_use.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
var trim = require('../utils/trim');
2-
31
function parse(content) {
4-
var name = trim(content);
2+
var name = content.trim();
53

64
if (name.length === 0)
75
return null;

lib/parsers/api_version.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
11
var semver = require('semver');
22

3-
var trim = require('../utils/trim');
4-
53
var ParameterError = require('../errors/parameter_error');
64

75
function parse(content) {
8-
content = trim(content);
6+
content = content.trim();
97

108
if (content.length === 0)
119
return null;

lib/utils/trim.js

-9
This file was deleted.

0 commit comments

Comments
 (0)