From a5c09c50c06b271ab4bba23a90153183430622ef Mon Sep 17 00:00:00 2001 From: Jessica Tallon Date: Mon, 8 Dec 2014 13:44:33 +0000 Subject: [PATCH] Fix OAuth length problems in clients by removing that constraint --- mediagoblin/oauth/oauth.py | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/mediagoblin/oauth/oauth.py b/mediagoblin/oauth/oauth.py index 100dc8eb..c7951734 100644 --- a/mediagoblin/oauth/oauth.py +++ b/mediagoblin/oauth/oauth.py @@ -28,9 +28,17 @@ class GMGRequestValidator(RequestValidator): self.POST = data super(GMGRequestValidator, self).__init__(*args, **kwargs) - @property - def nonce_length(self): - return 5, 30 + def check_nonce(self, nonce): + """ + This checks that the nonce given is a valid nonce + + RequestValidator.check_nonce checks that it's between a maximum and + minimum length which, not only does pump.io not do this from what + I can see but there is nothing in rfc5849 which suggests a maximum or + minium length should be required so I'm removing that check + """ + # Check the nonce only contains a subset of the safe characters. + return set(nonce) <= self.safe_characters def save_request_token(self, token, request): """ Saves request token in db """ -- 2.25.1