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

feat: Bitbucket Adapter #115

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

Conversation

jonathanbecerra
Copy link
Contributor

Bitbucket Adapter

  • Integrated Bitbucket adapter using Bitbucket's authentication documentation.
  • There are many scopes that can be added. Refer to Bitbucket OAuth 2.0 scopes for the actual list of scopes supported by oauth2.
    • Keep in mind they do mention in the docs OAuth 2.0 documentation the following statement:

      Our OAuth 2 implementation is merged in with our existing OAuth 1 in such a way that existing OAuth 1 consumers automatically become valid OAuth 2 clients. The only thing you need to do is edit your existing consumer and configure a callback URL.

      Therefore, using the endpoints for oauth2 should not affect anything.

  • Overrode the scopes[] to use a union for type safety. Same as the SlackAdapter (GitHub Pull Request #101).

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.

1 participant