chore: drop comment and issue refrences, drop sparse array test

Author: jonchurch

lib/utils.js

@@ -288,11 +288,6 @@ function createETagGenerator (options) {
 function parseExtendedQueryString(str) {
   return qs.parse(str, {
     allowPrototypes: true,
-    // qs >= 6.14 enforces a default arrayLimit of 20, which collapses larger
-    // repeated-key query strings into objects. Raise the limit to 1000 to
-    // restore the pre-6.14 behavior for common payloads while still bounding
-    // single-parameter array expansions like `arr[999999]=x`. Matches the
-    // parameter limit used elsewhere in the query parser. See #7147.
     arrayLimit: 1000
   });
 }

test/req.query.js

@@ -39,7 +39,7 @@ describe('req', function(){
         .expect(200, '{"user.name":"tj"}', done);
       });
 
-      it('should parse more than 20 repeated values as an array (#7147)', function (done) {
+      it('should parse more than 20 repeated values as an array', function (done) {
         var app = createApp('extended');
         var ids = [];
         var expected = [];
@@ -52,16 +52,6 @@ describe('req', function(){
         .get('/?' + ids.join('&'))
         .expect(200, JSON.stringify({ ids: expected }), done);
       });
-
-      it('should still reject array expansion beyond arrayLimit (#7147)', function (done) {
-        var app = createApp('extended');
-
-        // A single parameter with an index past arrayLimit (1000) must not
-        // allocate a sparse array; qs collapses it to an object.
-        request(app)
-        .get('/?arr[9999]=x')
-        .expect(200, '{"arr":{"9999":"x"}}', done);
-      });
     });
 
     describe('when "query parser" is simple', function () {