github-tools / github-tools/github

Supporting github enterprise

Open
#322 8 comments 0 reactions 0 assignees View on GitHub
needs clarification
Dominant language
JavaScript
Stars
3.8k
Forks
809
PR merge metrics
No merged PRs in 30d

Description

Of all the github api libs, I find this one to be the most well written. I've been using it for working with gihub enterprise and it works fine for most gets.

However, it fails on POST/PUT where:
- SSL errors crop up due to enterprise being signed by an internal certificate signing authority
- no ability to pass extra headers

Above 2 can be solved if there was a way to pass on additional headers for the request. Currently I'm using request-promise and passing the CA chain as well as custom headers when POST/PUT are needed but this is sub optimal solution.

I'm using this in a Node (v4.4.3 and above) app.

Contributor guide

No contributing guide indexed for this repository

Research direction

No files or tests are named. Start by tracing how POST and PUT requests are constructed, then review the existing request-promise usage and how request options could carry a CA chain and additional headers; done means Enterprise POST/PUT requests can use both without the reported SSL failures.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
api
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.