carocad commented on issue #7435:
URL: 
https://github.com/apache/incubator-devlake/issues/7435#issuecomment-2133677199

   @klesh we continued to follow up on this and now I am 99% sure that this is 
an issue on devlake. The issue is that for Github Servers it is required to set 
the endpoint URL. This generally ends in `/api/v3/{{suffix}}` as per Rest guide 
of Github. Unfortunately the graphql endpoint doesn't follow this convention. 
So its endpoint is `/api/graphql` (notice the missing `/v3/`).
   
   This line 
[here](https://github.com/apache/incubator-devlake/blob/main/backend/plugins/github_graphql/impl/impl.go#L207).
 Assumes that the graphql suffix can be added to previously defined endpoint 
URL resulting in `/api/v3/graphql` which doesn't exists and therefore results 
in a 404 error --> panic. Since graphql doesn't follow the convention of 
suffix, simply harcoding it to `/api/graphql` should solve the issue. Please 
let me know if you would like me to make a PR for it or if you prefer to do it 
yourselves. Either way works for me.
   
   
   <img width="818" alt="image" 
src="https://github.com/apache/incubator-devlake/assets/10408729/b2bc245d-fda1-4796-a6ca-73b6d6ff2c08";>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@devlake.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to