Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Github-Serivce which alerts a URL when the repo visibility is changed from 'Private' to 'Public' #285

Closed
wants to merge 5 commits into
from

Conversation

Projects
None yet
2 participants

none-da commented Apr 13, 2012

When a repo visibility is changed from Private to Public, this Service Hook fires a POST request to a configured URL. Additionally if a email address is configured, an email will be sent mentioning the action made. Usecases for this include:

  1. Shooting an email about the Open-Sourced event to a common mailing list.
  2. If a enterprise repo is made Public accidentally, a web-service can be setup which uses Github API to flip the repo visibility back to Private.

Nanda Kishore and others added some commits Apr 6, 2012

Functionality related to "Notifying, when a repo visibility is switch…
…ed from Private to Public" is in place. So are the testcases for the total service code.

Testcases related to
Removed the unnecessary testcases. Commented the raise_config_error w…
…hen the POST url webservice call fails, as "Public" Event emailing functionality is also provided and that can still be done.
Member

technoweenie commented Apr 17, 2012

I don't really want to maintain another Service if possible. I think we can add the email capabilities to the Email Service, and start sending the entire Event data instead of just the payload.

Can you get by with just the Email hook for now? We have to do a careful migration with documentation when updating the outgoing Hook payload format.

Member

technoweenie commented Apr 18, 2012

Closing this. I'll see about throwing the email code to the existing email service.

none-da commented Apr 18, 2012

Adding the email capabilities to the existing Web Hook(which listens to even Public events) makes more sense. Doesn't it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment