Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

EVEREST-486: Zitadel integration #269

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft

EVEREST-486: Zitadel integration #269

wants to merge 15 commits into from

Conversation

michal-kralik
Copy link
Contributor

@michal-kralik michal-kralik commented Oct 23, 2023 •
edited by pull-request-badge bot
Loading

EVEREST-486 Powered by Pull Request Badge

CHANGE DESCRIPTION

Problem:
EVEREST-527
EVEREST-528
EVEREST-530

Short explanation of the problem.

Related pull requests

  • [link]

Cause:
Short explanation of the root cause of the issue if applicable.

Solution:

  • During boot we ensure Zitadel is properly configured
  • Everest proxies API requests to Zitadel and adds authentication
  • Everest API is protected by authentication
  • /v1/public/configuration endpoint is available without authentication and provides general inforrmation

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?

Tests

  • Is an Integration test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@recharte recharte Awaiting requested review from recharte recharte will be requested when the pull request is marked ready for review recharte is a code owner

@oksana-grishchenko oksana-grishchenko Awaiting requested review from oksana-grishchenko oksana-grishchenko will be requested when the pull request is marked ready for review oksana-grishchenko is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@michal-kralik