Oauthlib: populate_token_attributes () ValueError لقيم float expires_at

تم إنشاؤها على ١٠ ديسمبر ٢٠٢٠  ·  7تعليقات  ·  مصدر: oauthlib/oauthlib

https://github.com/oauthlib/oauthlib/blob/d54965b86ce4ede956db70baff0b3d5e9182a007/oauthlib/oauth2/rfc6749/clients/base.py#L516

بعض الرموز لها قيمة expires_at كعوامة (مثال: 1612807300.1613762 ) مما يجعل هذا السطر يرمي ValueError: invalid literal for int() with base 10 . يمكن حل هذا باستخدام int(float(response.get('expires_at'))) (أو التقريب لعدم الاقتطاع).

Bug Contributor Friendly OAuth2-Client

التعليق الأكثر فائدة

لم يتم تعريف استخدام expires_at في RFC ، لذلك يبدو أن هناك ارتباكًا في الكود أيضًا. يمكننا أن نفترض أنها int مثل expires_in .

نرحب بأي علاقات عامة لإصلاح استخدامها وإضافة اختبار وحدة لتغطية هذه الحالة.

ال 7 كومينتر

فقط فوق هذا self._expires_at يتم تعويمه:

self._expires_at = time.time() + int(self.expires_in)

لم يتم تعريف استخدام expires_at في RFC ، لذلك يبدو أن هناك ارتباكًا في الكود أيضًا. يمكننا أن نفترض أنها int مثل expires_in .

نرحب بأي علاقات عامة لإصلاح استخدامها وإضافة اختبار وحدة لتغطية هذه الحالة.

مهلا ، هل يمكنني الحصول على هذه المشكلة؟ لقد بدأت للتو مع المصدر المفتوح ، لذلك ستكون هذه بداية جيدة ig.

مهلا ، هل يمكنني الحصول على هذه المشكلة؟ لقد بدأت للتو مع المصدر المفتوح ، لذلك ستكون هذه بداية جيدة ig.

@ default-303 اذهب لذلك!

ggiill شكرًا ، لقد أجريت التغييرات التالية

- self._expires_at = int(response.get('expires_at')) 
+ self._expires_at = round(float(response.get('expires_at')))

واجتياز جميع الاختبارات عندما أقوم بتشغيل pytest

ولكن ما نوع الاختبارات التي تريد مني إضافتها؟ مثل اختبار بسيط typecheck شيء كهذا -
""
استيراد unittest
استيراد الإصلاح ## نموذج نصي أعددته لهذا العرض التوضيحي
وقت الاستيراد

فئة TestFix (unittest.TestCase):

def test_float(self) : 
    string_time = str(time.time())
    self.assertIsInstance(fix.get_time("123345.1222"), int)

def test_int(self) : 
    string_time = str(round(time.time()))
    self.assertIsInstance(fix.get_time("12333"), int)

""

أو كامل على mock test ؟

ggiill مرحبا ، هل يمكنني الحصول على مراجعة؟
lmao لقد مر شهر حتى نسيت أبت.

ggiill مرحبا ، هل يمكنني الحصول على مراجعة؟
lmao لقد مر شهر حتى نسيت أبت.

@ default-303 أنا لست مشرفًا - pingingJonathanHuot. ربما ترغب في وضع العلاقات العامة للمراجعة أيضًا.

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

القضايا ذات الصلة

JonathanHuot picture JonathanHuot  ·  26تعليقات

polamayster picture polamayster  ·  19تعليقات

thedrow picture thedrow  ·  31تعليقات

ViktorHaag picture ViktorHaag  ·  11تعليقات

JonathanHuot picture JonathanHuot  ·  15تعليقات