Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic
 
PerlMonks  

Re: code snippet

by Thelonius (Priest)
on Jun 17, 2008 at 19:34 UTC ( #692577=note: print w/replies, xml ) Need Help??


in reply to code snippet

The quoting may confuse an editor (it does confuse the perl.vim code).

First, and most important, you should use placeholders instead of inserting Perl variables directly into SQL. Your code will break, for example, if HTTP_USER_AGENT or HTTP_REFERER contain an apostrophe (both of these values are under user control). Serious security risks can occur if you ignore this issue. For info, google SQL injection.

As a minor matter, you can avoid quoting literal hash keys if they are made up of all word characters (i.e. match /^\w+$/). For example, you can say <code>$Session->{idSession}. I see you did that one place.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://692577]
help
Chatterbox?
LanX googling for ones own childish humour has advantages ... see Pray for Paris
[LanX]: Moon not Mars

How do I use this? | Other CB clients
Other Users?
Others avoiding work at the Monastery: (9)
As of 2017-03-27 15:32 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Should Pluto Get Its Planethood Back?



    Results (320 votes). Check out past polls.