Skip to content

Commit

Permalink
https-request: prepare for exporting more than a single function
Browse files Browse the repository at this point in the history
Signed-off-by: Johannes Schindelin <[email protected]>
  • Loading branch information
dscho committed Aug 22, 2023
1 parent 3a9fc91 commit 1dce392
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 8 deletions.
8 changes: 4 additions & 4 deletions GitForWindowsHelper/azure-pipelines.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const triggerAzurePipeline = async (context, token, organization, project, build
'parameters': JSON.stringify(parameters),
}

const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
return await httpsRequest(
context,
'dev.azure.com',
Expand Down Expand Up @@ -49,7 +49,7 @@ const listReleases = async (context, token, organization, project) => {
'Authorization': 'Basic ' + auth,
}

const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
return await httpsRequest(
context,
'vsrm.dev.azure.com',
Expand All @@ -67,7 +67,7 @@ const getRelease = async (context, token, organization, project, releaseId) => {
'Authorization': 'Basic ' + auth,
}

const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
return await httpsRequest(
context,
'vsrm.dev.azure.com',
Expand Down Expand Up @@ -120,7 +120,7 @@ const createRelease = async (
properties: { ReleaseCreationSource: "GitForWindowsHelper" },
}

const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
return await httpsRequest(
context,
'vsrm.dev.azure.com',
Expand Down
2 changes: 1 addition & 1 deletion GitForWindowsHelper/github-api-request-as-app.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ module.exports = async (context, requestMethod, requestPath, body) => {

const token = `${headerAndPayload}.${signature}`

const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
return await httpsRequest(
context,
null,
Expand Down
2 changes: 1 addition & 1 deletion GitForWindowsHelper/github-api-request.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
module.exports = async (context, token, method, requestPath, payload) => {
const httpsRequest = require('./https-request')
const { httpsRequest } = require('./https-request')
const headers = token ? { Authorization: `Bearer ${token}` } : null
const answer = await httpsRequest(context, null, method, requestPath, payload, headers)
if (answer.error) throw answer.error
Expand Down
6 changes: 5 additions & 1 deletion GitForWindowsHelper/https-request.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
const gently = require('./gently')

module.exports = async (context, hostname, method, requestPath, body, headers) => {
const httpsRequest = async (context, hostname, method, requestPath, body, headers) => {
headers = {
'User-Agent': 'GitForWindowsHelper/0.0',
Accept: 'application/json',
Expand Down Expand Up @@ -60,3 +60,7 @@ module.exports = async (context, hostname, method, requestPath, body, headers) =
}
})
}

module.exports = {
httpsRequest
}
2 changes: 1 addition & 1 deletion GitForWindowsHelper/search.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
* https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests
*/
const searchIssues = async (context, searchTerms) => {
const httpsRequest = require('./https-request');
const { httpsRequest } = require('./https-request');
const answer = await httpsRequest(
context,
'api.github.com',
Expand Down

0 comments on commit 1dce392

Please sign in to comment.