ループは1つの電子メールのみをチェックします

Question

ユーザーのデータベースから電子メールアドレスを取得します。次に、これらのメールアドレスをapiで確認し、違反件数を返します。

問題は、最初のメールアドレスを確認するようになっているように見えることです。私たちが複数ある場合は、最初は、侵害の正確な数を取得し、第二は、あなたのループで0

<?php 
ini_set("allow_url_fopen", 1); 
include '/home/actiorwd/include/dbinfo.php'; 

$servername = "localhost"; 
$dbname = "actiorwd_websec"; 

// Create connection 
$conn = new mysqli($servername, $user, $passw, $dbname); 
// Check connection 
if ($conn->connect_error) { 
    die("Connection failed: " . $conn->connect_error); 
} 
//users contains two columns; email and breaches. 
$sql = "SELECT * FROM users"; 
$result = $conn->query($sql); 

$ch = curl_init(); 

if ($result->num_rows > 0) { 
    // output data of each row 

    while($row = $result->fetch_assoc()) { 
     $url = 'https://haveibeenpwned.com/api/v2/breachedaccount/'; 
     $url.= $row["email"]; 

     curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false); 
     curl_setopt($ch, CURLOPT_RETURNTRANSFER, true); 
     curl_setopt($ch,CURLOPT_USERAGENT,'Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.13) Gecko/20080311 Firefox/2.0.0.13'); 
     curl_setopt($ch, CURLOPT_URL, $url); 
     $results = curl_exec($ch); 
     curl_close($ch); 
     $obj = json_decode($results, TRUE); 

     $usermail = $row["email"]; 



     $newbreaches = count($obj); 

     //run if new breaches are found 
     if($row["breaches"]!==$newbreaches){ 

      require_once('/home/actiorwd/public_html/PHPMailer/PHPMailerAutoload.php'); 
      $mail = new PHPMailer; 

      $mail->isSMTP();          // Set mailer to use SMTP 
      $mail->Host = 'cpanel40.proisp.no'; // Specify main and backup SMTP servers 
      $mail->SMTPAuth = true;        // Enable SMTP authentication 
      $mail->Username = 'breached@actionscript.no';     // SMTP username 
      $mail->Password = 'PASSWORDHERE';       // SMTP password 
      $mail->SMTPSecure = 'ssl';       // Enable TLS encryption, `ssl` also accepted 
      $mail->Port = 465;         // TCP port to connect to 

      $mail->setFrom('breached@actionscript.no', 'WebSec'); 
      $mail->addAddress($row["email"]);  // Add a recipient 
      $mail->addReplyTo('breached@actionscript.no', 'WebSec'); 

      $mail->isHTML(true);         // Set email format to HTML 

      $mail->Subject = 'Breached!!!!'; 
      $mail->Body = 'Someone breached your account, there are: '.$newbreaches."breaches"; 
      $mail->AltBody = 'Someone breached your account in plain text'; 
      $mail->send(); 
      /* 
      if(!$mail->send()) { 
       echo 'Message could not be sent.'; 
       echo 'Mailer Error: ' . $mail->ErrorInfo; 
      } else { 
       echo 'Message has been sent'; 
      } 

      */ 


      $updatebreach = "INSERT INTO users (`email`, `breaches`) VALUES ('$usermail', '$newbreaches') ON DUPLICATE KEY UPDATE `breaches` = '$newbreaches' "; 
      //echo $updatebreach; 

      if ($conn->query($updatebreach) === TRUE) { 
       //echo "New record created successfully"; 
      } else { 
       //echo "Error: " . $updatebreach . "<br>" . $conn->error; 
      } 


     } 


    } 
} else { 
    echo "0 results"; 
} 
$conn->close(); 
?> 

Answer 1

ルックを取得したり、ループの前の行で、より良いです。あなたはしています：

$ch = curl_init(); - あなたはcURLセッションを初期化します。これは、ループの外側から1回だけ実行します。

はその後、数行後に、youreのはやって：

curl_close($ch);を - あなたはcURLのセッションを閉じます。毎回の呼び出しの後でこれを行うのは、あなたのループの中にあるからです。だからあなたはcURLセッションを閉じて新しいセッションを開かないので、主にあなたは初期化し、cURLを実行します（それが最初のもので動作する理由）。

ループからcurl_close$ch);を取り出してファイルの最後に置くだけで、同じセッションですべての要求を実行することができます。そして、それが動作するはずです:)

のファイルのあなたの終わりは次のようになります。

} 
curl_close($ch); 
$conn->close(); 
?> 

それが助け願っています。