Skip to content

Commit

Permalink
Use rfc2396 parser (#139)
Browse files Browse the repository at this point in the history
`URI::DEFAULT_PARSER.escape` is obsoleted at next stable release of URI. We should use `URI::RFC2396_PARSER.escape` instead of `URI::DEFAULT_PARSER`.

Follow-up to #138
  • Loading branch information
hsbt authored Aug 9, 2024
1 parent 60d8d81 commit 015b05d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion mustermann/lib/mustermann/ast/translator.rb
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ def decorator_for(node)

# @return [String] escaped character
# @!visibility private
def escape(char, parser: URI::DEFAULT_PARSER, escape: URI::RFC2396_Parser.new.regexp[:UNSAFE], also_escape: nil)
def escape(char, parser: URI::RFC2396_Parser.new, escape: URI::RFC2396_Parser.new.regexp[:UNSAFE], also_escape: nil)
escape = Regexp.union(also_escape, escape) if also_escape
char.to_s =~ escape ? parser.escape(char, Regexp.union(*escape)) : char
end
Expand Down
2 changes: 1 addition & 1 deletion mustermann/lib/mustermann/pattern.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ module Mustermann
# @abstract
class Pattern
include Mustermann
@@uri ||= URI::Parser.new
@@uri ||= URI::RFC2396_Parser.new

# List of supported options.
#
Expand Down

0 comments on commit 015b05d

Please sign in to comment.