taskcluster / ec2-manager

Mozilla Public License 2.0
2 stars 14 forks source link

Switch terminateInstance to a post method #42

Closed helfi92 closed 6 years ago

helfi92 commented 6 years ago

Per our agreement in RFC#97, actions will be sent using a POST request.

helfi92 commented 6 years ago

@jhford I don't seem to have enough permission on this repo to add you as a reviewer. Could you please take a look and merge if OK. Thanks.

@djmitche Would it be possible to grant me more permission on this repo so I could at least add reviewers to future PRs :)

jhford commented 6 years ago

@helfi92 fyi, this repo is locked down because of the ucosp students. you should already be on the team of people who can do reviews.

jhford commented 6 years ago

@helfi92 I'm going to close this PR because we can't really do this. Actions should be Method + URL, so fixing this wouldn't actually require any changes to ec2-manager.