-
Notifications
You must be signed in to change notification settings - Fork 10
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
Improve server debugging experience #24
Comments
The first should be much easier, once we complete the message bus work. The second we can do by making a POST to the webhook URL with the following format data:
We can write a script with a signature like |
I think we’ll need more header information when tying to debug this in the past. |
Also, I’m not sure how we’re going to redirect the Webhook calls from |
The README explains how to add a webhook:
|
Can you explain what you mean by this? |
Sorry, my tenses got messed up. When I was trying to mock up these requests with Katy last year we needed more header information from the GitHub webhook for the request to work properly. I don’t remember what that information was off the top of my head. |
You can see the code that handles the incoming request here: https://github.com/scipy-conference/procbuild/blob/master/procbuild/server.py#L249 It only reads the fields I mentioned. |
Right now it's not straight-forward to debug the server, because the easiest way to actually deposit a payload to a running server is to actually create a new PR that meets the
Paper:
naming criteria on scipy-conference/scipy_proceedings.There are a number of things to improve:
The text was updated successfully, but these errors were encountered: