inch-ci / inch_ci-web

Web frontend for Inch CI
http://inch-ci.org/
MIT License
139 stars 20 forks source link

git commits no longer trigger automatic doc check #168

Closed sagiegurari closed 7 years ago

sagiegurari commented 7 years ago

It used to work, maybe with the updated site it stopped. basically now I have to run it manually.

sagiegurari commented 7 years ago

any updates on this one?

rrrene commented 7 years ago

@sagiegurari Hi, sorry for not replying. I checked this when you first reported it, but to me it seems that all GitHub web hooks are executed as expected. Did I misunderstand something?

sagiegurari commented 7 years ago

So for me the githooks are not running or not triggering the inch build. This started after the new website change. anything I need to do on my end to make it work again?

sagiegurari commented 7 years ago

got into a repo webhooks and I see that the inch webhook returns 301 responses (redirect) http://inch-ci.org/rebuild so it is failing.

sagiegurari commented 7 years ago

ok so I deleted the webhooks but i'm unable to add it via inch website, keeps telling me something is wrong. Added it manually and now it works. So i'll have to do it manually for all repos but this is something you should check.

sagiegurari commented 7 years ago

dug into this a little bit more and the issue is that the old webhook was http and the new is https. when going to http I guess you are now redirecting to https and github doesn't follow redirects (which i think is ok from their part).

rrrene commented 7 years ago

Excellent analysis, that fixed it. Thanks for digging so deep! :+1: