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

PRO-4176: add auth scopes #365

Merged
merged 4 commits into from
Jan 24, 2024
Merged

PRO-4176: add auth scopes #365

merged 4 commits into from
Jan 24, 2024

Conversation

artem-malko
Copy link
Contributor

@artem-malko artem-malko commented Dec 12, 2023

Добавлена возможность задать необходимые scopes для openid-провайдера по просьбе Дениса Яковлева для релиза в Momrah, так как им нужны свои скоупы, не те, что захардкожены у нас в коде.
Подробнее тут: https://jira.2gis.ru/browse/PRO-4176


Изменения:


pro-ui: https://tsup.2gis.dev/pro/pro-ui/changelog/1.5.0..1.7.5

Что можно проверить?

UI:

  1. Добавление даты для карточки в библиотеке (если включена библиотека в онпреме, я так и не понял это) https://jira.2gis.ru/browse/PRO-3468

  2. Добавлена возможность задать список scopes для openid-connect Переменная в helm — oAuthScopes (https://jira.2gis.ru/browse/PRO-4176)

  3. В настройках визуализации изохрона выводим его опции в иконке рядом с названием ассета (https://jira.2gis.ru/browse/PRO-3250)
    Нужно перейти в точечный слой с <500 объектов и нажать Proximity zones. Далее перейти в новый слой и нажать иконку !

Screenshot 2023-12-13 at 22 32 21
  1. Invalid_scope with Momrah SSO (https://jira.2gis.ru/browse/PRO-4176)

Наверное лучше всего проверить у Момры авторизацию. Я проверял на нашем кейклоаке (https://momrah-keycloak.on-premise.2gis.dev/)

  1. В онпреме должна появиться возможность загружать свои ассеты. На главной странице будет кнпока Upload data
Screenshot 2023-12-14 at 10 12 57
  1. Также на главной странице должна появиться кнопка All datasets
Screenshot 2023-12-14 at 10 13 09
  1. Если задать в values appLocale=ar-AE (или ar_AE, мы оба формата поддерживаем), то pro-ui должен сразу стартануть на арабском

UPD: 1.7.5:

  1. На дашборде появился поиск https://jira.2gis.ru/browse/PRO-3759

На главной странице нажать на кнопку All datasets, ввести что-то в инпут поиска

Screenshot 2024-01-22 at 14 13 27

@ostrovskiy2gis ostrovskiy2gis changed the title WIP: PRO-4176: add auth scopes PRO-4176: add auth scopes Dec 13, 2023
@endryhold
Copy link
Contributor

Задача на проверку
https://jira.2gis.ru/browse/ONPREM-579

charts/pro-api/README.md Outdated Show resolved Hide resolved
charts/pro-api/README.md Outdated Show resolved Hide resolved
image_versions.txt Outdated Show resolved Hide resolved
@artem-malko
Copy link
Contributor Author

@endryhold ребят, а шо с этим MR?)

@artem-malko artem-malko force-pushed the PRO-4176-scope-config branch from f31e8fc to c9e9ed7 Compare January 11, 2024 08:51
@artem-malko
Copy link
Contributor Author

Если что, убрал тут все обновления бэка. Они сделаны в #382

@artem-malko artem-malko requested review from a team as code owners January 19, 2024 09:52
@dbelyaev-nsk dbelyaev-nsk merged commit bc3f69b into develop Jan 24, 2024
1 check passed
@dbelyaev-nsk dbelyaev-nsk deleted the PRO-4176-scope-config branch January 24, 2024 11:52
dbelyaev-nsk added a commit that referenced this pull request Mar 18, 2024
* Twins importer (#360)

* Twins importer

* Twins importer add dgctlStorage settings

* Twins importer fix chart

* fix chart

* small fix

* add logs

* [keys-api] up keys-backend. Support truck, map-matching api keys (#386)

* Update citylens to 1.4.0 (#388)

* Update citylens to 1.4.0

* bump to 1.4.1

* Feature: add codeowners file (#383)

* [license] Reworked securityContext for license type 2 (#387)

* [PRO-4361] Обновление pro-api до версии 1.1.79 (#382)

* [PRO-4361] Обновление pro-api до версии 1.1.74

* PRO-4361: update to 1.1.77

* PRO-4371: update to 1.1.78

* PRO-4361: update to 1.1.79

* PRO-4176: add auth scopes (#365)

* PRO-4176: add auth scopes

* PRO-4176: update pro-ui version

* PRO-4176: update app version

* PRO-4176: update to 1.7.5

* navi-castle: bugfix (#393)

- scheduled tasks without PV
- configuration parameters mistaken

* navi-restrictions-api 1.0.1 release (#392)

* navi-restrictions-api 1.0.1 release

* readme upd

* [DEVOPS-982] navi-front: update default locations (#389)

* [DEVOPS-982] navi-front: Add pedestrian location (#396)

* [platform] Upgrade version (#381)

* PRO-3766: update platform to 0.8.1

* Fix Breaking-Changes (#401)

* generic-chart added (#398)

* PRO-3609: upgrade pro ui to 1.9.0 (#399)

* navi-back: upstream update 7.15.2.4 (#402)

* [PRO-4517] обновил хелм-чарт pro-api в onprem до версии 1.5.0 (#394)

* PRO-0: upgrade PRO API to 1.5.1 (#404)

* Feature: add PR template (#410)

* PRO-4774: upgraded PRO API to 1.6.0 (#407)

* [PRO-4635] pro-ui, версия 2.1.1 (#405)

* Citylens 1.6.0 (#406)

* Update citylens to 1.5.0

* Bump appversion

* Fix citylens-api configmap

* Fix license server url template

* Fix review notes

* Update headerLinks

* Add camcom to headerLinks

* Fix page name in configmap

* Bump to 1.5.1

* Bump to 1.6.0

* Update breaking changes & bump app version in chart

* [navi-router] rename router.keyManagementServices to keys (#409)

* [navi-router] Remove trailing dots in param descriptions

* [navi-router] Rename router.keyManagementService -> keys

* [navi-router] Rename router.castleHost -> router.castleUrl

* release-1.20.0

* add breacking-changes to template
dbelyaev-nsk added a commit that referenced this pull request Mar 20, 2024
* Twins importer (#360)

* Twins importer

* Twins importer add dgctlStorage settings

* Twins importer fix chart

* fix chart

* small fix

* add logs

* [keys-api] up keys-backend. Support truck, map-matching api keys (#386)

* Update citylens to 1.4.0 (#388)

* Update citylens to 1.4.0

* bump to 1.4.1

* Feature: add codeowners file (#383)

* [license] Reworked securityContext for license type 2 (#387)

* [PRO-4361] Обновление pro-api до версии 1.1.79 (#382)

* [PRO-4361] Обновление pro-api до версии 1.1.74

* PRO-4361: update to 1.1.77

* PRO-4371: update to 1.1.78

* PRO-4361: update to 1.1.79

* PRO-4176: add auth scopes (#365)

* PRO-4176: add auth scopes

* PRO-4176: update pro-ui version

* PRO-4176: update app version

* PRO-4176: update to 1.7.5

* navi-castle: bugfix (#393)

- scheduled tasks without PV
- configuration parameters mistaken

* navi-restrictions-api 1.0.1 release (#392)

* navi-restrictions-api 1.0.1 release

* readme upd

* [DEVOPS-982] navi-front: update default locations (#389)

* [DEVOPS-982] navi-front: Add pedestrian location (#396)

* [platform] Upgrade version (#381)

* PRO-3766: update platform to 0.8.1

* Fix Breaking-Changes (#401)

* generic-chart added (#398)

* PRO-3609: upgrade pro ui to 1.9.0 (#399)

* navi-back: upstream update 7.15.2.4 (#402)

* [PRO-4517] обновил хелм-чарт pro-api в onprem до версии 1.5.0 (#394)

* PRO-0: upgrade PRO API to 1.5.1 (#404)

* Feature: add PR template (#410)

* PRO-4774: upgraded PRO API to 1.6.0 (#407)

* [PRO-4635] pro-ui, версия 2.1.1 (#405)

* Citylens 1.6.0 (#406)

* Update citylens to 1.5.0

* Bump appversion

* Fix citylens-api configmap

* Fix license server url template

* Fix review notes

* Update headerLinks

* Add camcom to headerLinks

* Fix page name in configmap

* Bump to 1.5.1

* Bump to 1.6.0

* Update breaking changes & bump app version in chart

* [navi-router] rename router.keyManagementServices to keys (#409)

* [navi-router] Remove trailing dots in param descriptions

* [navi-router] Rename router.keyManagementService -> keys

* [navi-router] Rename router.castleHost -> router.castleUrl

* WAPI-23469 Add configuration for audit logging (#414)

* WAPI-23469 Add configuration for audit logging

* fixes

* upd version tag

* release-1.20.2
Efber added a commit that referenced this pull request Apr 2, 2024
* Twins importer (#360)

* Twins importer

* Twins importer add dgctlStorage settings

* Twins importer fix chart

* fix chart

* small fix

* add logs

* [keys-api] up keys-backend. Support truck, map-matching api keys (#386)

* Update citylens to 1.4.0 (#388)

* Update citylens to 1.4.0

* bump to 1.4.1

* Feature: add codeowners file (#383)

* [license] Reworked securityContext for license type 2 (#387)

* [PRO-4361] Обновление pro-api до версии 1.1.79 (#382)

* [PRO-4361] Обновление pro-api до версии 1.1.74

* PRO-4361: update to 1.1.77

* PRO-4371: update to 1.1.78

* PRO-4361: update to 1.1.79

* PRO-4176: add auth scopes (#365)

* PRO-4176: add auth scopes

* PRO-4176: update pro-ui version

* PRO-4176: update app version

* PRO-4176: update to 1.7.5

* navi-castle: bugfix (#393)

- scheduled tasks without PV
- configuration parameters mistaken

* navi-restrictions-api 1.0.1 release (#392)

* navi-restrictions-api 1.0.1 release

* readme upd

* [DEVOPS-982] navi-front: update default locations (#389)

* [DEVOPS-982] navi-front: Add pedestrian location (#396)

* [platform] Upgrade version (#381)

* PRO-3766: update platform to 0.8.1

* Fix Breaking-Changes (#401)

* generic-chart added (#398)

* PRO-3609: upgrade pro ui to 1.9.0 (#399)

* navi-back: upstream update 7.15.2.4 (#402)

* [PRO-4517] обновил хелм-чарт pro-api в onprem до версии 1.5.0 (#394)

* PRO-0: upgrade PRO API to 1.5.1 (#404)

* Feature: add PR template (#410)

* PRO-4774: upgraded PRO API to 1.6.0 (#407)

* [PRO-4635] pro-ui, версия 2.1.1 (#405)

* Citylens 1.6.0 (#406)

* Update citylens to 1.5.0

* Bump appversion

* Fix citylens-api configmap

* Fix license server url template

* Fix review notes

* Update headerLinks

* Add camcom to headerLinks

* Fix page name in configmap

* Bump to 1.5.1

* Bump to 1.6.0

* Update breaking changes & bump app version in chart

* [navi-router] rename router.keyManagementServices to keys (#409)

* [navi-router] Remove trailing dots in param descriptions

* [navi-router] Rename router.keyManagementService -> keys

* [navi-router] Rename router.castleHost -> router.castleUrl

* WAPI-23469 Add configuration for audit logging (#414)

* WAPI-23469 Add configuration for audit logging

* fixes

* upd version tag

* fix duplication key

* [navi-restrictions] update to style guide (#408)

* [tiles-api] Upgrade to 4.52.8 (#416)

* WAPI-23326 Add new s3 params, custom ca configuration (#390)

* WAPI-23326 Add new s3 config params to keys and catalog-api charts

* WAPI-23326 Add custom certs configuration to keys, catalog-api charts

* fix typos

* more fixes

* fix configmap deploy

* split configmap config for jobs and deploys

* upd config for keys

* fix to keys chart

* upd image tags

* upd docs

* Chore: fix PR template (#418)

* [PRO-4789] поправили values для сервиса ключей

* Citylens 1.7.0 (#422)

* Citylens 1.7.0

* bump version to 1.7.1

* Make reporterProTracks deploy one or no replicas

* Bump version to 1.7.2

* [tiles-api] Upgrade to 4.52.9 (#421)

* [+] release 1.21.0 branch

* mapgl values fix

---------

Co-authored-by: Dmitriy Donov <dmitriy@donov.ru>
Co-authored-by: Voronkov Alexander <voronkov.alexander@gmail.com>
Co-authored-by: Петр Беклемишев <pbekl@bk.ru>
Co-authored-by: Andrey Morozov <62840181+endryhold@users.noreply.github.com>
Co-authored-by: Vladimir Popov <v.popov@2gis.ru>
Co-authored-by: Dmitrii Molochnikov <d.molochnikov@2gis.ru>
Co-authored-by: Artem Malko <a.malko@2gis.ru>
Co-authored-by: Alexander Voronkov <a.voronkov@2gis.ru>
Co-authored-by: i-bogomazov <106957509+i-bogomazov@users.noreply.github.com>
Co-authored-by: vgivanov <v.ivanov@2gis.ru>
Co-authored-by: Dmitry Milov <senneerr@gmail.com>
Co-authored-by: ostrovskiy2gis <108522609+ostrovskiy2gis@users.noreply.github.com>
Co-authored-by: Andrew Mikhailov <Andrew.Mikhailov.18.01@gmail.com>
Co-authored-by: Денис Беляев <d.belyaev@2gis.ru>
Co-authored-by: Kirill Salnikov <k.salnikov@2gis.ru>
Co-authored-by: Власов Сергей Сергеевич <s.vlasov@2Gis.ru>
Ashigo added a commit that referenced this pull request May 3, 2024
* Twins importer (#360)

* Twins importer

* Twins importer add dgctlStorage settings

* Twins importer fix chart

* fix chart

* small fix

* add logs

* [keys-api] up keys-backend. Support truck, map-matching api keys (#386)

* Update citylens to 1.4.0 (#388)

* Update citylens to 1.4.0

* bump to 1.4.1

* Feature: add codeowners file (#383)

* [license] Reworked securityContext for license type 2 (#387)

* [PRO-4361] Обновление pro-api до версии 1.1.79 (#382)

* [PRO-4361] Обновление pro-api до версии 1.1.74

* PRO-4361: update to 1.1.77

* PRO-4371: update to 1.1.78

* PRO-4361: update to 1.1.79

* PRO-4176: add auth scopes (#365)

* PRO-4176: add auth scopes

* PRO-4176: update pro-ui version

* PRO-4176: update app version

* PRO-4176: update to 1.7.5

* navi-castle: bugfix (#393)

- scheduled tasks without PV
- configuration parameters mistaken

* navi-restrictions-api 1.0.1 release (#392)

* navi-restrictions-api 1.0.1 release

* readme upd

* [DEVOPS-982] navi-front: update default locations (#389)

* [DEVOPS-982] navi-front: Add pedestrian location (#396)

* [platform] Upgrade version (#381)

* PRO-3766: update platform to 0.8.1

* Fix Breaking-Changes (#401)

* generic-chart added (#398)

* PRO-3609: upgrade pro ui to 1.9.0 (#399)

* navi-back: upstream update 7.15.2.4 (#402)

* [PRO-4517] обновил хелм-чарт pro-api в onprem до версии 1.5.0 (#394)

* PRO-0: upgrade PRO API to 1.5.1 (#404)

* Feature: add PR template (#410)

* PRO-4774: upgraded PRO API to 1.6.0 (#407)

* [PRO-4635] pro-ui, версия 2.1.1 (#405)

* Citylens 1.6.0 (#406)

* Update citylens to 1.5.0

* Bump appversion

* Fix citylens-api configmap

* Fix license server url template

* Fix review notes

* Update headerLinks

* Add camcom to headerLinks

* Fix page name in configmap

* Bump to 1.5.1

* Bump to 1.6.0

* Update breaking changes & bump app version in chart

* [navi-router] rename router.keyManagementServices to keys (#409)

* [navi-router] Remove trailing dots in param descriptions

* [navi-router] Rename router.keyManagementService -> keys

* [navi-router] Rename router.castleHost -> router.castleUrl

* WAPI-23469 Add configuration for audit logging (#414)

* WAPI-23469 Add configuration for audit logging

* fixes

* upd version tag

* fix duplication key

* [navi-restrictions] update to style guide (#408)

* [tiles-api] Upgrade to 4.52.8 (#416)

* WAPI-23326 Add new s3 params, custom ca configuration (#390)

* WAPI-23326 Add new s3 config params to keys and catalog-api charts

* WAPI-23326 Add custom certs configuration to keys, catalog-api charts

* fix typos

* more fixes

* fix configmap deploy

* split configmap config for jobs and deploys

* upd config for keys

* fix to keys chart

* upd image tags

* upd docs

* Chore: fix PR template (#418)

* [PRO-4789] поправили values для сервиса ключей

* Citylens 1.7.0 (#422)

* Citylens 1.7.0

* bump version to 1.7.1

* Make reporterProTracks deploy one or no replicas

* Bump version to 1.7.2

* [tiles-api] Upgrade to 4.52.9 (#421)

* Чек-лист код-ревью для описания PR (#424)

* [PRO-4789] Change PRO UI to 2.5.1 version and PRO API to 1.11.2 version (#419)

* [PRO-4789] обновил PRO API до версии 1.11.2
Change UI and API Version

* [PRO-4789] поправили values для сервиса ключей

* [PRO-4789] добавлена обязательность параметров kafka.eventsTopic.name и kafka.eventsTopic.readerGroupId

* [PRO-4789] добавлен хук удаления в asset-importer-starter

* WAPI-23416 поддержка tls для коннектов к postgres в catalog (#395)

* WAPI-23417 возможность указать кол-во ретраев на коннект к базе в twi… (#397)

* WAPI-23417 возможность указать кол-во ретраев на коннект к базе в twins api

* WAPI-23417 возможность указать кол-во ретраев на коннект к базе в twins api

* PRO-4936: upgrade keys-ui to 0.7.0 (#425)

* [DEVOPS-1025] Upgrade castle version (#417)

* Update citylens chart to 1.8.0 version (#427)

* Update citylens chart to 1.8.0 version

* Fix typo in pull request template

* Fix kafka.predictorsExtraTopics description

* Simplify chart

* update values

---------

Co-authored-by: Dmitriy Donov <dmitriy@donov.ru>
Co-authored-by: Voronkov Alexander <voronkov.alexander@gmail.com>
Co-authored-by: Петр Беклемишев <pbekl@bk.ru>
Co-authored-by: Andrey Morozov <62840181+endryhold@users.noreply.github.com>
Co-authored-by: Vladimir Popov <v.popov@2gis.ru>
Co-authored-by: Dmitrii Molochnikov <d.molochnikov@2gis.ru>
Co-authored-by: Artem Malko <a.malko@2gis.ru>
Co-authored-by: Alexander Voronkov <a.voronkov@2gis.ru>
Co-authored-by: i-bogomazov <106957509+i-bogomazov@users.noreply.github.com>
Co-authored-by: vgivanov <v.ivanov@2gis.ru>
Co-authored-by: Dmitry Milov <senneerr@gmail.com>
Co-authored-by: ostrovskiy2gis <108522609+ostrovskiy2gis@users.noreply.github.com>
Co-authored-by: Andrew Mikhailov <Andrew.Mikhailov.18.01@gmail.com>
Co-authored-by: Денис Беляев <d.belyaev@2gis.ru>
Co-authored-by: Kirill Salnikov <k.salnikov@2gis.ru>
Co-authored-by: Власов Сергей Сергеевич <s.vlasov@2Gis.ru>
Co-authored-by: Aveldin1 <164148461+Aveldin1@users.noreply.github.com>
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.

5 participants