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

Make moira-url configurable in web docker container #173

Closed
3 tasks done
borovskyav opened this issue Dec 28, 2018 · 16 comments
Closed
3 tasks done

Make moira-url configurable in web docker container #173

borovskyav opened this issue Dec 28, 2018 · 16 comments
Assignees
Projects

Comments

@borovskyav
Copy link
Contributor

borovskyav commented Dec 28, 2018

ToDo:

  • придумать, где хранить подобный конфиг
  • реализовать такой конфиг
  • научить веб с ним работать
@borovskyav borovskyav created this issue from a note in 2.5 (Ready to develop) Dec 28, 2018
@beevee beevee changed the title Сделать возможность пробрасывания moira-url во фронте прямо из докер контейнера Make moira-url configurable in web docker container Dec 29, 2018
@sashasushko
Copy link
Contributor

Подытожу:
веб-морда долбится в /api того хоста, где развернута. Если АПИ где-то в другом месте, то сейчас это правится только костылём:

  • правкой передаваемого урла в src/index.js и пересборкой морды
  • либо в сбилженном файле app.[hash].js через F3, если совсем нельзя пересобрать морду

Думаю, из описания понятно, что это требует простой настройки.

@sashasushko
Copy link
Contributor

@beevee @kamaev нужны ваши мозги

@titusjaka
Copy link
Contributor

Это не то, что нужно? https://webpack.js.org/api/cli/

@sashasushko
Copy link
Contributor

sashasushko commented Feb 1, 2019

Это не то, что нужно? https://webpack.js.org/api/cli/

Это тогда нужно пользователю package.json править или Makefile

@borovskyav
Copy link
Contributor Author

А это проблема? Мы же все равно пользователю уже готовую статику даем, все билды через make сами вызываем.

@sashasushko
Copy link
Contributor

Давай сегодня на планировании обсудим. Моя твоя не понимать

@sashasushko
Copy link
Contributor

Итак, проблема такая:

путь до АПИ передаётся в конструкторе АПИ в джаваскрипте. При выполнении в браузере. Поэтому его нельзя зачитать ниоткуда, кроме как из window. Зачитать, например, из переменной окружения можно только при сборке.

@sashasushko
Copy link
Contributor

Закинул в чат разработчиков, может, кто-то подскажет идею.

@borovskyav
Copy link
Contributor Author

Сейчас как костыльный вариант обсуждаем дать пользователям возможность билдить веб с указанием node env {api_path}

@sashasushko
Copy link
Contributor

@borovskyav
А как это попало в 2.5 и почему?

@borovskyav
Copy link
Contributor Author

borovskyav commented Feb 25, 2019

Ну мы такие сидели на планировании, а потом я такой: "о, давайте возьмем эту задачу!". Вот так и попало)

@sashasushko
Copy link
Contributor

sashasushko commented Feb 25, 2019

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

@borovskyav что думаешь?

@borovskyav
Copy link
Contributor Author

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

@sashasushko
Copy link
Contributor

Тогда запиши себе на встречу 1 марта обсудить

@borovskyav
Copy link
Contributor Author

договорились

@kamaev
Copy link
Contributor

kamaev commented Apr 11, 2019

Closed in: moira-alert/web2.0#258

@kamaev kamaev closed this as completed Apr 11, 2019
2.5 automation moved this from In progress to Done Apr 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
2.5
  
Done
Development

No branches or pull requests

4 participants