[
https://issues.apache.org/jira/browse/TC-46?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15702275#comment-15702275
]
ASF GitHub Bot commented on TC-46:
----------------------------------
Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/104#discussion_r89808929
--- Diff: traffic_ops/client/delivery_service_resources.go ---
@@ -35,49 +34,49 @@ type DeleteDeliveryServiceResponse struct {
// DeliveryService ...
type DeliveryService struct {
- ID string `json:"id"`
+ ID int `json:"id"`
XMLID string `json:"xmlId"`
Active bool `json:"active"`
- DSCP string `json:"dscp"`
+ DSCP int `json:"dscp"`
Signed bool `json:"signed"`
- QStringIgnore string `json:"qstringIgnore"`
- GeoLimit string `json:"geoLimit"`
- GeoProvider string `json:"geoProvider"`
+ QStringIgnore int `json:"qstringIgnore"`
+ GeoLimit int `json:"geoLimit"`
+ GeoProvider int `json:"geoProvider"`
HTTPBypassFQDN string `json:"httpBypassFqdn"`
DNSBypassIP string `json:"dnsBypassIp"`
DNSBypassIP6 string `json:"dnsBypassIp6"`
DNSBypassCname string `json:"dnsBypassCname"`
- DNSBypassTTL string `json:"dnsBypassTtl"`
+ DNSBypassTTL int `json:"dnsBypassTtl"`
OrgServerFQDN string `json:"orgServerFqdn"`
Type string `json:"type"`
ProfileName string `json:"profileName"`
ProfileDesc string `json:"profileDescription"`
CDNName string `json:"cdnName"`
- CCRDNSTTL string `json:"ccrDnsTtl"`
- GlobalMaxMBPS string `json:"globalMaxMbps"`
- GlobalMaxTPS string `json:"globalMaxTps"`
+ CCRDNSTTL int `json:"ccrDnsTtl"`
+ GlobalMaxMBPS int `json:"globalMaxMbps"`
+ GlobalMaxTPS int `json:"globalMaxTps"`
LongDesc string `json:"longDesc"`
LongDesc1 string `json:"longDesc1"`
LongDesc2 string `json:"longDesc2"`
- MaxDNSAnswers string `json:"maxDnsAnswers"`
+ MaxDNSAnswers int `json:"maxDnsAnswers"`
InfoURL string `json:"infoUrl"`
- MissLat string `json:"missLat"`
- MissLong string `json:"missLong"`
+ MissLat float64 `json:"missLat"`
+ MissLong float64 `json:"missLong"`
CheckPath string `json:"checkPath"`
LastUpdated string `json:"lastUpdated"`
- Protocol string `json:"protocol"`
+ Protocol int `json:"protocol"`
IPV6RoutingEnabled bool `json:"ipv6RoutingEnabled"`
- RangeRequestHandling string
`json:"rangeRequestHandling"`
+ RangeRequestHandling int
`json:"rangeRequestHandling"`
HeaderRewrite string `json:"headerRewrite"`
EdgeHeaderRewrite string `json:"edgeHeaderRewrite"`
MidHeaderRewrite string `json:"midHeaderRewrite"`
TRResponseHeaders string `json:"trResponseHeaders"`
RegexRemap string `json:"regexRemap"`
CacheURL string `json:"cacheurl"`
RemapText string `json:"remapText"`
- MultiSiteOrigin string `json:"multiSiteOrigin"`
+ MultiSiteOrigin int `json:"multiSiteOrigin"`
--- End diff --
this should be a bool
> Update Traffic Ops client to work with output of Postgres APIs
> --------------------------------------------------------------
>
> Key: TC-46
> URL: https://issues.apache.org/jira/browse/TC-46
> Project: Traffic Control
> Issue Type: Bug
> Components: Traffic Ops Client
> Affects Versions: 2.0.0
> Reporter: David Neuman
> Assignee: David Neuman
> Fix For: 2.0.0
>
>
> As we move to postgres, the Traffic Ops APIs now return fields in formats
> other than string. The Traffic Ops client needs to be updated to support the
> types that are returned, otherwise json decode fails.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
