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
Implement streaming support from bitbucket. #378
Conversation
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.
Can we also remove some of the code we have for paging through the file contents with this change?
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClient.java
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClient.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketRequestExecutor.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/HttpRequestExecutor.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClient.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClientImpl.java
Outdated
Show resolved
Hide resolved
Fixes JENKINS-66829, I think. |
…NextPageFetcher, BitbucketFilePage objects
…NextPageFetcher, BitbucketFilePage objects
…rsion. Addresses review comments
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClient.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/BitbucketFilePathClientImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/HttpRequestExecutor.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/client/HttpRequestExecutor.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/http/HttpRequestExecutorImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/http/HttpRequestExecutorImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/com/atlassian/bitbucket/jenkins/internal/http/HttpRequestExecutorImpl.java
Outdated
Show resolved
Hide resolved
…version, inline private method
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.
I'm going to hit approve, as I think this is there, it is just the unused consumer
left, but happy for us to fix that as a separate PR if we want to, it is a trivial change.
* | ||
* @param url The URL to hit on bitbucket server end | ||
* @param additionalConfig additional configuration, such as credentials, that will be used in making calls | ||
* @param consumer on successful execution, {@link Response} will be passed to consumer |
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.
consumer
is unused, and should be removed I think. It doesn't quite make sense to both return an InputStream
AND take in a consumer
This PR seeks to introduce support for the plugin to retrieve files from bitbucket using the streaming endpoint provided to us. The caller is then presented with an InputStream that they may use.