Observer — распаковка файлов из различных контейнеров

Здесь можно обсудить ваш любимый плагин.
Ariman
Posts: 1203
Joined: Fri 30 Nov, 2007 15:32
Location: Крым
Has thanked: 12 times
Been thanked: 26 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by Ariman » Mon 31 Aug, 2009 11:29

Тема для обсуждения плагина Observer.
Плагин предназначен только для распаковки файлов. Запаковка/редактирование архивов не планируется.

Хотелки/баги:

Известные проблемы:

Планируемые новые модули:

Если "ничего не работает", то прежде чем писать сюда:


Сторонние модули:
Dir — поддержка вывода команд dir и ls в качестве виртуального архива. (на версии 1.10 работать не будет в связи с изменением API модулей)
Inno Setup - поддержка файлов Inno Setup (сделан на основе innounp).

User avatar
cmd.exe
Posts: 358
Joined: Fri 16 Apr, 2010 18:26
Location: Нижний Новгород
Has thanked: 178 times
Been thanked: 5 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by cmd.exe » Wed 13 Dec, 2017 17:21

Latest post of the previous page:

As implemented in Chrome jianli Nov 21, 2017
Примечание: длинную строку заголовка можно разбить на несколько строк с использованием разрыва мягкой строки "CRLF+SPACE/TAB" на RFC 2047.
Не следует использовать мягкий разрыв строки "=CRLF", используемый для разрыва длинной строки в теле сообщения, как определено в RFC 2045.
Тут просто возникает вопрос к стандарту, в теории поймать такое поведение не сложно.
Last edited by cmd.exe on Wed 13 Dec, 2017 17:21, edited 1 time in total.
I've been using Vim for about 2 years now, mostly because I can't figure out how to exit it. (c) I Am Devloper
I know nothing except the fact of my ignorance. (с) Socrates

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Wed 13 Dec, 2017 21:04

Ariman wrote:
Tue 12 Dec, 2017 15:21
Надо в Оперу писать
Написал. Реакции (пока) нет. Все заинтересованные могут поддержать топик, написав в нем что нибудь вроде "Да, так и есть! Opera не права! Требуем соблюдать конституцию RFC!"
Last edited by Dec on Wed 13 Dec, 2017 21:05, edited 1 time in total.

User avatar
cmd.exe
Posts: 358
Joined: Fri 16 Apr, 2010 18:26
Location: Нижний Новгород
Has thanked: 178 times
Been thanked: 5 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by cmd.exe » Wed 13 Dec, 2017 21:59

Dec, Я думаю что лучше писать сразу разработчикам, а не на форум. Лучше всего пулреквестом в гит.
I've been using Vim for about 2 years now, mostly because I can't figure out how to exit it. (c) I Am Devloper
I know nothing except the fact of my ignorance. (с) Socrates

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Wed 13 Dec, 2017 22:17

cmd.exe, Я посмотрел ссылку, и честно говоря, не понял, куда конкретно писать.

User avatar
cmd.exe
Posts: 358
Joined: Fri 16 Apr, 2010 18:26
Location: Нижний Новгород
Has thanked: 178 times
Been thanked: 5 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by cmd.exe » Wed 13 Dec, 2017 22:30

Dec, думаю в десктоп версию https://github.com/operasoftware/desktop/issues
Пишу с планшетника. В исходники пока несмотрел подробно.
I've been using Vim for about 2 years now, mostly because I can't figure out how to exit it. (c) I Am Devloper
I know nothing except the fact of my ignorance. (с) Socrates

User avatar
HaRT
Moderator
Posts: 8532
Joined: Tue 30 Aug, 2005 17:21
Location: Санкт-Петербург
Has thanked: 75 times
Been thanked: 138 times

Observer — распаковка файлов из различных контейнеров

Post by HaRT » Wed 13 Dec, 2017 23:17

2useven10 wrote:
Tue 12 Dec, 2017 15:34
Может стоит посмотреть rfc2184 ?
Кто-нибудь из участников дискуссии её изучил?
Фар есть инструмент, а не нянька. © 2009 DrKnS

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Thu 14 Dec, 2017 00:25

HaRT wrote:
Wed 13 Dec, 2017 23:17
Если будете делать им патч
Если это мне адресовано, то я даже не знаю, где найти и посмотреть код, который отвечает на генерацию mhtml. И вряд ли моих талантов хватит,что бы делать патч.
HaRT wrote:
Wed 13 Dec, 2017 23:17
вставить разрыв строки и пробел недостаточно
Недостаточно для полного соблюдения rfc. Но для решения конкретной проблемы вполне достаточно.
HaRT wrote:
Wed 13 Dec, 2017 23:17
rfc2184 Кто-нибудь из участников дискуссии её изучил?
Не имеет особого отношения к проблеме.

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Thu 14 Dec, 2017 04:58

На форуме Opera подтвердили наличие проблемы - она в движке chromium, соответственно дефект имею все браузеры, основанные на этом движке, включая сам Chrome. И предложили писать в СпортЛото на баг трекер chromium.

Ariman
Posts: 1203
Joined: Fri 30 Nov, 2007 15:32
Location: Крым
Has thanked: 12 times
Been thanked: 26 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by Ariman » Thu 14 Dec, 2017 08:42

HaRT wrote:
Wed 13 Dec, 2017 23:17
Кто-нибудь из участников дискуссии её изучил?
Я смотрел, там не отменили наличие пробела в начале перенесенной строки.
Dec wrote:
Thu 14 Dec, 2017 04:58
соответственно дефект имею все браузеры, основанные на этом движке, включая сам Chrome.
Такие дефектные файлы с любой страницы сохраняются? А то у меня Оперы новой нет чтобы проверить.

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Thu 14 Dec, 2017 09:12

Ariman wrote:
Thu 14 Dec, 2017 08:42
там не отменили наличие пробела в начале перенесенной строки.
Да и вряд ли когда нибудь отменят. Я когда писал eDecoder (продукт, конкурирующий с Observer в плане работы с mime) все эти RFC перечитал от корки до корки. И сквозь все RFC проходит красной нитью одна идея - совместимость. Т.е. есть изначальный RFC 822 и куча расширений. И клиент, написанный для обработки документов, созданных в соответствии с RFC 822, должен автоматически уметь открывать и документы, созданные в соответствии с дополнениями. Под открывать понимается не полная декодировка, а тот факт, что клиент просто отображает то, что смог понять. Т.е. если клиент не умеет декодировать =?<Charset>?<Encoding>?<Data>?= в человеческий вид, то он просто (автоматически) отобразит эту строку без декодировки. RFC 822 определяет дополнительный пробел при переносе, и все расширяющие RFC также пишут английскими буквами на белом фоне, что пробел нужен.
Ariman wrote:
Thu 14 Dec, 2017 08:42
Такие дефектные файлы с любой страницы сохраняются?
Я думаю не с любой, а только с той, у которой title содержит символы с кодом > 127. Я запостил на трекер chromium о проблеме, подождем реакции.
Last edited by Dec on Thu 14 Dec, 2017 09:13, edited 1 time in total.

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Fri 15 Dec, 2017 08:45

Судя по скорости наполнения трекера chromium, мой запрос окажется на 50-й странице уже через неделю. Так что опять же обращение ко всем заинтересованным и просто сочувствующим зайти и поставить звездочку. Чем больше звездочек - тем выше шанс, что на проблему обратят внимание.

User avatar
cmd.exe
Posts: 358
Joined: Fri 16 Apr, 2010 18:26
Location: Нижний Новгород
Has thanked: 178 times
Been thanked: 5 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by cmd.exe » Fri 15 Dec, 2017 14:17

Dec, тут просят:
Укажите пример URL-адреса, чтобы проверить эту проблему, а также сообщите нам, где это проверить. Это поможет в дальнейшем решить данную проблему.
Last edited by cmd.exe on Fri 15 Dec, 2017 14:17, edited 1 time in total.
I've been using Vim for about 2 years now, mostly because I can't figure out how to exit it. (c) I Am Devloper
I know nothing except the fact of my ignorance. (с) Socrates

User avatar
cmd.exe
Posts: 358
Joined: Fri 16 Apr, 2010 18:26
Location: Нижний Новгород
Has thanked: 178 times
Been thanked: 5 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by cmd.exe » Fri 19 Jan, 2018 11:27

Ошибку в Mime поправили пишут разработчики.
https://bugs.chromium.org/p/chromium/is ... 794835#c10
I've been using Vim for about 2 years now, mostly because I can't figure out how to exit it. (c) I Am Devloper
I know nothing except the fact of my ignorance. (с) Socrates

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Fri 19 Jan, 2018 11:36

Поздравляю, господа! Нам удалось раскрутить эту махину на исправление бага. Всего лишь месяц потребовался, я считаю это успехом! Ждем результатов в обновлениях браузеров.

Ariman
Posts: 1203
Joined: Fri 30 Nov, 2007 15:32
Location: Крым
Has thanked: 12 times
Been thanked: 26 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by Ariman » Fri 19 Jan, 2018 12:06

Хороший результат.
Надо было бы, наверное, девелоперов направить на либу, которая в плагине используется, а не только на сам плагин. Но это уже не важно, раз починили и так.

Dec
Posts: 66
Joined: Wed 14 Jul, 2010 08:59
Been thanked: 3 times

Observer — распаковка файлов из различных контейнеров

Post by Dec » Fri 19 Jan, 2018 12:12

Девелоперов направили на правильный RFC, что более правильно. То, как реализован парсинг в какой то конкретной сторонней библиотеке дело десятое. imho.

Ariman
Posts: 1203
Joined: Fri 30 Nov, 2007 15:32
Location: Крым
Has thanked: 12 times
Been thanked: 26 times
Contact:

Observer — распаковка файлов из различных контейнеров

Post by Ariman » Fri 19 Jan, 2018 12:52

Это больше вопрос удобства отладки и тестирования. На сайте либы есть список всех RFC, которые там реализованы, плюс список разного софта, который её использует. Если, скажем, девелопер этого кода сидит на линухе, то возиться с Far-ом ему будет не удобно.
Ну и достаточно часто на несовместимость с распространенным софтом реагируют охотнее, чем на несовместимость с более редким :)

Post Reply

Return to “Обсуждение плагинов”