The launchpadlib test is awesome. Yay.
The potential security issues are concerning, but they do seem fine now.
IRC discussion:
[10:23am] gary_poster: leonardr: xx-service.txt looks like it would fail now. Your code says self.response.setHeader('Vary', 'Accept') but your test says [10:23am] gary_poster: 83+ >>> response = webservice.get( [10:23am] gary_poster: 84+ ... 'http://bugs.launchpad.dev/api/devel/') [10:23am] gary_poster: 85+ >>> print response.getheader('Vary') [10:23am] gary_poster: 86+ Authorization, Accept [10:23am] leonardr: gary: thanks, don't know how i missed that
« Back to merge proposal
The launchpadlib test is awesome. Yay.
The potential security issues are concerning, but they do seem fine now.
IRC discussion:
[10:23am] gary_poster: leonardr: xx-service.txt looks like it would fail now. Your code says self.response. setHeader( 'Vary', 'Accept') but your test says bugs.launchpad. dev/api/ devel/') getheader( 'Vary')
[10:23am] gary_poster: 83+ >>> response = webservice.get(
[10:23am] gary_poster: 84+ ... 'http://
[10:23am] gary_poster: 85+ >>> print response.
[10:23am] gary_poster: 86+ Authorization, Accept
[10:23am] leonardr: gary: thanks, don't know how i missed that