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

  • cocucka
    0·
    4 years ago

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

    Банально, если он выкатит изменение в DDL и я выкачу свои, то миграционные патчи уже будут конфликтовать.

    • cryptOP
      0·
      4 years ago

      отсюда есть два следствия: а) можно дропнуть все то легаси, что накопилось, вроде кучи каких-то SQL данных и тестов. б) встает вопрос в сабже. вот мы видим классную фичу в апстриме. сам ты ее не будешь делать, потому что некогда. он ее уже сделал. в итоге проект теряет.

      • 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
          • }