use user.pk instead of user.id

[sahama]

Fixes :

• use user.pk instead of user.id

Description of the Change

Checklist

• PR only contains one change (considered splitting up PR)
• unit-test added
• documentation updated
• CHANGELOG.md updated (only for user relevant changes)
• author name in AUTHORS

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.28%. Comparing base (102c851) to head (e50c58b).
Report is 44 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1445      +/-   ##
==========================================
- Coverage   97.42%   97.28%   -0.15%     
==========================================
  Files          32       32              
  Lines        2096     2096              
==========================================
- Hits         2042     2039       -3     
- Misses         54       57       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[n2ygk]

Please explain why this is needed. Is there an open issue that this fixes?

[sahama]

Please explain why this is needed. Is there an open issue that this fixes?

Django allow's to use custom primary key.
for example:

class User(AbstractUser):
    uid = models.UUIDField(primary_key=True, editable=False, default=uuid.uuid4, unique=True, db_index=True)

And also in django oauth toolkit it is ok to use custom Application and other models.

class Application(AbstractApplication):
    uid = models.UUIDField(primary_key=True, editable=False, default=uuid.uuid4, unique=True, db_index=True)

so for example if i use uuid as primary key and name primary key as uid or gid, this accure bug in some cases.