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

  • cryptOP
    link
    fedilink
    arrow-up
    0
    ·
    4 years ago

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

    • cocucka
      link
      fedilink
      arrow-up
      0
      ·
      4 years ago

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

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

      • cryptOP
        link
        fedilink
        arrow-up
        0
        ·
        4 years ago

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

        • cocucka
          link
          fedilink
          arrow-up
          0
          ·
          4 years ago

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

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

          • cryptOP
            link
            fedilink
            arrow-up
            0
            ·
            4 years ago

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

          • cryptOP
            link
            fedilink
            arrow-up
            0
            ·
            4 years ago

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

          • EXL
            link
            fedilink
            arrow-up
            0
            ·
            4 years ago

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

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

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

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

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

            • cocucka
              link
              fedilink
              arrow-up
              0
              ·
              4 years ago

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

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

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

              • l-xoid
                link
                fedilink
                arrow-up
                0
                ·
                4 years ago

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

              • cryptOP
                link
                fedilink
                arrow-up
                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
                link
                fedilink
                arrow-up
                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
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  4 years ago

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

                  • cryptOP
                    link
                    fedilink
                    arrow-up
                    0
                    ·
                    4 years ago

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

              • cryptOP
                link
                fedilink
                arrow-up
                0
                ·
                4 years ago

                зацени:

                commit 75ef1e3b960f75815d3e8849ca166cc1034b103f
                Author: Maxim Valyanskiy <max.valjanski@gmail.com>
                Date:   Thu Jul 7 00:01:02 2022 +0300
                
                    предупреждение об ответе на комментарий
                    
                    предупреждаем при ответе на комментарий в случае если
                    автор комментария больше не может писать в топик из-за
                    каких-то ограничений
                
      • cryptOP
        link
        fedilink
        arrow-up
        0
        ·
        4 years ago

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

        • cocucka
          link
          fedilink
          arrow-up
          0
          ·
          4 years ago

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

          • cryptOP
            link
            fedilink
            arrow-up
            0
            ·
            4 years ago

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

      • cryptOP
        link
        fedilink
        arrow-up
        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
        • }