Commit46b0dac

committed

Use new context manager

1 parentfe84d77 commit46b0dacCopy full SHA for 46b0dac

File tree

+10

-13

lines changed

+10

-13

lines changed

Lines changed: 10 additions & 13 deletions

Original file line number	Diff line number	Diff line change
`@@ -102,25 +102,22 @@ def authorize(self, login, password, scopes=None, note='', note_url='',`
`102`	`102`	`"""`
`103`	`103`	`json=None`
`104`	`104`	`# TODO: Break this behaviour in 1.0 (Don't rely on self._session.auth)`
`105`		`-auth=self._session.author (loginandpassword)`
	`105`	`+auth=None`
	`106`	`+ifself._session.auth:`
	`107`	`+auth=self._session.auth`
	`108`	`+elifloginandpassword:`
	`109`	`+auth= (login,password)`
	`110`	`+`
`106`	`111`	`ifauth:`
`107`	`112`	`url=self._build_url('authorizations')`
`108`	`113`	`data= {'note':note,'note_url':note_url,`
`109`	`114`	`'client_id':client_id,'client_secret':client_secret}`
`110`	`115`	`ifscopes:`
`111`	`116`	`data['scopes']=scopes`
`112`		`-# TODO: Unconditionally use the login and password, e.g.,`
`113`		`-# old_auth = self._session.auth`
`114`		`-# self.login(login, password)`
`115`		`-# json = self._json(...)`
`116`		`-# self._session.auth = old_auth`
`117`		`-do_logout=False`
`118`		`-ifnotself._session.auth:`
`119`		`-do_logout=True`
`120`		`-self.login(login,password)`
`121`		`-json=self._json(self._post(url,data=data),201)`
`122`		`-ifdo_logout:`
`123`		`-self._session.auth=None`
	`117`	`+`
	`118`	`+withself._session.temporary_basic_auth(*auth):`
	`119`	`+json=self._json(self._post(url,data=data),201)`
	`120`	`+`
`124`	`121`	`returnAuthorization(json,self)ifjsonelseNone`
`125`	`122`
`126`	`123`	`defcheck_authorization(self,access_token):`

Comments

(0)