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

Self-hosted GitLab instance #89

Closed
chris-dura opened this issue Apr 10, 2018 · 3 comments
Closed

Self-hosted GitLab instance #89

chris-dura opened this issue Apr 10, 2018 · 3 comments

Comments

@chris-dura
Copy link
Contributor

chris-dura commented Apr 10, 2018

  • all-contributors-cli version: 4.11.1
  • node version: 8.11.1
  • npm (or yarn) version: 5.8.0

Relevant code or config

// .all-contributorsrc
{
  "projectName": "hub",
  "projectOwner": "prototyping",
  "repoType": "gitlab",
  "repoHost": "https://gitlab.mycompany.com",
  "files": [
    "README.md"
  ],
  "imageSize": 100,
  "commit": false,
  "contributors": []
}

What you did:
I tried to add a contributor.

all-contributors add dura code

What happened:
I got this error:

Cannot read property 'username' of undefined

Reproduction repository:
n/a since we have a private self-hosted gitlab instance...

Problem description:

Changing the repoType, repoHost and username to my public GitHub accounts seems to work, so I suspect it has something to do with the self-hosted repo.
However, you mention changing repoHost to a self hosted repo in the docs... so I'm assuming it is expected to work.

We do have 2FA enabled, could that be the problem?

Suggested solution:

Even with 2FA, we usually use the GitLab API by passing a personal access token.

Perhaps providing an option to do that will suffice?

@machour
Copy link
Collaborator

machour commented Apr 11, 2018

Hi @chris-dura. As far as I remember, the GitLab connector was only tested with the public GitLab.com setup, and authentication is not handled yet.

This definitely needs some more work. Would you be available to work on a PR for that?

@chris-dura
Copy link
Contributor Author

@machour I'll stick it in my wishlist and try to find some time, thanks for the response!

@xuchaoying
Copy link
Contributor

same problem here

xuchaoying added a commit to xuchaoying/all-contributors-cli that referenced this issue May 31, 2018
xuchaoying added a commit to xuchaoying/all-contributors-cli that referenced this issue Jun 1, 2018
chris-dura added a commit to chris-dura/all-contributors-cli that referenced this issue Jul 30, 2018
xuchaoying added a commit to xuchaoying/all-contributors-cli that referenced this issue Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants