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

dailymotion fix, ok.ru support + some fixed #476

Merged
merged 42 commits into from
Jan 21, 2024
Merged

Conversation

SashaXser
Copy link

Поменял подход получения videoID, теперь он работает как надо.
P.S.: Зачем было всё так усложнять?

Поменял подход получения videoID, теперь он работает как надо
@ilyhalight
Copy link
Owner

при перезагрузке страницы сыпет ошибки, но работает
изображение

@SashaXser
Copy link
Author

при перезагрузке страницы сыпет ошибки, но работает изображение

Может быть плеер не успевает загрузить видео?

@ilyhalight
Copy link
Owner

при перезагрузке страницы сыпет ошибки, но работает изображение

Может быть плеер не успевает загрузить видео?

Сначало появляется плеер, начинает крутиться прелоадер и после этого идет несколько таких ошибок пока страница полностью не прогрузится
изображение

@SashaXser
Copy link
Author

SashaXser commented Jan 5, 2024

при перезагрузке страницы сыпет ошибки, но работает изображение

Может быть плеер не успевает загрузить видео?

Сначало появляется плеер, начинает крутиться прелоадер и после этого идет несколько таких ошибок пока страница полностью не прогрузится изображение

Это вероятно из-за того, что не успел подгрузится m3u8, поэтому и ошибки появляются.

А почему инициализация плеера происходит раньше инициализации базы данных??

@SashaXser SashaXser changed the title dailymotion fix dailymotion fix + ok.ru support Jan 14, 2024
@ilyhalight
Copy link
Owner

Этот код заставляет переводить видео без описания, и не важно какой у него там язык. Может быть стоит его переработать или удалить?

  // the "delayed video upload" fix for YouTube (#387)
  if (!(description && title)) {
    return "en";
  }

Возможно действительно стоит

@SashaXser
Copy link
Author

Этот код заставляет переводить видео без описания, и не важно какой у него там язык. Может быть стоит его переработать или удалить?

  // the "delayed video upload" fix for YouTube (#387)
  if (!(description && title)) {
    return "en";
  }

Возможно действительно стоит

Протестируй как сможешь

@ilyhalight
Copy link
Owner

ilyhalight commented Jan 20, 2024

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео
изображение

@SashaXser
Copy link
Author

SashaXser commented Jan 20, 2024

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

Хм, интересно. Это в любом браузере? У меня в Firefox вроде пока не воспроизводится

@SashaXser
Copy link
Author

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

@ilyhalight
Copy link
Owner

ilyhalight commented Jan 20, 2024

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

на других сайтах такого нет. Тестил на этом видео, но на других так же
изображение

@ilyhalight
Copy link
Owner

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

на других сайтах такого нет. Тестил на этом видео, но на других так же изображение

хм, возможно какой-то локальный баг. В других браузерах у меня не воспроизводится.

@SashaXser
Copy link
Author

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

на других сайтах такого нет. Тестил на этом видео, но на других так же изображение

хм, возможно какой-то локальный баг. В других браузерах у меня не воспроизводится.

Больше никаких ошибок не обнаружено?

@ilyhalight
Copy link
Owner

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

на других сайтах такого нет. Тестил на этом видео, но на других так же изображение

хм, возможно какой-то локальный баг. В других браузерах у меня не воспроизводится.

Больше никаких ошибок не обнаружено?

Ничего другого не нашел

@SashaXser
Copy link
Author

на рамбле после этих фиксов стала дублироваться кнопка. Чтобы вопроизвести нужно нажать на кнопку перевода видео изображение

А на других сайтах проблема присутствует? Протестировал в хроме и Edge, кнопка два раза не вылезала.

на других сайтах такого нет. Тестил на этом видео, но на других так же изображение

хм, возможно какой-то локальный баг. В других браузерах у меня не воспроизводится.

Больше никаких ошибок не обнаружено?

Ничего другого не нашел

Можно объединять, исправил конфликт с headers.json

@ilyhalight
Copy link
Owner

при перезагрузке страницы сыпет ошибки, но работает изображение

это не исправлено, все равно есть ошибки при загрузке видео, но перевод работает
изображение

@SashaXser
Copy link
Author

SashaXser commented Jan 21, 2024

при перезагрузке страницы сыпет ошибки, но работает изображение

это не исправлено, все равно есть ошибки при загрузке видео, но перевод работает изображение

Объясняю, теперь получения айди на dailymotion зависит от прогрузки m3u8 от плеера, тоесть, когда плеер начнёт грузить m3u8, тогда он получает айди и перевод происходит.

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

Когда скрипт загружается, он вызвает getVideoData, поэтому при загрузке появляются ошибки :/

    case "dailymotion": {
      // we work in the context of the player
      // geo.dailymotion.com
      const plainPlayerConfig = Array.from(
        document.querySelectorAll("*"),
      ).filter((s) => s.innerHTML.trim().includes(".m3u8")); // получаем данные из элемента содержащий .m3u8 тоесть плеера
      try {
        let videoUrl = plainPlayerConfig[1].lastChild.src;
        return videoUrl.match(/\/video\/(\w+)\.m3u8/)?.[1]; // удаляем всё лишнее и оставляем videoID
      } catch (e) {
        console.error("[VOT]", e);
        return false;
      }
    }
    ```

@ilyhalight
Copy link
Owner

при перезагрузке страницы сыпет ошибки, но работает изображение

это не исправлено, все равно есть ошибки при загрузке видео, но перевод работает изображение

Объясняю, теперь получения айди на dailymotion зависит от прогрузки m3u8 от плеера, тоесть, когда плеер начнёт грузить m3u8, тогда он получает айди и перевод происходит.

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

Когда скрипт загружается, он вызвает getVideoData, поэтому при загрузке появляются ошибки :/

Да это я понимаю, просто, не заметил, что там console.error стоял и из-за этого ошибки пишутся в консоли.

@SashaXser
Copy link
Author

при перезагрузке страницы сыпет ошибки, но работает изображение

это не исправлено, все равно есть ошибки при загрузке видео, но перевод работает изображение

Объясняю, теперь получения айди на dailymotion зависит от прогрузки m3u8 от плеера, тоесть, когда плеер начнёт грузить m3u8, тогда он получает айди и перевод происходит.
Ошибка происходит из-за того, что плеер еще не загрузил m3u8 плейлист видео, там же цикл стоит, пока m3u8 не загрузится, будет сыпать ошибку.
Когда скрипт загружается, он вызвает getVideoData, поэтому при загрузке появляются ошибки :/

Да это я понимаю, просто, не заметил, что там console.error стоял и из-за этого ошибки пишутся в консоли.

Убрать console.error или оставить?

@ilyhalight ilyhalight changed the base branch from master to dev January 21, 2024 12:18
@ilyhalight
Copy link
Owner

при перезагрузке страницы сыпет ошибки, но работает изображение

это не исправлено, все равно есть ошибки при загрузке видео, но перевод работает изображение

Объясняю, теперь получения айди на dailymotion зависит от прогрузки m3u8 от плеера, тоесть, когда плеер начнёт грузить m3u8, тогда он получает айди и перевод происходит.
Ошибка происходит из-за того, что плеер еще не загрузил m3u8 плейлист видео, там же цикл стоит, пока m3u8 не загрузится, будет сыпать ошибку.
Когда скрипт загружается, он вызвает getVideoData, поэтому при загрузке появляются ошибки :/

Да это я понимаю, просто, не заметил, что там console.error стоял и из-за этого ошибки пишутся в консоли.

Убрать console.error или оставить?

Пускай остается

@ilyhalight ilyhalight merged commit bfb53b6 into ilyhalight:dev Jan 21, 2024
3 checks passed
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

Successfully merging this pull request may close these issues.

None yet

2 participants