kerberos: move renew_till default into serializer

roost-im · Jun 24, 2023 · 787ae2d · 787ae2d
1 parent 21d3d66
commit 787ae2d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/roost_backend/serializers.py b/roost_backend/serializers.py
@@ -73,7 +73,7 @@ class KerberosCredentialsSerializer(serializers.Serializer):
     authtime = serializers.IntegerField()
     starttime = serializers.IntegerField()
     endtime = serializers.IntegerField()
-    renew_till = serializers.IntegerField(required=False)
+    renew_till = serializers.IntegerField(required=False, default=0)
     srealm = serializers.CharField()
     sname = _InlineNameSerializer()
 

diff --git a/roost_backend/utils/kerberos.py b/roost_backend/utils/kerberos.py
@@ -93,7 +93,7 @@ def verify_same_princ(client):
     kcreds.times.authtime = creds['authtime'] // 1000
     kcreds.times.starttime = creds['starttime'] // 1000
     kcreds.times.endtime = creds['endtime'] // 1000
-    kcreds.times.renew_till = creds.get('renew_till', 0) // 1000
+    kcreds.times.renew_till = creds['renew_till'] // 1000
     kcreds.is_skey = False
 
     # This makes roost's python sad. Add a null check there before dereferencing.