@anirudhvr I’ll take a look, but I’m pretty busy with another project ATM. For now, I’d suggest (at a minimum) changing
mail(‘[email protected]’, ‘subjectir’ , $body);
…to something like
mail(‘[email protected]’, ‘subjectir’ , $body, “From: [email protected]”);
The `From` header is required, and not all servers add it automatically.
It would also be a good idea to add some error-handling checks. Things like
$this = $_POST;
…should always be
$this = empty( $_POST )? ”: $_POST;
which both avoids a warning from PHP, and allows you to provide a default value. Likewise,
mail( … );
echo “success”;
should be structured like
if( mail( … ) ){ echo “success”; }
else{ echo “failure”; }
You might consider adding a way for users to pass configuration options to your script without making them edit the script itself.
