[
https://issues.apache.org/jira/browse/TC-486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16109243#comment-16109243
]
ASF GitHub Bot commented on TC-486:
-----------------------------------
Github user dneuman64 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/770#discussion_r130659120
--- Diff: traffic_ops/app/lib/API/DeliveryService/SteeringTarget.pm ---
@@ -134,7 +134,7 @@ sub create {
my $steering_ds_id = $self->param('id');
my $target_ds_id = $params->{targetId};
- if ( !&is_admin($self) ) {
+ if ( !&is_admin($self) && !&is_steering($self) ) {
--- End diff --
should this be an OR not an AND?
> Add steering role to steering apis
> ----------------------------------
>
> Key: TC-486
> URL: https://issues.apache.org/jira/browse/TC-486
> Project: Traffic Control
> Issue Type: Bug
> Components: Traffic Ops API
> Reporter: Jeremy Mitchell
> Assignee: Jeremy Mitchell
> Priority: Minor
>
> Create, update and delete of steering targets should be available to users
> with the "steering" role.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
