Regex crash with docs search

Zackster · April 12, 2018, 11:54am

the docs search http://docs.lucee.org currently crashes if you search for a function like Min()

can anyone suggest a more robust approach?

lucee/lucee-docs/blob/master/builders/html/assets/js/src/uberCoolLuceeDocsSearch.js#L105


      
          		};
          		fulltextitem.display = fulltextitem.text;
          		matches.unshift( fulltextitem );
          
          
		return matches;
          	};
          
          
	generateRegexForInput = function( input ){
          		var inputLetters = input.replace(/[\W]+/g, '').split('')
          		,  reg = {}, i;
          
          
        if( inputLetters.length < 14 ){
          		
                      reg.expr = new RegExp('(' + inputLetters.join( ')(.*?)(' ) + ')', 'i');
                      reg.replace = "";
                      
                      for( i=0; i < inputLetters.length; i++ ) {
                          reg.replace += ( '<b>$' + (i*2+1) + '</b>' );
                          if ( i + 1 < inputLetters.length ) {
                              reg.replace += '$' + (i*2+2);
                          }

martin · April 13, 2018, 9:19am

Hi @Zackster

I have created a pull request that should fix this problem.

Zackster · April 14, 2018, 4:45am

thanks @martin, this is now live