-
Notifications
You must be signed in to change notification settings - Fork 133
Open
Description
Code below needs to also pass in config.client_id as it is mandatory
app.get('/github/auth', function(req,res){
// redirect the user to github authorization url
return res.redirect(config.authorize_url);
});
I have changed my sample to be like..
app.get('/github/auth', function(req,res){
// redirect the user to github authorization url
console.log("In github/auth , redirecting to " + client.redirect_url );
return res.redirect(`${config.authorize_url}?client_id=${config.client_id}?redirect_url=${client.redirect_url}`);
});
This is based on the github documentation here
https://developer.github.com/enterprise/2.19/apps/building-oauth-apps/authorizing-oauth-apps/#web-application-flow
which calls out that client_id is mandatory
Metadata
Metadata
Assignees
Labels
No labels