You might recode this using WWW::Mechanize. Your code will be simpler, and you won't have to worry about missing a required form variable. It'll also be more robust in the event of the site changing their form. Might even be easier to debug.