diff options
| author | mike bayer <mike_mp@zzzcomputing.com> | 2021-05-24 18:16:01 +0000 |
|---|---|---|
| committer | Gerrit Code Review <gerrit@ci3.zzzcomputing.com> | 2021-05-24 18:16:01 +0000 |
| commit | a344538e6128ac798c5fde2202ae895fcd1329c0 (patch) | |
| tree | 247797f729a6547eab3cb40ee236a85d865b9634 /lib/sqlalchemy | |
| parent | bba5aa9adeb10df2953f1dc7b519f8b2e4e8a084 (diff) | |
| parent | 2ab67552a097522a82fd250b8b36995a9bc2cd80 (diff) | |
| download | sqlalchemy-a344538e6128ac798c5fde2202ae895fcd1329c0.tar.gz | |
Merge "URL parsing fixes"
Diffstat (limited to 'lib/sqlalchemy')
| -rw-r--r-- | lib/sqlalchemy/engine/url.py | 39 |
1 files changed, 17 insertions, 22 deletions
diff --git a/lib/sqlalchemy/engine/url.py b/lib/sqlalchemy/engine/url.py index 7898f9121..f60b9d56d 100644 --- a/lib/sqlalchemy/engine/url.py +++ b/lib/sqlalchemy/engine/url.py @@ -702,16 +702,17 @@ def _parse_rfc1738_args(name): (?P<name>[\w\+]+):// (?: (?P<username>[^:/]*) - (?::(?P<password>.*))? + (?::(?P<password>[^@]*))? @)? (?: (?: - \[(?P<ipv6host>[^/]+)\] | - (?P<ipv4host>[^/:]+) + \[(?P<ipv6host>[^/\?]+)\] | + (?P<ipv4host>[^/:\?]+) )? - (?::(?P<port>[^/]*))? + (?::(?P<port>[^/\?]*))? )? - (?:/(?P<database>.*))? + (?:/(?P<database>[^\?]*))? + (?:\?(?P<query>.*))? """, re.X, ) @@ -719,23 +720,17 @@ def _parse_rfc1738_args(name): m = pattern.match(name) if m is not None: components = m.groupdict() - if components["database"] is not None: - tokens = components["database"].split("?", 2) - components["database"] = tokens[0] - - if len(tokens) > 1: - query = {} - - for key, value in util.parse_qsl(tokens[1]): - if util.py2k: - key = key.encode("ascii") - if key in query: - query[key] = util.to_list(query[key]) - query[key].append(value) - else: - query[key] = value - else: - query = None + if components["query"] is not None: + query = {} + + for key, value in util.parse_qsl(components["query"]): + if util.py2k: + key = key.encode("ascii") + if key in query: + query[key] = util.to_list(query[key]) + query[key].append(value) + else: + query[key] = value else: query = None components["query"] = query |