]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commit
Fix dependency_for final argument
authorJoe Urciuoli <joseph.w.urciuoli@gmail.com>
Wed, 19 Sep 2018 17:40:23 +0000 (13:40 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 1 Oct 2018 20:58:13 +0000 (16:58 -0400)
commiteb1f5eb674ade28fdce505a2b35e18c0304ff64e
tree3d888385305f46ff06566d74705f84e317b810d4
parent16c48847ca608a415a6adb78215f2d13d28932f4
Fix dependency_for final argument

Fixed issue where part of the utility language helper internals was passing
the wrong kind of argument to the Python ``__import__`` builtin as the list
of modules to be imported.  The issue produced no symptoms within the core
library but could cause issues with external applications that redefine the
``__import__`` builtin or otherwise instrument it. Pull request courtesy Joe
Urciuoli.

Per the submitter: "The fourth argument provided to `__import__`  (which
`import_` feeds in to) is supposed to be a a list of strings, but this code is
passing a single string. This was causing the sqlalchemy `import_` function to
break the string (for example 'interfaces') into an array of single characters
['i', 'n', ...], which causes the actual `__import__` to not find the module
`sqlalchemy.orm.i` (since it's trying to import `sqlalchemy.orm.i` and
`sqlalchemy.orm.n` .. etc)"

No issue could be reproduced locally as it seems you can put anything non-
empty/None into that last argument, even a list like ``['X']``, and  all the
sub-modules seem to appear.  Omit it, and then the sub-modules aren't present.
Perhaps it just runs the module or not if this attribute is present.

Change-Id: Ia15c74620f24d24f0df4882f9b36a04e2c3725b8
Pull-request: https://github.com/zzzeek/sqlalchemy/pull/473
(cherry picked from commit cacc3c2057ab14faaf6c5b679bb2dbb6f8b98f8e)
doc/build/changelog/unreleased_12/pr473.rst [new file with mode: 0644]
lib/sqlalchemy/util/langhelpers.py