there has some bugs.

[LiLong1105]

it has unnecessary comma, and a error variable name.

`
def _get_proxy_url(self, ticket):
template = '{server_url}{auth_prefix}/proxy?'
template += 'targetService={proxy_callback}&pgt={ticket}'
url = template.format(
auth_prefix=self.auth_prefix,
proxy_callback=self.proxy_callback,
server_url=self.server_url,
ticket=ticket,#comma
)
return url

def _get_proxy_validate_url(self, ticket):
    template = '{validate_url}{auth_prefix}/proxy?'
    template += 'ticket={ticket}&service={proxy_callback}'
    url = template.format(
        auth_prefix=self.auth_prefix,
        proxy_callback=self.proxy_callback,
        validate=self.validate_url,#"validate" means "validate_url"
        ticket=ticket,#comma
        )
    return url

def _get_service_validate_url(self, ticket, service_url=None):
    template = '{validate_url}{auth_prefix}/serviceValidate?'
    template += 'ticket={ticket}&service={service_url}'
    url = template.format(
        auth_prefix=self.auth_prefix,
        validate_url=self.validate_url,
        service_url=service_url or self.service_url,
        ticket=ticket,#comma
        )
    if self.proxy_url:
        url = '{url}&pgtUrl={proxy_url}'.format(url, self.proxy_url)
    return url

`