Remove requestOptions2GotOptions compatibility layer (#7270)
* gzip --> decompress * strictSSL --> https.rejectUnauthorized * auth --> username/password * qs --> searchParams * fix base service auth docs * completely remove requestOptions2GotOptions layer * update the docs Co-authored-by: repo-ranger[bot] <39074581+repo-ranger[bot]@users.noreply.github.com>
This commit is contained in:
@@ -40,7 +40,7 @@ export default class AzureDevOpsBase extends BaseJsonService {
|
||||
// Microsoft documentation: https://docs.microsoft.com/en-us/rest/api/azure/devops/build/builds/list?view=azure-devops-rest-5.0
|
||||
const url = `https://dev.azure.com/${organization}/${project}/_apis/build/builds`
|
||||
const options = {
|
||||
qs: {
|
||||
searchParams: {
|
||||
definitions: definitionId,
|
||||
$top: 1,
|
||||
statusFilter: 'completed',
|
||||
@@ -49,7 +49,7 @@ export default class AzureDevOpsBase extends BaseJsonService {
|
||||
}
|
||||
|
||||
if (branch) {
|
||||
options.qs.branchName = `refs/heads/${branch}`
|
||||
options.searchParams.branchName = `refs/heads/${branch}`
|
||||
}
|
||||
|
||||
const json = await this.fetch({
|
||||
|
||||
@@ -104,7 +104,7 @@ export default class AzureDevOpsBuild extends BaseSvgScrapingService {
|
||||
// Microsoft documentation: https://docs.microsoft.com/en-us/rest/api/vsts/build/status/get
|
||||
const { status } = await fetch(this, {
|
||||
url: `https://dev.azure.com/${organization}/${projectId}/_apis/build/status/${definitionId}`,
|
||||
qs: {
|
||||
searchParams: {
|
||||
branchName: branch,
|
||||
stageName: stage,
|
||||
jobName: job,
|
||||
|
||||
@@ -101,7 +101,7 @@ export default class AzureDevOpsCoverage extends AzureDevOpsBase {
|
||||
// Microsoft documentation: https://docs.microsoft.com/en-us/rest/api/azure/devops/test/code%20coverage/get%20build%20code%20coverage?view=azure-devops-rest-5.0
|
||||
const url = `https://dev.azure.com/${organization}/${project}/_apis/test/codecoverage`
|
||||
const options = {
|
||||
qs: {
|
||||
searchParams: {
|
||||
buildId,
|
||||
'api-version': '5.0-preview.1',
|
||||
},
|
||||
|
||||
@@ -15,12 +15,15 @@ const schema = Joi.object({
|
||||
.required(),
|
||||
}).required()
|
||||
|
||||
async function fetch(serviceInstance, { url, qs = {}, errorMessages }) {
|
||||
async function fetch(
|
||||
serviceInstance,
|
||||
{ url, searchParams = {}, errorMessages }
|
||||
) {
|
||||
// Microsoft documentation: https://docs.microsoft.com/en-us/rest/api/vsts/build/status/get
|
||||
const { message: status } = await serviceInstance._requestSvg({
|
||||
schema,
|
||||
url,
|
||||
options: { qs },
|
||||
options: { searchParams },
|
||||
errorMessages,
|
||||
})
|
||||
return { status }
|
||||
|
||||
@@ -160,7 +160,7 @@ export default class AzureDevOpsTests extends AzureDevOpsBase {
|
||||
return await this.fetch({
|
||||
url: `https://dev.azure.com/${organization}/${project}/_apis/test/ResultSummaryByBuild`,
|
||||
options: {
|
||||
qs: { buildId },
|
||||
searchParams: { buildId },
|
||||
},
|
||||
schema: buildTestResultSummarySchema,
|
||||
errorMessages,
|
||||
|
||||
Reference in New Issue
Block a user