search

Krukov / amocrm_api

Python amocrm API wrapper
MIT License
104 stars 55 forks source link

fix: TokenManager._is_expire() #39

Closed dinodigital closed 3 years ago

dinodigital commented 3 years ago

_is_expire() function now works correctly

Krukov commented 3 years ago

Сори, но не уверен что это поможет, так как сама pyjwt валидирует exp по умолчанию https://pyjwt.readthedocs.io/en/stable/usage.html#reading-the-claimset-without-validation

dinodigital commented 3 years ago

У меня версия 2.0.0 pyjwt и _is_expire() не работает. Прогонял этот кусок кода отдельно с ключом eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImp0aSI6IjUzMzYwZjM1OTE4NzIyOThkYjliOTE4MGExOGIwZWRkMDQyOGVlMWY1MzA1NTYyNmNmMTQ4ZGZmYjBmYWQ4ZTBiOGI1N2QxNzI3YTE2YzdjIn0.eyJhdWQiOiJiYjZhOTljNC1kMWY4LTQwMjgtODRmNC02MDZmNDQyYzdmYTEiLCJqdGkiOiI1MzM2MGYzNTkxODcyMjk4ZGI5YjkxODBhMThiMGVkZDA0MjhlZTFmNTMwNTU2MjZjZjE0OGRmZmIwZmFkOGUwYjhiNTdkMTcyN2ExNmM3YyIsImlhdCI6MTYxMDU1MDM1OCwibmJmIjoxNjEwNTUwMzU4LCJleHAiOjE2MTA2MzY3NTgsInN1YiI6IjYwOTE4MCIsImFjY291bnRfaWQiOjk2NjczNDcsInNjb3BlcyI6WyJwdXNoX25vdGlmaWNhdGlvbnMiLCJjcm0iLCJub3RpZmljYXRpb25zIl19.MUzDFcL0kkpuJAggXBy4TU1R_9PKrCJpihJc1byRszLNaLJbln7hl9beMGP4qBMUwG9cMzpd_7V8A0mbTy0uBWxIEzGvhOrxNZDz1u-yqK99wCYH0eoBAIv6yh6yw4NdfyOcQ1lU0nvR0whta6D5HRmih7TV8EX_n74KO3YtYGmNkq-08eS-QgraCPf0YfIF2_jEuoNFCkPKusFQv6QXdJ_JNPjUk0WDKbF5U4K_UJZZ-9YS0bE5lBVCPqQJdmdUCry1J-ne2cvrZ0ARpIrPs_-O-FcA3-L6293DxKovwPeJ6QN_ShHQiKPFTsPn8v5b6oR0q-elALYv3CrMvD8n9Q

Так он показывает, что False, мол не истек срок действия. Даже сейчас.

Krukov commented 3 years ago

Спасибо, отписал в основной задаче, но ваш фикс действительно актуален. Можно тогда попросить вас пофиксить тесты так там есть генерация токена и они должны падать сейчас?