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

conventional-github-releaser 3.1.2 issue #93

Open
AlecRust opened this issue Jul 5, 2018 · 2 comments
Open

conventional-github-releaser 3.1.2 issue #93

AlecRust opened this issue Jul 5, 2018 · 2 comments
Labels
platform:github Issues related to the GitHub platform. type::defect Functionality not meeting the documented behavior for that project. type::help wanted We need the community's help to address this issue through the contribution of a merge request.

Comments

@AlecRust
Copy link

AlecRust commented Jul 5, 2018

In conventional-github-releaser versions below 3.1.2 my gulp release script works correctly:

function createRelease(done) {
  githubReleaser(
    {
      type: 'oauth',
      token: process.env.CONVENTIONAL_GITHUB_RELEASER_TOKEN
    },
    done
  )
}

With version 3.1.2 though I get:

[18:58:56] Starting 'createRelease'...
[18:58:56] 'createRelease' errored after 255 ms
[18:58:56] RequestError: getaddrinfo ENOTFOUND undefinedrepos undefinedrepos:443
    at ClientRequest.req.once.err (/Users/alec/projects/my-site/node_modules/got/index.js:182:22)
    at Object.onceWrapper (events.js:272:13)
    at ClientRequest.emit (events.js:180:13)
    at ClientRequest.emit (domain.js:421:20)
    at TLSSocket.socketErrorListener (_http_client.js:395:9)
    at TLSSocket.emit (events.js:180:13)
    at TLSSocket.emit (domain.js:421:20)
    at emitErrorNT (internal/streams/destroy.js:64:8)
    at process._tickCallback (internal/process/next_tick.js:114:19)
[18:58:56] 'release' errored after 5.73 s
@tinesoft
Copy link

Hi,

i have a similar issue. I found out it is due to a bug in the way the dependency package gh-got (before v.8.0.0) merges its options with defaults ( via Object.assign() see https://github.com/sindresorhus/gh-got/blob/69d1fec99cfdb45450e2626075c87ca2347d7e3a/index.js#L12). Basically, values like endpoint remain undefined after the merging. This seems to be addressed in latest version of gh-got (v8.0.0)

In the meantime, adding url: 'https://api.github.com/' to the auth param of conventionalGithubReleaser() solved the problem for me, because the value endpoint is no longer undefined

@hutson hutson added type::defect Functionality not meeting the documented behavior for that project. type::help wanted We need the community's help to address this issue through the contribution of a merge request. labels May 28, 2019
@hutson
Copy link
Contributor

hutson commented May 28, 2019

@AlecRust thank you for reporting this issue.

@tinesoft thank you for diving into this issue and providing an explanation as to the cause.

I'm not sure when I'll have the time to address this issue in the GitHub releaser, so I've marked the issue as help wanted in hopes the community is able and willing to contribute a fix sooner.

@hutson hutson added the platform:github Issues related to the GitHub platform. label May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform:github Issues related to the GitHub platform. type::defect Functionality not meeting the documented behavior for that project. type::help wanted We need the community's help to address this issue through the contribution of a merge request.
Development

No branches or pull requests

3 participants