[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Omaha.pm] Database -> XML



On Apr 16, 2007, at 1:20 AM, George Neill wrote:
Does Omni::DB take care of cleansing $prop from injections and such?

No, it doesn't. But that's an excellent concern to raise.

$prop (aka property aka hotel) is just hard-coded in my program source code for now. Later it will be in a loop controlled by the database (our "hotels" table has one row per hotel and prop is a key).

If I was ever going to accept $prop as input from the outside I'd (1) sanitize it with a regex, (2) make sure it was in a DBI placeholder, not just running loose in an SQL string, and (3) use Perl's taint mode.

... err: confession: I never use Perl's taint mode. But at least I'm aware that I probably should. Does that count? :)

I think your code below is simple enough and would be easy to maintain
for a handful of tables.

Ya. I like it. Obviously it gets uglier as the table joins get uglier and/or the mapping needs to be nested/regrouped/etc, but I think the final result is as pretty as its going to be in any language. :)

j