смотрите, на оригинальнмом лоре появился разделитель в треде по дням! эта фича закомичена после нашего форка. @cocucka, отсюда вопрос, как обстоят у нас дела с синхронизацией с апстримом? хочу такие же некромаркеры! я не могу их просто так перенести, т.к. maxcom постепенно переписывает весь код с java на scala. мне удается бекпортировать изменения из апстрима maxcom, но у нас они не собираются с ошибкой (-ами) скалы.

  • cocucka
    0·
    4 years ago

    вот мы видим классную фичу в апстриме

    Не увидишь ты там особо классных фич. Ту хрень с датой я могу добавить легко, а срань типа постинга в телеграм топ-10 постов и ограничение по странам нахрен не нужна.

    • cryptOP
      0·
      4 years ago

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

      • cocucka
        0·
        4 years ago

        Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.

        Банально убрать хардкод путей и сделать нормальный конфиг – уже геморрой с интеграцией (не сильный, но всё ж время займёт). Плюс код надо тестировать, проверять, а не ебашить на прод в режиме нонстоп, надеясь что оно заработает.

        • cryptOP
          0·
          4 years ago

          «Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.» ок, точки над i. тогда нахрена весь тот легаси в sql с 1998 тащить?) можно rebase делать?:) «уже геморрой с интеграцией (не сильный, но всё ж время займёт).» а так у нас получается тоже время замерло на 10 дней. зато я за выходные добавил нижнюю панель, в середине разработки PM. ты готов это такими же темпами делать? мне кажется, PM - это must have. тесты ты сейчас игнорируешь вообще при сборке. почему бы не сделать мне возможность подымать новые инстансы для теста просто прописывая нужный домен в конфиге?

        • cryptOP
          0·
          4 years ago

          «Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot.» там какой-то бамп версий spring у него. в связи с безопасностью, наверное. не знаю, важно это или нет. я не стал переносить, т.к. сразу несколько файлов затрагивает.

        • EXL
          0·
          4 years ago

          Ну а у меня другое видение, мне, например, скала тут нахер не нужна. Я буду выпиливать необходимость запускать томкат отдельно – переведу движок на spring boot. Выкину хрень с телеграммом, добавлю нормальные ACL для контроля доступа к разделам и т.д.

          В продолжение этого: https://linuxtalks.co/forum/feedback/4?cid=2605

          Может действительно проще будет развернуть второй инстанс-скелет LOR’а с базовой функциональностью (профили/темы/комменты) на Spring Boot и твоём_любимом_шаблонизаторе вместо JSP и потихоньку переносить туда код и логику из оригинального движка? Натянуть похожий дизайн.

          В итоге движок сразу получится без всего этого тяжёлого Legacу-наследия и гибридизации со Scala, которая многим непонятна. И ты сможешь добавлять в него фичи гораздо активнее.

          Как думаешь? Или всё-таки намерен джедайствовать со всем этим?

          • cocucka
            0·
            4 years ago

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

            Мы подняли ЛТ на лоровском движке т.к. он привычен для народа с ЛОРа и поднять его оказалось не так уж и сложно. Понятное дело, что тупо лор один-в-один с другими модераторами никому не нужен, поэтому мы тут вдыхаем свежую струю в концепт форума линуксоидов и в плане модерации, и в плане функционала.

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

            • l-xoid
              0·
              4 years ago

              У лора норм движок, из него можно сделать офигенно удобный сайт, если упростить структуру и придать более динамичный характер. Сделать трекер главной страницей, например.

            • cryptOP
              0·
              4 years ago

              код начал загнивать:(

              [WARNING] /root/lor_crypt/lorsource/src/test/scala/ru/org/linux/search/ElasticsearchIndexServiceIntegrationSpec.scala:76: warning: object ElasticsearchClientUri in package elastic4s is deprecated (since 6.3.3): Use ElasticNodeEndpoint

              обратил на это внимание, т.к. как раз вчера для эластика правил. и что мне с этим делать? подправить существующий scala или тут вообще все надо на java переписывать?

              я понимаю, что это ворнинг, но …

            • cryptOP
              0·
              4 years ago

              кстати, макском активно комитит эти несколько дней

              Date:   Sat Jun 18 10:43:51 2022 +0300
Date:   Sat Jun 18 09:20:52 2022 +0300
Date:   Fri Jun 17 20:37:36 2022 +0300
Date:   Fri Jun 17 20:19:48 2022 +0300
Date:   Fri Jun 17 19:32:52 2022 +0300
Date:   Fri Jun 17 18:50:11 2022 +0300
Date:   Fri Jun 17 13:46:22 2022 +0300
Date:   Fri Jun 17 13:36:25 2022 +0300
Date:   Thu Jun 16 23:14:50 2022 +0300
Date:   Thu Jun 16 23:13:15 2022 +0300
              • CrocodoomBanned
                0·
                4 years ago

                Почувствовал, что запахло жареным 🙂

                • cryptOP
                  0·
                  4 years ago

                  нет, там в основном правила, косметика и багфиксы. просто нам бы багфиксы тоже не помешали.

            • cryptOP
              0·
              4 years ago

              зацени:

              commit 75ef1e3b960f75815d3e8849ca166cc1034b103f
Author: Maxim Valyanskiy <max.valjanski@gmail.com>
Date:   Thu Jul 7 00:01:02 2022 +0300

    предупреждение об ответе на комментарий
    
    предупреждаем при ответе на комментарий в случае если
    автор комментария больше не может писать в топик из-за
    каких-то ограничений
    • cryptOP
      0·
      4 years ago

      перед этим он добавлял какие-то мелочи для модераторов, чтобы им было удобнее. не будешь ты всю эту хрень делать. забьешь со словами: “и так сканает! есть куча более прикольных вещей.” я посмотрел, изменения макскома очень дотошные. в этом проекте этого не будет. а ведь это вцелом будет определять качество сайта и его юзабилити. во-вторых, он тоже может копировать хорошие фичи отсюда, если раскрутимся. единственное, почему он до сих пор не сделал PM - это отсутствие фантазии. если возникнет такая гонка, я не уверен, что у нашего проекта есть ресурс для переимплеементации фич. но когда кодовая база синхронизирована с апстримом “лямзить” фичи проще.

      • cocucka
        0·
        4 years ago

        Макском ещё месяцок фичи попилит и опять забъёт на пару лет, так всегда было. Он сейчас активизировался из-за войны и боязни проблем.

        • cryptOP
          0·
          4 years ago

          «Макском ещё месяцок фичи попилит и опять забъёт на пару лет, так всегда было. Он сейчас активизировался из-за войны и боязни проблем.» значит, тем легче следовать за апстримом. да, работает неравномерно и нам это наруку.

    • cryptOP
      0·
      4 years ago

      «Не увидишь ты там особо классных фич.» он собирается внедрять какой-то http firewall. хз что это ка. + @Bean

      • def httpFirewall: StrictHttpFirewall = {
      • val firewall = new StrictHttpFirewall
      • firewall.setAllowedHeaderValues { header =>
      •  val parsed = new String(header.getBytes(StandardCharsets.ISO_8859_1), StandardCharsets.UTF_8)

      •  allowed.matcher(parsed).matches
      • }
      • firewall
      • }