Comments (3)
Seems like I can hook in with WithClientMiddleware
- can I return my own configured transport from this function?
from go-githubapp.
This is not currently supported (other than via system wide mechanisms like SSL_CERT_FILE
and SSL_CERT_DIR
), but I agree that it makes sense. In #92, we discussed adding a WithHTTPClient
option, but ended up not needing it for the feature there. This seems like a good justification for adding one (or a more limited WithTLSConfig
.)
While you could maybe work around this using WithClientMiddleware
, you'll have to be careful to still delegate to the incoming http.RoundTripper
. If you return your own configured transport and ignore the input, you'll remove the GitHub authentication and other helpers provided by this library.
I might have some time today to add the mentioned option function.
from go-githubapp.
This is not currently supported (other than via system wide mechanisms like
SSL_CERT_FILE
andSSL_CERT_DIR
), but I agree that it makes sense. In #92, we discussed adding aWithHTTPClient
option, but ended up not needing it for the feature there. This seems like a good justification for adding one (or a more limitedWithTLSConfig
.)
withTlsConfig would suffice and makes sense, it also leaves things better encapsulated than messing around and the comms-level which feels a bit low.
While you could maybe work around this using
WithClientMiddleware
, you'll have to be careful to still delegate to the incominghttp.RoundTripper
. If you return your own configured transport and ignore the input, you'll remove the GitHub authentication and other helpers provided by this library.I might have some time today to add the mentioned option function.
That would be awesome and highly appreciated.
from go-githubapp.
Related Issues (20)
- Upgrade to v2 of github.com/alexedwards/scs HOT 7
- can't use app authentication with v4 client HOT 6
- Context Deadline Exceeded in Google Cloud Run HOT 4
- avoid jwt-go HOT 3
- Consider defining new module in `example` package HOT 1
- Exposing metrics via HTTP HOT 2
- 404 page not found HOT 2
- Configure redirect_uri for token request HOT 2
- Example app clarification HOT 4
- PR #212 has broken the example code HOT 3
- Pass token to net/http for asset download HOT 6
- Errors loading config larger than 1MB HOT 2
- Capture stack traces for recovered handler panics
- Add GraphQL query logging middleware
- Emit metrics for async webhook processing errors
- Duplicate request with same delivery ID HOT 2
- Upgrade to be compatible with github.com/google/go-github/v33 HOT 1
- PrepareRepoContext should take an interface
- Updating to go-github/v38/github HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from go-githubapp.