hawker2 — 2010-02-10T03:29:56-05:00 — #1
I have a registration website that collects name, email, address etc from a form. The php code that processes the form validates the inputs and creates sessions for the variables so I can use them on a confirmation page once the form is submitted.
I have used a shell script .. eg exec ("/usr/local/bin/php emailsend.php >/dev/null &");
on the confirmation page that runs a php file in the background once the confirmation page has loaded in the browser. This php file sends a "thank you for registering" email as well as sending a copy to us. (It runs in the background because it has a delay built in before it sends the emails -if I put the code directly onto the confirmation page then that page would not be able to load fully until the delay was finished) I want to personalise the thank you email with the persons name. However for some reason the session variables that are created by the form submission won't print on the thank you email. It keeps coming up blank where the persons name should be. Is there a reason why session variables that I can access and write to any other page on the site won't show up on a page that hasn't been loaded into a browser but simply runs in the background and sends out emails?
I have posted the code for the emailsend.php page below with the name greeting in red. Can anyone let me know whether I'm coding this wrong in the way I'm trying to access the username session variable. Any help would be appreciated.
$email = $REQUEST['accesslogin'] ;
//define the receiver of the email
$to = "<$email>";
$to2 = "email@example.com";
//define the subject of the email
$subject = 'Welcome to our website';
//create a boundary string. It must be unique
//so we use the MD5 algorithm to generate a random hash
$random_hash = md5(date('r', time()));
//define the headers we want passed. Note that they are separated with \r\
$headers = "From: reply@emailaddress\r\
//add boundary string and mime type specification
$headers .= "\r\
Content-Type: multipart/alternative; boundary=\"PHP-alt-".$random_hash."\"";
//define the body of the message.
ob_start(); //Turn on output buffering
--PHP-alt-<?php echo $random_hash; ?>
Content-Type: text/html; charset="iso-8859-1"
$message = '
<font face="arial" color="black" size="2">
<table cellSpacing="0" cellPadding="0" width="605" align="center" border="1">
<td style="WHITE-SPACE: nowrap" vAlign="top" width="684">
<div style="MARGIN-BOTTOM: 0px" align="center">
<a href="http://www.ourwebsite.com" target="_blank">
<img alt hspace="0" src="http://www.ourwebsite/image.jpg" border="0" width="664" height="385"></font></a>
<table cellSpacing="8" cellPadding="8" width="678">
<td vAlign="top" width="697">
<p style="margin-top: 0; margin-bottom: 0" align="left">
<font color="#000000" size="4" face="Arial Narrow">Dear <?php echo $_SESSION['username']; ?> ,</font>
<font face="Arial Narrow" size="4"><font color="#000000">Welcome
to our website, </font>
--PHP-alt-<?php echo $random_hash; ?>--
//copy current buffer contents into $message variable and delete current output buffer
$message = ob_get_clean();
//send the email
$mail_sent = @mail( $to, $subject, $message, $headers );
$mail_sent = @mail( $to2, $subject, $message, $headers );
system — 2010-02-10T03:35:59-05:00 — #2
You have to pass session id via command line and then set it manually
But it is most complicated feedback form design I've ever seen in my whole life.
raju_gautam — 2010-02-10T03:52:34-05:00 — #3
What does that emailsend.php file do actually? Why do you need to run that script in the background in the confirmation page. Quite strange! Can we try to avoid the use of exec? If you say the purpose of that, someone can present you a solution so that you don't need to use exec.
system — 2010-02-10T03:57:13-05:00 — #4
He's explained that
hash — 2010-02-10T04:24:37-05:00 — #5
If it has a delay built in, then why not just use cron?
system — 2010-02-10T04:29:09-05:00 — #6
cron will make things even more complicated
hawker2 — 2010-02-10T05:21:20-05:00 — #7
Thanks for the help guys. It's not my preference for it to be like this but it's not just a simple feedback form. It has to do a whole bunch of things all at once that normally shouldn't go together...but that's the brief I've been given so I'm going to piece it together roughly and get it working and then simplify it down the track.
hash — 2010-02-10T05:40:07-05:00 — #8
Not to my small mind
Perhaps I'm not getting it, but OP wants to do a bunch of stuff with a delay before sending emails but not delay user page loads? Log it to db and run cron??
system — 2010-02-10T05:46:38-05:00 — #9
Why to involve database into process, when we need to pass just a few variables to the script, which already runs at background?
Even sessions not needed here, as we can pass variables itself using command line parameters
hash — 2010-02-10T06:03:04-05:00 — #10
shrug perhaps the script dies before txn complete.
raju_gautam — 2010-02-10T06:31:36-05:00 — #11
Actually what I mean is, what are the things that are to be done before/after that delay? If they are manageable with some other way then we all can think of it.
So you also don't know now that what are the tasks/things to be done before that delay or with that delay in that script? Are those things to be done at the time of this email sending or they can be done periodically with some schedules as has suggested before? I am quite confused what the delay is for. Did OP mention about it?
hawker2 — 2010-02-10T07:34:31-05:00 — #12
OK, using the command line trick has worked..trial and error produced the following..
exec('/usr/local/bin/php emailsend.php '.$_SESSION['username'].' >/dev/null &');
Question: Is it possible to add another variable to that line, say $SESSION['email'] as well as $SESSION['username'].
hawker2 — 2010-02-10T14:46:28-05:00 — #13
Solved! The answer is exec ('/usr/local/bin/php emailsend.php ' . $SESSION['username'] .' '. $SESSION['email'] . ' >/dev/null &');
system — 2010-02-10T14:56:18-05:00 — #14
i'd suggest to encode variables somehow.
what if name would consist of 2 words? or even of something like |rm -rf /?
base64 looks good enough