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

IGitApi.getBlobsZip Fails. Sends GET request instead of POST #615

Open
cjh15hub opened this issue Nov 7, 2024 · 4 comments · May be fixed by #618
Open

IGitApi.getBlobsZip Fails. Sends GET request instead of POST #615

cjh15hub opened this issue Nov 7, 2024 · 4 comments · May be fixed by #618

Comments

@cjh15hub
Copy link

cjh15hub commented Nov 7, 2024

Environment

Node version: v18.20.4
Npm version: 10.7.0
OS and version: Windows 11
azure-devops-node-api version: 7.0.0

Issue Description

Duplicate of unresolved, stale issue 362

IGitAPI.getBlobsZip should send a POST request according to Rest API documentation (https://docs.microsoft.com/en-us/rest/api/azure/devops/git/blobs/get%20blobs%20zip) but sends GET request instead:

azure-devops-node-api/api/GitApi.ts

Expected behavior

IGitAPI.getBlobsZip should return a Promise<NodeJS.ReadableStream> transferring the requested zip.

Actual behavior

The returned data is: "{"count":1,"value":{"Message":"The requested resource does not support http method 'GET'."}}"

Steps to reproduce

  1. Instantiate a connection to the IGitAPI
    const gitAPI = await connection.getGitApi();
  2. Request ZIP file
    gitAPI.getBlobsZip(blobdIds, repositoryId, projectId);
  3. Stream response to log.txt
  4. Check content of log.txt
@ivanduplenskikh
Copy link
Contributor

Hi @cjh15hub, thanks for reporting! We are working on more prioritized issues at the moment, but will get back to this one soon.

@cjh15hub
Copy link
Author

cjh15hub commented Nov 9, 2024

Okay, thank you. I'll try to have this thread stay active so this doesn't get lost again.

@cjh15hub
Copy link
Author

Still experiencing this issue, and want to keep the thread alive.

@ivanduplenskikh ivanduplenskikh linked a pull request Nov 25, 2024 that will close this issue
@cjh15hub
Copy link
Author

cjh15hub commented Dec 4, 2024

Happy to see that a Pull Request is associated with the reported issue. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants