Hi Andy
I stopped reading after the first 9 lines.
use strict;
use warnings;
And if you're going to be dealing with user input, remember to turn on Taint checking with the T switch.
Oh, and never, ever, ever, leave use CGI::Carp qw(fatalsToBrowser); in a script that's in a production environment. Never. If anything goes wrong with your script, you're just going to outputting tons of useful information to whatever wants to hack you.
You should only output this kind of information if a DEBUG flag has been set from within your code.
Cheers
- wil
I stopped reading after the first 9 lines.
use strict;
use warnings;
And if you're going to be dealing with user input, remember to turn on Taint checking with the T switch.
Oh, and never, ever, ever, leave use CGI::Carp qw(fatalsToBrowser); in a script that's in a production environment. Never. If anything goes wrong with your script, you're just going to outputting tons of useful information to whatever wants to hack you.
You should only output this kind of information if a DEBUG flag has been set from within your code.
Cheers
- wil