crypt/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-01-21 19:01:07 +03:00
Code Activity

Merge pull request #380 from kuba/boulder-130-fixed

Browse Source 
boulder#130 fixed
This commit is contained in:
James Kasten
2015-05-03 13:06:32 -07:00
parent 93cdad120d 0216ea3f26
commit e4ac29ef81
2 changed files with 3 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
letsencrypt/client/network2.py
View File

@@ -315,11 +315,7 @@ class Network(object):
try:
authzr_uri = response.links['up']['url']
except KeyError:
# TODO: Right now Boulder responds with the authorization resource
# instead of a challenge resource... this can be uncommented
# once the error is fixed (boulder#130).
return None
# raise errors.NetworkError('"up" Link header missing')
raise errors.NetworkError('"up" Link header missing')
challr = messages2.ChallengeResource(
authzr_uri=authzr_uri,
body=messages2.ChallengeBody.from_json(response.json()))

7
letsencrypt/client/tests/network2_test.py
View File

@@ -286,11 +286,8 @@ class NetworkTest(unittest.TestCase):
def test_answer_challenge_missing_next(self):
self._mock_post_get()
self.assertTrue(self.net.answer_challenge(
self.challr.body, challenges.DNSResponse()) is None)
# TODO: boulder#130, acme-spec#110
# self.assertRaises(errors.NetworkError, self.net.answer_challenge,
# self.challr.body, challenges.DNSResponse())
self.assertRaises(errors.NetworkError, self.net.answer_challenge,
self.challr.body, challenges.DNSResponse())
def test_retry_after_date(self):
self.response.headers['Retry-After'] = 'Fri, 31 Dec 1999 23:59:59 GMT'