pulibrary / allsearch_api / ddf6baed-4c41-4153-adc7-f101ad4b9561

100%

main: 100%

push

circleci

Skip nokogiri-based sanitization for strings that don't contain angle brackets

It can be expensive to run the full sanitization routine on long
strings, so let's skip it if they don't need HTML-parser based sanitization,
using the heuristic of checking for an opening angle bracket.

Also, use ruby's squeeze method to avoid an expensive regular expression.

According to the attached microbenchmark:

before:
```
Warming up --------------------------------------
           sanitizer   583.000 i/100ms
Calculating -------------------------------------
           sanitizer      6.308k (± 4.5%) i/s  (158.54 μs/i) -     31.482k in   5.001876s
```

after:
```
Warming up --------------------------------------
           sanitizer    26.392k i/100ms
Calculating -------------------------------------
           sanitizer    258.436k (± 2.7%) i/s    (3.87 μs/i) -      1.293M in   5.008248s
```

6 of 6 new or added lines in 1 file covered. (100.0%)

1427 of 1427 relevant lines covered (100.0%)

76.63 hits per line