Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FR - Пожелания по новым функциям #129

Open
alzhur opened this issue Dec 1, 2024 · 6 comments
Open

FR - Пожелания по новым функциям #129

alzhur opened this issue Dec 1, 2024 · 6 comments

Comments

@alzhur
Copy link

alzhur commented Dec 1, 2024

  1. Сохранение данных авторизации (куки?) в файл, и их использование когда нет явно заданных логина и пароля в опциях. Удобно, а также поможет решить проблемы типа "Не удалось авторизоваться. Вы превысили лимит попыток входа." при скачивании множества книг.
  2. Возможность указать имя файла, из которого брать ссылки на скачиваемые книги (вместо -u).
  3. Скорректировать генерацию файлов в формате fb2, чтобы получались более-менее валидные файлы. Если интересно что сейчас не так - подробности напишу.
@OnlyFart
Copy link
Owner

OnlyFart commented Dec 1, 2024

  1. Можно передавать сразу список ссылок, через запятую. Тогда авторизация будет только 1 раз

  2. Уже есть

  3. Пишите

@alzhur
Copy link
Author

alzhur commented Dec 1, 2024

  1. Даже если передавать несколько ссылок за раз, ИМХО, это не очень удобно. Просто мне понравилось, как это сделано у gogrepoc, один раз вызываешь программу с опциями для логина, а потом пользуешься, пока куки не протухнут. Но там своя специфика, скрипт нужно вызывать с разным набором опций для разных операций, тут, возможно, это менее актуально. Но лично мне так было бы удобнее.
  2. Может и есть, но не задокументировано, а значит как бы и нет.
  3. Кое-что уже писал вот тут, на днях перепроверю, уточню, может что-то уже не актуально, или ещё какие-то ошибки есть.

@OnlyFart
Copy link
Owner

OnlyFart commented Dec 1, 2024

  1. Все задукоментировано --book-name-pattern | (По-умолчанию: {Author.Name} - {Book.Title}) Шаблон для имени файла.
  2. --

@alzhur
Copy link
Author

alzhur commented Dec 1, 2024

2а) Задокументировано так себе. Дефолтное значение приведено, но какие кроме этого доступны элементы - не указано.

2б). Но, похоже, мы друг друга не поняли. Я вообще не про шаблон имени файла скачанной книги, а про чтение списка скачиваемых книг из файла, что-то типа вот такого (цитата из мана к mpg123, но подобное встречается много где, этакий примитивный вариант плейлиста в данном случае):

-@ file, --list file
Read filenames and/or URLs of MPEG audio streams from
the specified file in addition to the ones specified on
the command line (if any). Note that file can be
either an ordinary file, a dash ``-'' to indicate that
a list of filenames/URLs is to be read from the stan-
dard input, or an URL pointing to a an appropriate list
file. Note: only one -@ option can be used (if more
than one is specified, only the last one will be recog-
nized).

То есть что-то типа
Elib2EbookCli.exe -@ books.txt -f fb2 -l vasya -p pupkin
А в файле books.txt - ссылки на книги для скачивания, одна на строку, или через запятую, или ещё как-то (определиться со списком возможных разделителей).
Если такое реализовать - то и пункт первый не очень-то нужен.

@alzhur
Copy link
Author

alzhur commented Dec 3, 2024

Что касается валидности генерируемых fb2-файлов, повторю тут по свежим перепроверенным файлам.
Основная часть ошибок - в заголовочной части файла.
Скачал для теста 1 книгу, отрезал почти весь текст, и минимально поправил секцию description, чтобы файл проходил валидацию.
Изменения: добавить 3 тега, 2 существующих поменять местами, исправить имя обложки на валидное добавлением буквы в начало (касается всех картинок).
Результат - 2 файла в приложенном архиве.
example.zip
По общей части текста - нужно для пустой строки <p><br/></p> заменить на <empty-line/>.

@alzhur
Copy link
Author

alzhur commented Dec 20, 2024

В части фикса заголовков и имён картинок - сделал фикс.
#131
С заменой тегов разобраться сходу не удалось, поэтому для обработки <br/> всё ещё требуется отдельный фикс.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants