[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-21 Thread amiryesh
Github user amiryesh commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/567 @dewrich Hi, Thanks for your comments. Regarding better error handling, I my tests I do get a 502 response in this case. Can you share your setup details to help me

[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-20 Thread amiryesh
Github user amiryesh commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/567 @limited Right --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-20 Thread amiryesh
Github user amiryesh commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/567 @limited Hi, I agree with your view on the benefits of using industry standard software wherever applicable. Let me share my view on how the API GW and the auth server

[GitHub] incubator-trafficcontrol issue #567: API GW phase 0 (replaces #551, depends ...

2017-06-19 Thread amiryesh
Github user amiryesh commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/567 Hi, glad to hear you are taking the effort of using the GW. Did you get to see webfront/README.md or auth/README.md? The auth server is responsible for login. The auth

[GitHub] incubator-trafficcontrol pull request #567: [New PR] API GW phase 0 (depends...

2017-05-11 Thread amiryesh
GitHub user amiryesh opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/567 [New PR] API GW phase 0 (depends on PR #544) This replaces PR #551 that was closed by mistake. First operational version of API GW You can merge this pull request into a Git

[GitHub] incubator-trafficcontrol pull request #551: API GW phase 0 (depends on PR #5...

2017-05-10 Thread amiryesh
Github user amiryesh closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/551 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafficcontrol pull request #551: API GW phase 0 (depends on PR #5...

2017-05-09 Thread amiryesh
Github user amiryesh commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/551#discussion_r115421714 --- Diff: traffic_ops/experimental/webfront/webfront.go --- @@ -122,188 +145,311 @@ func main() { Logger.Fatal

[GitHub] incubator-trafficcontrol pull request #551: API GW phase 0 (depends on PR #5...

2017-05-09 Thread amiryesh
Github user amiryesh commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/551#discussion_r115421632 --- Diff: traffic_ops/experimental/webfront/webfront.go --- @@ -122,188 +145,311 @@ func main() { Logger.Fatal

[GitHub] incubator-trafficcontrol pull request #551: API GW phase 0 (depends on PR #5...

2017-05-09 Thread amiryesh
Github user amiryesh commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/551#discussion_r115421286 --- Diff: traffic_ops/experimental/auth/auth.go --- @@ -132,83 +150,197 @@ func InitializeDatabase(username, password, dbname, server

[GitHub] incubator-trafficcontrol pull request #551: API GW phase 0 (depends on PR #5...

2017-05-07 Thread amiryesh
GitHub user amiryesh opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/551 API GW phase 0 (depends on PR #544) First operational version of API GW You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-trafficcontrol issue #313: API GW (initial)

2017-02-27 Thread amiryesh
Github user amiryesh commented on the issue: https://github.com/apache/incubator-trafficcontrol/pull/313 1. I'd keep the key and pem files in the repo just to have a quick start. they can even be used in testing environments. of course, they will not be used in a production env. what

[GitHub] incubator-trafficcontrol pull request #313: API GW (initial)

2017-02-26 Thread amiryesh
GitHub user amiryesh opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/313 API GW (initial) Initial implementation of API GW, and authentication service, based on https://github.com/rarenivar/project5799 The auth service authenticates a user

[GitHub] incubator-trafficcontrol pull request #310: Api gw

2017-02-26 Thread amiryesh
Github user amiryesh closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/310 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafficcontrol pull request #310: Api gw

2017-02-26 Thread amiryesh
GitHub user amiryesh reopened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/310 Api gw Adding API GW (webfront) and a simple authentication service (auth) under traffic_ops/experimental. The auth server authenticates a login request against

[GitHub] incubator-trafficcontrol pull request #310: Api gw

2017-02-24 Thread amiryesh
Github user amiryesh closed the pull request at: https://github.com/apache/incubator-trafficcontrol/pull/310 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-trafficcontrol pull request #310: Api gw

2017-02-24 Thread amiryesh
GitHub user amiryesh opened a pull request: https://github.com/apache/incubator-trafficcontrol/pull/310 Api gw Adding API GW (webfront) and a simple authentication service (auth) under traffic_ops/experimental. The auth server authenticates a login request against

[GitHub] incubator-trafficcontrol pull request #240: [TC-99] api/{version}/cdns/:cdn_...

2017-02-01 Thread amiryesh
Github user amiryesh commented on a diff in the pull request: https://github.com/apache/incubator-trafficcontrol/pull/240#discussion_r98848532 --- Diff: traffic_ops/app/lib/MojoPlugins/Health.pm --- @@ -159,6 +159,9 @@ sub register { cachegroups