rob05c commented on a change in pull request #2406: Re-added anonymous_blocking_enabled field to deliveryservice queries URL: https://github.com/apache/trafficcontrol/pull/2406#discussion_r195501918
########## File path: lib/go-tc/deliveryservices.go ########## @@ -58,59 +58,60 @@ type DeleteDeliveryServiceResponse struct { // DeliveryService ... // TODO move contents to DeliveryServiceV12, fix references, and remove type DeliveryService struct { - Active bool `json:"active"` - CacheURL string `json:"cacheurl"` - CCRDNSTTL int `json:"ccrDnsTtl"` - CDNID int `json:"cdnId"` - CDNName string `json:"cdnName"` - CheckPath string `json:"checkPath"` - DeepCachingType DeepCachingType `json:"deepCachingType"` - DisplayName string `json:"displayName"` - DNSBypassCname string `json:"dnsBypassCname"` - DNSBypassIP string `json:"dnsBypassIp"` - DNSBypassIP6 string `json:"dnsBypassIp6"` - DNSBypassTTL int `json:"dnsBypassTtl"` - DSCP int `json:"dscp"` - EdgeHeaderRewrite string `json:"edgeHeaderRewrite"` - ExampleURLs []string `json:"exampleURLs"` - GeoLimit int `json:"geoLimit"` - FQPacingRate int `json:"fqPacingRate"` - GeoProvider int `json:"geoProvider"` - GlobalMaxMBPS int `json:"globalMaxMbps"` - GlobalMaxTPS int `json:"globalMaxTps"` - HTTPBypassFQDN string `json:"httpBypassFqdn"` - ID int `json:"id"` - InfoURL string `json:"infoUrl"` - InitialDispersion float32 `json:"initialDispersion"` - IPV6RoutingEnabled bool `json:"ipv6RoutingEnabled"` - LastUpdated *TimeNoMod `json:"lastUpdated" db:"last_updated"` - LogsEnabled bool `json:"logsEnabled"` - LongDesc string `json:"longDesc"` - LongDesc1 string `json:"longDesc1"` - LongDesc2 string `json:"longDesc2"` - MatchList []DeliveryServiceMatch `json:"matchList,omitempty"` - MaxDNSAnswers int `json:"maxDnsAnswers"` - MidHeaderRewrite string `json:"midHeaderRewrite"` - MissLat float64 `json:"missLat"` - MissLong float64 `json:"missLong"` - MultiSiteOrigin bool `json:"multiSiteOrigin"` - OrgServerFQDN string `json:"orgServerFqdn"` - ProfileDesc string `json:"profileDescription"` - ProfileID int `json:"profileId,omitempty"` - ProfileName string `json:"profileName"` - Protocol int `json:"protocol"` - QStringIgnore int `json:"qstringIgnore"` - RangeRequestHandling int `json:"rangeRequestHandling"` - RegexRemap string `json:"regexRemap"` - RegionalGeoBlocking bool `json:"regionalGeoBlocking"` - RemapText string `json:"remapText"` - RoutingName string `json:"routingName"` - SigningAlgorithm string `json:"signingAlgorithm" db:"signing_algorithm"` - TypeID int `json:"typeId"` - Type string `json:"type"` - TRResponseHeaders string `json:"trResponseHeaders"` - TenantID int `json:"tenantId,omitempty"` - XMLID string `json:"xmlId"` + Active bool `json:"active"` + AnonymousBlockingEnabled bool `json:"anonymousBlockingEnabled"` Review comment: Eh, it makes me really sad. We really, really need to stop breaking the API promise. We're going to lose new users, because our API is unreliable and unstable. But I won't -1 this, since at least @limited is already using it, and it's in 2.2.0-RC6 :( ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services