Callbackapi vk, php, mysql: что не так с этим кодом?

Question

[Franco Mannino]

Взял из примера https://vk.com/dev/callback_api
И немного изменил - чтобы шла запись в базу данных.
В результате получаю на одно событие несколько записей в базу, причем они еще и идут вперемежку: после второго, третьего и т.д. может еще раз записаться предыдущее - как на картинке.

Что не так с кодом?

<?php
 
//var_dump($_REQUEST);

if (!isset($_REQUEST)) {
  return;
}

$txt = '';
$date = '';
	
//Строка для подтверждения адреса сервера из настроек Callback API
$confirmation_token = 'dkrijsakcnalfn';
 
//Ключ доступа сообщества
$token = 'trjhrtjrtjjrtjrjurtererherjherhe';
   
//Получаем и декодируем уведомление
$data = json_decode(file_get_contents('php://input'));

//Проверяем, что находится в поле "type"
switch ($data->type) {
  //Если это уведомление для подтверждения адреса сервера...
  case 'confirmation':
    //...отправляем строку для подтверждения адреса
    echo $confirmation_token;
    break;

//Если это уведомление о новом сообщении...
  case 'wall_post_new':
    //...получаем
    $txt = $data->object->text;
    $date = $data->object->date;


//Возвращаем "ok" серверу Callback API
    echo('ok');

break;
}	
	require('bd.php');

	$host = "localhost";
	$user = "user";
	$password = "hdrgeegse";
	$datebase = "dbname";
	updateDate ($host, $user, $password, $datebase, $txt, $date);
?>

<?php

	function updateDate ($host, $user, $password, $datebase, $txt, $date) {

	$link = mysqli_connect($host, $user, $password, $datebase) or die("Не могу соединиться с MySQL.");		
	$query = "INSERT INTO `text`(`id`, `text`, `date`) VALUES (NULL,$txt,$date)";	
	$result = mysqli_query ($link, $query)  or die("ERROR! " . mysqli_error($link)); 		
		if($result)
		{
			echo "<span style='color:blue;'>OK!</span>";
		}		
	// закрываем подключение
	mysqli_close($link);		
}

Comments

[Ramm]

Логи ВК запросов в студию (act=api_requests)

Answer 1

[Сережа Ахен]

Че? Может лучше не... продолжать?

А по делу - в выводе для callback vk не должно быть ничего, кроме "ok".
У вас же вывод "ok<span style='color:blue;'>OK!</span>", вк считает это ошибкой и продолжает слать уведомления в надежде, что Вы сможете его наконец обработать.

Зачем?!

if($result)
    {
      echo "<span style='color:blue;'>OK!</span>";
    }

Плюс ко всему - ЗАЧЕМ Вы производите запись в БД вне кейса? А что, если это запрос на валидацию сервера? Зачем тогда производить запись в БД?

Comments

[Franco Mannino]

вне кейса - для простоты и читабельности, а внутри кейса тоже помещал код - никакой разницы,
вот, все что можно убрал - один черт дублирует, и многократно:

<?php
 
if (!isset($_REQUEST)) {
  return;
}

$txt;
$date;
$confirmation_token = '9081a5bb';
$token = 'c0223f775444cf3d58a8a1442ec76a9';
$data = json_decode(file_get_contents('php://input'));

switch ($data->type) {
  case 'confirmation':
    echo $confirmation_token;
    break;

  case 'wall_post_new':
    $txt = $data->object->text;
	$date = $data->object->date;
    echo ('ok');
	break;
}

require_once('bd.php');
update ($txt, $date);

<?php

	function update ($txt, $date) {		
		$link = mysqli_connect("123.456.789.012", "u123456_call", "qwerty", "db123456_call") or die();	
		mysqli_set_charset($link, "utf8");
		$query = "INSERT INTO `text`(`id`, `text`, `date`) VALUES (NULL,\"" . $txt . "\"," . $date . ")";
		mysqli_query ($link, $query) or die(); 	
		mysqli_close($link);		
	}

[Сережа Ахен]

Franco Mannino: во первых require_once('bd.php'); сунь в начало файла, как принято. Во вторых update ($txt, $date); унеси в кейс обработки события wall_post_new.
В третьих - смотри в админке паблика вк исходящие запросы, раздел неудавшихся запросов.

Почти уверен что вк тут отрабатывает все неудавшиеся ранее запросы, либо у тебя акромя уведомления о новых постах включено еще куча различных уведомлений, которые ты даже не обрабатываешь, а запись в базу ведешь вне кейса.

[Franco Mannino]

Сережа Ахен:

блин, как выяснилось в итоге - причиной не-ОК ответов был пробравшийся в ютф БОМ.