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.

Sign up for GitHub

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

Jump to bottom

Auth flow #58

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Auth flow #58

wants to merge 4 commits into from

Conversation

girish17
Copy link
Owner

  1. added new integration tests
  2. created /auth endpoint for oAuth flow

@girish17 girish17 self-assigned this Aug 27, 2023
This was linked to issues Aug 27, 2023
Integration tests needed for routes.js #7
Open
Implementation of oAuth flow for the integration as described here - https://www.openproject.org/docs/api/introduction/#oauth20-authentication #45
Open
@VincentSC
Copy link

What I see is that the environment is not changed for the tests, and thus these fail.

Invalid options provided to simple-oauth2 "auth.tokenHost" is required

Or is there another reason why this PR is stalled? Can I help?

@girish17
Copy link
Owner Author

girish17 commented Feb 2, 2024

Hi @VincentSC, thanks for taking note of the issue. You're welcome to help and make changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@girish17 girish17

Labels
None yet
Projects
op-mattermost
Status: In progress
Milestone
No milestone
2 participants
@girish17 @VincentSC