With the API GW, such duplications, or modifications can be defined in the GW, if required, instead of in TO
On Thu, Mar 16, 2017, 5:52 PM Jan van Doorn <[email protected]> wrote: > We should also think about the API gateway future.... I think with that, we > don't need these special routes at all anymore, right Amir? > > Cheers, > JvD > > > On Wed, Mar 15, 2017 at 9:24 PM Dewayne Richardson <[email protected]> > wrote: > > > I think we should do as Dave mentioned, assess and rename. > > > > > On Mar 15, 2017, at 2:18 PM, Jeremy Mitchell <[email protected]> > > wrote: > > > > > > I don't like duplicating routes either but I thought it would ease the > > > transition rather than just changing the route. So no code duplication, > > > just 2 routes that go to the same place: > > > > > > $r->get("/internal/api/$version/steering")->over( authenticated => 1 > > )->to( > > > 'Steering#index', namespace => 'API::DeliveryService' ); > > > $r->get("/api/$version/steering")->over( authenticated => 1 )->to( > > > 'Steering#index', namespace => 'API::DeliveryService' ); > > > > > > And then we circle back and delete > > > > > > $r->get("/internal/api/$version/steering")->over( authenticated => 1 > > )->to( > > > 'Steering#index', namespace => 'API::DeliveryService' ); > > > > > > at some point. > > > > > > And yes, this internal namespace was introduced for comcast-specific > > > reasons that I believe no longer exist. > > > > > > Jeremy > > > > > > > > > > > > On Wed, Mar 15, 2017 at 2:13 PM, David Neuman > > > wrote: > > > > > > > At least a few of those (Steering, federations) were put in the > > "internal" > > > > namespace to work around Comcast specific issues. I don't know that I > > like > > > > the idea of duplicating routes, if anything we should see what is > > impacted > > > > by moving them out of the internal namespace. > > > > > > > > On Wed, Mar 15, 2017 at 1:30 PM, Jeremy Mitchell > > > > wrote: > > > > > > > > > Currently, we have a number of API routes scoped as "internal". > Here > > are > > > > a > > > > > few examples: > > > > > > > > > > https://github.com/apache/incubator-trafficcontrol/blob/ > > > > > master/traffic_ops/app/lib/TrafficOpsRoutes.pm#L516 > > > > > > > > > > I believe this is going to make it more difficult as we try to > > implement > > > > > more granular roles / capabilities coupled with tenancy. > > > > > > > > > > So I'm proposing that we create a duplicate non-internal route like > > this, > > > > > for example: > > > > > > > > > > $r->get("/api/$version/steering")->over( authenticated => 1 )->to( > > > > > 'Steering#index', namespace => 'API::DeliveryService' ); > > > > > > > > > > that way we can slowly move away from the "internal" routes and > > > > eventually > > > > > deprecate them. > > > > > > > > > > I think with our upcoming more robust role / tenancy model, there > is > > no > > > > > longer a need for "internal". > > > > > > > > > > Thoughts? > > > > > > > > > > Jeremy > > > > > > > > > >
