258 сообщений в этой теме

Спасибо большое Всем кто принимал участие в выходе этого релиза! Ребята от души низкий Вам всем поклон

3

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Спасибо! Сейчас буду ставить и тестировать)

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Приемного благодарен.

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Дождались ) Спасибо огромное

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Супер, спасибо!

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

track_no-oc2.3.x.ocmod.xml в папке system немного штормит
Вот так норм

track_no-oc2.3.x.ocmod.xml

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Если в процессе роботы\тестирования будут выявлены баги - добавляйте пулл-реквесты на гитхаб https://github.com/myopencart/ocStore/tree/dev

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

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

 

 

Сори, разобрался. Проблема была в браузере.

Изменено пользователем kopiyos
0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Спасибо за релиз! Очень ждал. :eek:

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Огромное спасибо за развитие OcStore!!! :eek:

Изменено пользователем rihardknauff
0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах
  • Добавлена проверка SSL сертификата в catalog/controller/startup/startup.php

 

$_SERVER['HTTPS'] = 'off';

?

if ($this->request->server['HTTPS']) { ...

$_SERVER['HTTPS'] = 'off';

 

Не, не встречали ? А я частенько. Что будет с кодом выполнения IF ? Да, баг, к сожалению

 

Настройки https у разных хостеров могут диаметрально отличаться. У 50% хостеров они вообще криво - левые. Может вообще не быть $_SERVER['HTTPS], да и такое часто бывает

 

Так что код не зачтен = баг

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Марк, поделитесь своим решением https://github.com/m...cStore/tree/dev

ArtemPitov - однозначно правильного решения нет

Есть на 99%. T.е. при 10`000 скачиваний ocStore 100  магазинов все равно словят баг связанный с https

Но

if ($this->request->server['HTTPS']) { 

Имеет и того меньший процент ~ 50%

Да еще и notice получат, так как у многих вместо $_SERVER['HTTPS'] используется $_SERVER['HTTP_X_FORWARDED_PROTO'] или $_SERVER['HTTP_X_FORWARDED_SSL']

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Марк, поделитесь своим решением https://github.com/m...cStore/tree/dev

Код на 99%

if ((isset($_SERVER['HTTPS']) && ($_SERVER['HTTPS'] == 'on' || $_SERVER['HTTPS'] == '1')) || (!empty($_SERVER['HTTP_X_FORWARDED_PROTO']) && (strtolower($_SERVER['HTTP_X_FORWARDED_PROTO']) == 'https') || (!empty($_SERVER['HTTP_X_FORWARDED_SSL']) && $_SERVER['HTTP_X_FORWARDED_SSL'] == 'on'))) {
...
}
1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

 

Код на 99%

if ((isset($_SERVER['HTTPS']) && ($_SERVER['HTTPS'] == 'on' || $_SERVER['HTTPS'] == '1')) || (!empty($_SERVER['HTTP_X_FORWARDED_PROTO']) && (strtolower($_SERVER['HTTP_X_FORWARDED_PROTO']) == 'https') || (!empty($_SERVER['HTTP_X_FORWARDED_SSL']) && $_SERVER['HTTP_X_FORWARDED_SSL'] == 'on'))) {
...
}

Поправим  :-)

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Но все это можно чаcтично решить строкой в htaccess

 

RewriteEngine On
RewriteCond %{HTTP:X-Forwarded-Proto} https
RewriteRule .* - [E=HTTPS:on]
 

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Поправим  :-)

Толку... никакого  - там в opencart еще туча багов связанных с "https"

Я промолчу еще сколько их в модулях. Все равно на выходе получается "каша"

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Толку... никакого  - там в opencart еще туча багов связанных с "https"

Я промолчу еще сколько их в модулях. Все равно на выходе получается "каша"

нужно все собирать и фиксить и только тогда мы все получим хороший продукт 

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

1

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

https://github.com/myopencart/ocStore/blob/dev/upload/admin/controller/extension/feed/google_base.php

66 строка  :-D

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

Та да. вот и я о том же :(

И это помимо багов opencart в стиле

if ($this->request->server['HTTPS']) {

Где может запросто быть $_SERVER['HTTPS'] = 'off'

Там куда не кинь взгляд с https везде баги

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Да в opencart они на каждом шагу

 

order.php !!!

 

Что будет при $_SERVER['HTTPS'] = 'off'  или отсутствии $_SERVER['HTTPS']

if ($order_info) {
                $this->load->language('extension/payment/ocstore_payeer');
                if (($this->config->get('ocstore_payeer_order_later_status_id') == $this->request->post['order_status_id']) && ($order_info['payment_code'] == 'ocstore_payeer')) {
                        $onpay_url = sprintf('%sindex.php/?route=extension/payment/ocstore_payeer/laterpay&order_id=%s&order_tt=%s',
                                                defined('HTTPS_SERVER') ? HTTPS_SERVER : HTTP_SERVER,
                                                $order_info['order_id'],
                                                $order_info['total']
                        );
                        $this->request->post['comment'] .= "\n" . sprintf($this->language->get('text_payeer_onpay'), $onpay_url);
                }

                $this->load->language('extension/payment/ocstore_yk');
                if (($this->config->get('ocstore_yk_order_later_status_id') == $this->request->post['order_status_id']) && (strpos($order_info['payment_code'], 'ocstore_yk') !== false)) {
                    $onpay_url = sprintf('%sindex.php/?route=extension/payment/ocstore_yk/laterpay&order_id=%s&order_ttl=%s&paymentType=%s',
                                                                isset($this->request->server['HTTPS']) && $this->request->server['HTTPS'] ? $this->config->get('config_ssl') : $this->config->get('config_url'),
                                                                $order_info['order_id'],
                                                                $order_info['total'],
                                                                $order_info['payment_code']
                                                             );
                    $this->request->post['comment'] .= "\n" . sprintf($this->language->get('text_yk_onpay'), $onpay_url);
                }

                $this->load->language('api/order');

                $this->model_checkout_order->addOrderHistory($order_id, $this->request->post['order_status_id'], $this->request->post['comment'], $this->request->post['notify'], $this->request->post['override']);

                $json['success'] = $this->language->get('text_success');
            } else {
                $json['error'] = $this->language->get('error_not_found');
            }

И так везде в opencart

Не могли уже сделать ОДНУ точку входа  этой определяемой переменной (или константы), чтобы потом через define('HTTPS', ...функция определения...); и т п

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Та да. вот и я о том же :(

И это помимо багов opencart в стиле

if ($this->request->server['HTTPS']) {

Где может запросто быть $_SERVER['HTTPS'] = 'off'

Там куда не кинь взгляд с https везде баги

 

off вернет только в одном случае 

 

 

Обратите внимание, что при использовании ISAPI с IIS значение будет off, если запрос не был произведен через протокол HTTPS.

 

Но самое интересное в том что есть 33 вставки по коду HTTP_CATALOG

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах

Ну, это информационная строка.

0

Поделиться этим сообщением


Ссылка на сообщение
Поделиться на других сайтах
Гость
Эта тема закрыта для публикации сообщений.

  • Недавно просматривали   0 пользователей

    Ни один зарегистрированный пользователь не просматривает эту страницу.