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
Support both Bitbucket Cloud and Server #3030
Support both Bitbucket Cloud and Server #3030
Conversation
1095868
to
f95f5e6
Compare
Hey @goober! Thanks for the contribution and welcome to the community! Ideally, it would be great if we could wait for your upstream changes to be merged first rather than using your own fork. There's also some conflicts that could do with fixing too! Thanks for your work! 🏅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool stuff!
@benjdlambert Thanks. Its a fast moving project.. I will rebase against master once again. It seems that the upstream changes are about to be merged so we can definitely wait with merging this until they have released the new version. I just wanted to make my changes available so people who wants to take it for a spin is able to, and to get some feedback |
After some further investigation it seems like the Bitbucket cloud api requires the username/appPassword combo for authentication, but also support an oauth2 setup. However, since the two apis are so different maybe it is better to split them into two readers? Keep the current one and add a new BitbucketServerUrlReader @freben @Rugvip what do you think? |
f95f5e6
to
166b3c1
Compare
@goober configuration-wise I'd like to keep them in one reader, but we could split logic into two helper classes or smth of that sort. I think the GitHub one uses a similar pattern. Regarding OAuth that may not support our use-case, as this is service to service authentication. Depends on what they implement though. |
166b3c1
to
6579769
Compare
e8f11ab
to
15b57a1
Compare
@goober thank you for add support for BitBucket Server. I've tested your branch using a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
@spotify/techdocs-core, could we also get a +1 :) |
Hi, is there an example of the what the auth or app-config should look like for the bitbucket.org server? thanks |
Hey @JdataNich, you might get more 👀 eyes on your question if you bring it up in the #support channel in the Backstage discord. |
Hey, I just made a Pull Request!
This pull request will add support for fetching files from Bitbucket Server.
Changes were required in an underlying library so merging this is still blocked by IonicaBizau/git-url-parse#113Fixes #2735
✔️ Checklist