Skip to content
Snippets Groups Projects

authz: utilize expired_at and last_valid_at columns of user_external_accounts

Merged authz: utilize expired_at and last_valid_at columns of user_external_accounts
jc/permsSyncer-expired-last-valid into main
Merged Warren Gifford requested to merge jc/permsSyncer-expired-last-valid into main

Created by: unknwon

This PR does two things in the PermsSyncer:

  • When successfully used a token from user_external_accounts, we update the last_valid_at column.
  • Otherwise, update the expired_at column.

Part of #14899 (closed)

TODOs

  • Implements errcode.IsUnauthorized for GitHub and GitLab errors
  • Add tests for db.ExternalAccounts
  • Add tests for PermsSyncer

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply