Почему не работает кодирование и декодирование в Base64?

Question

[sochi-russia]

Файл

direct.php

<?php if($_GET['url'])
{
$url = base64_decode($_GET['url']);
if (!$url)
{
$url = "https://вашдомен.ру";
}
if (!preg_match('#(https?|ftp)://\S+[^\s.,>)\];\'\"!?]#i',$url))
{
exit ("<p>Неверный формат запроса! Проверьте URL!</p>");
}
header("Location:$url");
exit();
} ?>

На главной

<?php
$mytext=preg_replace('~(<a(?>.*?href))=(["\'])([a-z0-9]++://(?![a-z0-9\.]*?site\.ru).*?)\2~eSi', '"\1=\"https://site.ru/direct.php?url=".base64_encode("$3")."\" target=\"_blank\"";', $mytext);?>

site.ru и вашдомен.ру

- заменить на свой домен
Суть такова что заменитель, подставляет в переменную внутреннего адреса внешний адрес, и дополнительно кодирует её в

base64

.
Но код не срабатывает почему? Абсолютно ничего он делает на сайте

Comments

[mayton2019]

У тебя в коде стоит 2 условных оператора if.

Следовательно у тебя есть 4 кейса. Комбинации из двух условий.

В случае когда у тебя не срабатывает - по какой линии мы идем?

[sochi-russia]

mayton2019, в случае не срабатывания не производим замену

[sochi-russia]

mayton2019, по факту сайт

avia-avia._ru

на главной есть ссылки на поддомен (внешняя) такого типа

https://bilety.avia-avia._ru/searches/new?origin_i...

Вот нужно с внешней и работать.

[mayton2019]

У тебя $_GET['url'] что возвращает? Возможно ты спутал base_encode и base_decode.

[sochi-russia]

mayton2019, попробовал не срабатывает

[AUser0]

sochi-russia, что, не можете проверить содержимое переменной $_GET['url']?

[sochi-russia]

AUser0, нет не знаю как

[Сергей delphinpro]

sochi-russia, Оно отображается в адресной строке браузера =))

[sochi-russia]

mayton2019, https://bilety.avia-avia.ru возвращает

Answer 1

[ThunderCat]

Смотреть, дебажить, думать...

<?php 
if($_GET['url']){
    $url = base64_decode($_GET['url']);
    var_dump($_GET);
    if (!$url){
        $url = "https://вашдомен.ру";
    }
    var_dump($url);
    if (!preg_match('#(https?|ftp)://\S+[^\s.,>)\];\'\"!?]#i',$url)){
        exit ("<p>Неверный формат запроса! Проверьте URL!</p>");
    }
    //header("Location:$url");
    exit();
} ?>

Comments

[sochi-russia]

Вот тестовая страница https://avia-avia.ru/intest.php
Не сработала (таблица в самом низу)

[sochi-russia]

Добавил header("Location:$url");

[AUser0]

sochi-russia, хммм, и где в ссылке

https://bilety.avia-avia.ru/searches/new?origin_iata=MOW&destination_iata=ARH&depart_date=2023-12-14&return_date=2023-12-17&adults=1&children=0&infants=0&trip_class=0&marker=87111&with_request=true

хоть что-то похожее на код Base64?

[sochi-russia]

AUser0, в том то и дело что не идет шифрование

[ThunderCat]

Суть такова что заменитель, подставляет в переменную внутреннего адреса внешний адрес, и дополнительно кодирует её в

$mytext=preg_replace(
'~(<a(?>.*?href))=(["\'])([a-z0-9]++://(?![a-z0-9\.]*?site\.ru).*?)\2~eSi', 
'"\1=\"https://site.ru/direct.php?url=".base64_encode("$3")."\" target=\"_blank\"";', /* 
очевидно что внутри этой строки base64_encode() работать не будет, 
так как даже по подсветке кода видно, что строка там не конкатенируется, 
а именно просто часть строки. Достаточно просто соблюсти правила разметки 
кода, и немного внимательнее глянуть на код. Да и сама регулярка выглядит сомнительно...*/
$mytext
);?>

Вы вообще проверяли вашу регулярку? Как минимум два неэкранированных // точно мимо и ++ явно излишние... и вообще, зачем вы там ищете а хреф как часть паттерна? Вам же тупо нужно заменить один урл другим?

[sochi-russia]

ThunderCat, не direct.php, а engine.php в коде ошибка здесь на сайте engine.php стоит. Проверял даже сейчас не работает.

[sochi-russia]

ThunderCat, сайт рухнул из-за обновления алгоритма Яндекса где стало учитываться кол-во внешних ссылок со страницы сайта. Поэтому обратился сюда.

[ThunderCat]

sochi-russia,

не direct.php, а engine.php в коде ошибка здесь на сайте engine.php стоит. Проверял даже сейчас не работает.

Что? Ничего не понятно... Кто куда и зачем стоит...

сайт рухнул из-за обновления алгоритма Яндекса где стало учитываться кол-во внешних ссылок со страницы сайта. Поэтому обратился сюда.

Я про это вообще не спрашивал.

Еще раз - регулярка неверная, строка замены тоже неправильная. Вы вообще читаете ответы?

[sochi-russia]

ThunderCat, Вы можете помочь подправить код?

[sochi-russia]

ThunderCat, мне нужно, чтобы ссылки вида

https://bilety.avia-avia.ru/searches/new?origin_ia...

были такого вида
https://bilety.avia-avia.ru/direct.php?url=aHR0cHM...
и при клике осуществлялся переход на реальный адрес
Как организованно вот здесь

вот на этом сайте
(см. Скрин жёлтые ссылки 'найти'), в ссылке (убрать нижнее подчеркивание)
https_://uniticket._ru/country/crimea/

[ThunderCat]

sochi-russia,

Вы можете помочь подправить код?

могу конечно. Хотя непонятно зачем было обращаться сюда, если вы не разбираетесь в коде вообще? Так то все необходимые для исправления ошибки я вам уже нашел, чините и вперед... Если вы ищете готовое решение, логичнее было бы обратиться на фриланс, а не нарушать правила данного ресурса (п. 5.12)

[sochi-russia]

ThunderCat, Знаете, а Вы правы, не стоило мне беспокоить профессионалов. С уважением.

[ThunderCat]

sochi-russia,

мне нужно, чтобы ссылки вида...

Я уже догадался, хотя описание задачи так себе...

В принципе, код идеологически верный, но реализован криво. Упростите себе задачу, вместо preg_replace возьмите preg_match() и ищите только урл, вся остальная фигня вам не нужна (да и ваш preg_replace полностью кривой). Получите урл, его закодируйте в бэйс64, и уже эту строку просто вставьте в

<a href="https://bilety.avia-avia.ru/direct.php?url=<?=$base64URL?>" target="_blank">

[ThunderCat]

sochi-russia,

Знаете, а Вы правы, не стоило мне беспокоить профессионалов. С уважением.

Правила ресурса требуют от вас постановки задачи в форме "что не так" и ожидания ответа в форме "делай это и это", а не готовый код, что четко прописано в правилах. Профессионалы тут поддерживают знаниями (цитата) "сервис вопросов и ответов, предназначенный для IT-специалистов и всех тех, кто отчаянно пытается ими стать.". Если вы не относитесь ни к первым, ни ко вторым, то какой смысл искать разъяснения по коду, который вы не хотите исправлять, желая лишь получить готовое решение?

[sochi-russia]

ThunderCat, Я не нашел в правилах, того, о чем Вы пишите. Я думаю правильный, точный ответ на вопрос поможет многим, столкнувшимся с такой задачей.

[ThunderCat]

sochi-russia, 1) Еще раз - все что нужно для самостоятельного решения вашей задачи я описал: нужно поправить регулярное выражение и замену, все остальное в порядке. Можно для упрощения задачи
а) использовать preg_match() для получения урл,
б) полученную строку закодировать
с) вставить полученный результат в хтмл шаблон указанным выше способом.

Я не нашел в правилах, того, о чем Вы пишите.

2) Правила:

5.12. Задачи и задания. В отличие от вопроса, задача и задание представляют собой частную проблемную ситуацию с явно заданной целью, которую необходимо достичь. Пользу от достижения этой цели получает, как правило, лишь её автор. И даже автору решение задачи или задания будет полезно лишь кратковременно (до тех пор, пока он не использует полученное решение).

Именно по этому я не пишу готовый код, а даю вам максимально четкое описание алгоритма решения, прям только переложить фразы в код. Возможно у вас есть какие-то конкретные вопросы, из-за которых вы не можете что-то сделать самостоятельно? Задавайте, и я расскажу как сделать.

[sochi-russia]

ThunderCat,
не получается

<?php if($_GET['url'])
{
$url = base64_decode($_GET['url']);
if (!$url)
{
$url = "https://avia-avia.ru";
}
if (!preg_match('#(https?|ftp)://\S+[^\s.,>)\];\'\"!?]#i',$url))
{
exit ("<p>Неверный формат запроса! Проверьте URL!</p>");
}
header("Location:$url");
$mytext=preg_match("/(?i)\b((?:https?://|www\d{0,3}[.]|[a-z0-9.\-]+[.][a-z]{2,4}/)(?:[^\s()<>]+|\(([^\s()<>]+|(\([^\s()<>]+\)))*\))+(?:\(([^\s()<>]+|(\([^\s()<>]+\)))*\)|[^\s`!()\[\]{};:'\".,<>?«»“”‘’]))/", "\1=\"https://avia-avia.ru/?url=<?=$base64URL?>\" target=\"_blank\"", $mytext);
exit();
} ?>

<a href="https://b.avia-avia.ru/lin">Ссылка</a>

[AUser0]

sochi-russia, в этом процитированном коде "не получается" вы пытаетесь раскодировать то, что вообще не пришло. Сначала исправьте в скрипте intest.php код генерации ссылки (тот самый $mytext=preg_replace(...)), добейтесь появления в ссылке строки "url=......". А уже после этого копайтесь в коде, принимающем и расшифровывающем ваш base64, который вы сейчас и привели.

[ThunderCat]

sochi-russia, Блин, вам даже готовый код не поможет, с вашей внимательностью...

не получается

Приведенный код во первых тоже имеет неэкранированные слэши в регулярке, во вторых речь шла о том что у вас помечено "на главной" - $mytext=preg_replace('..., который ничего не выдает, так как 1) регулярка неверная, 2) вам нужен только урл, а вы тянете зачем то весь хтмл тег, 3) не надо там что-то менять пока, для начала добейтесь чтобы "на главной" в $mytext попадал урл, после этого код можно будет достаточно просто поправить.

По шагам, что вы делали не так:
1) На главной надо получить только урл, вместо этого вы из строки <a href="some_url">пытаетесь получить другую строку<a href="other_url">, хотя она вам не нужна.
2) После получения чистого урл, кодируете его в бэйс64, и уже из этой строки нормально формируете хтмл ссылку (выше писал как).
3) в direct.php вы верно приняли гет переменную, но проверять ее через прег_матч смысла нет, так как она или пустая, что проверено выше, либо содержит урл, так как на главной вы уже проверили строку на совпадение с урл (тем более что у вас там в прег_матч нерабочая регулярка).
4) Редирект написан верно, после него в принципе ничего выполняться не будет, так что смысл располагать там какой-то код отсутствует. Для совместимости с очень старыми версиями пыха можно добавить exit();, но это больше перестраховка.