Handle 401, 403 and 404 status codes when hitting GitHub for webhook

[humitos]

This is a work in progress PR.

All of our integrations are not handling these error codes: 401, 403 and 404. All of them are being logged as errors in our application. This PR handle them to log as info instead of an error (via the else: clause).

Also, this code may need to be repeated across all the integrations and twice on each. So, maybe it's possible to refactor this all together: https://github.com/rtfd/readthedocs-corporate/issues/372

[ericholscher]

Looks reasonable, what else is needed here?

[humitos]

Changes are good, but I did them only for GitHub in this PR. I need to extend this to all the services and because of this, I wanted to refactor a little those files to share more code.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[ericholscher]

@humitos merge this now, and create an issue for the others?

[humitos]

You say it, you have it! :)

Issue to track the rest of this work at #5092

[codecov]

Codecov Report

Merging #4805 into master will decrease coverage by 0.63%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4805      +/-   ##
==========================================
- Coverage   76.87%   76.23%   -0.64%     
==========================================
  Files         158      158              
  Lines        9997    10037      +40     
  Branches     1255     1268      +13     
==========================================
- Hits         7685     7652      -33     
- Misses       1980     2042      +62     
- Partials      332      343      +11

Impacted Files	Coverage Δ
readthedocs/oauth/services/github.py	42.76% <0%> (-0.83%)	⬇️
readthedocs/vcs_support/backends/svn.py	27.69% <0%> (-11.71%)	⬇️
readthedocs/vcs_support/backends/hg.py	59.09% <0%> (-8.61%)	⬇️
readthedocs/builds/forms.py	87.87% <0%> (-7.96%)	⬇️
readthedocs/restapi/client.py	82.35% <0%> (-5.53%)	⬇️
readthedocs/core/utils/__init__.py	75% <0%> (-5.42%)	⬇️
readthedocs/builds/views.py	85.71% <0%> (-5.03%)	⬇️
readthedocs/restapi/views/integrations.py	87.23% <0%> (-4.72%)	⬇️
readthedocs/projects/utils.py	54.54% <0%> (-4.55%)	⬇️
readthedocs/builds/models.py	75.98% <0%> (-2.85%)	⬇️
... and 43 more

[codecov]

Codecov Report

Merging #4805 into master will decrease coverage by 0.63%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4805      +/-   ##
==========================================
- Coverage   76.87%   76.23%   -0.64%     
==========================================
  Files         158      158              
  Lines        9997    10037      +40     
  Branches     1255     1268      +13     
==========================================
- Hits         7685     7652      -33     
- Misses       1980     2042      +62     
- Partials      332      343      +11

Impacted Files	Coverage Δ
readthedocs/oauth/services/github.py	42.76% <0%> (-0.83%)	⬇️
readthedocs/vcs_support/backends/svn.py	27.69% <0%> (-11.71%)	⬇️
readthedocs/vcs_support/backends/hg.py	59.09% <0%> (-8.61%)	⬇️
readthedocs/builds/forms.py	87.87% <0%> (-7.96%)	⬇️
readthedocs/restapi/client.py	82.35% <0%> (-5.53%)	⬇️
readthedocs/core/utils/__init__.py	75% <0%> (-5.42%)	⬇️
readthedocs/builds/views.py	85.71% <0%> (-5.03%)	⬇️
readthedocs/restapi/views/integrations.py	87.23% <0%> (-4.72%)	⬇️
readthedocs/projects/utils.py	54.54% <0%> (-4.55%)	⬇️
readthedocs/builds/models.py	75.98% <0%> (-2.85%)	⬇️
... and 43 more