diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-05-15 12:01:07 +0200 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-05-15 12:01:07 +0200 |
commit | bbba6d7e62fb9d8bca635d57604fd503fb3c4645 (patch) | |
tree | 75e6d810370791ba991d007ab67b0ca265ab7ec6 /lib/gitlab/untrusted_regexp.rb | |
parent | f16f2b599412ed1514ba96d81758b9a2e6fd9c1f (diff) | |
parent | a78b1b27b86d34c00e1b0631e967d637f8a6714b (diff) | |
download | gitlab-ce-bbba6d7e62fb9d8bca635d57604fd503fb3c4645.tar.gz |
Merge branch 'master' into feature/gb/add-regexp-variables-expression
* master: (76 commits)
Conflicts:
lib/gitlab/untrusted_regexp.rb
Diffstat (limited to 'lib/gitlab/untrusted_regexp.rb')
-rw-r--r-- | lib/gitlab/untrusted_regexp.rb | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/lib/gitlab/untrusted_regexp.rb b/lib/gitlab/untrusted_regexp.rb index fb25755391d..ce1cf737663 100644 --- a/lib/gitlab/untrusted_regexp.rb +++ b/lib/gitlab/untrusted_regexp.rb @@ -11,7 +11,11 @@ module Gitlab class UntrustedRegexp delegate :===, :source, to: :regexp - def initialize(pattern) + def initialize(pattern, multiline: false) + if multiline + pattern = "(?m)#{pattern}" + end + @regexp = RE2::Regexp.new(pattern, log_errors: false) raise RegexpError.new(regexp.error) unless regexp.ok? @@ -35,6 +39,19 @@ module Gitlab self.source == other.source end + # Handles regular expressions with the preferred RE2 library where possible + # via UntustedRegex. Falls back to Ruby's built-in regular expression library + # when the syntax would be invalid in RE2. + # + # One difference between these is `(?m)` multi-line mode. Ruby regex enables + # this by default, but also handles `^` and `$` differently. + # See: https://www.regular-expressions.info/modifiers.html + def self.with_fallback(pattern, multiline: false) + UntrustedRegexp.new(pattern, multiline: multiline) + rescue RegexpError + Regexp.new(pattern) + end + private attr_reader :regexp |