Skip to content

Remove unused access_tokens table#119

Merged
jeroenrinzema merged 2 commits intomainfrom
copilot/remove-access-token-registry
Jan 9, 2026
Merged

Remove unused access_tokens table#119
jeroenrinzema merged 2 commits intomainfrom
copilot/remove-access-token-registry

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 9, 2026

The access_tokens table was part of the initial schema but is unused. Authentication is now JWT-based with tokens validated but not stored in the database.

Changes

  • Created migration 1767972708_remove_access_tokens to drop the table
  • Down migration recreates full table structure (columns, foreign keys, indexes, trigger) for rollback capability
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove access token registry</issue_title>
<issue_description>Users sign in using a JWT token, tokens should not be stored within the access token registry. We should remove/cleanup the database tables.</issue_description>

<agent_instructions>The access_token table is not being used anymore in nexus. Please create a migration to remove this table.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: jeroenrinzema <3440116+jeroenrinzema@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove access token registry from database Remove unused access_tokens table Jan 9, 2026
Copilot AI requested a review from jeroenrinzema January 9, 2026 15:37
@jeroenrinzema jeroenrinzema marked this pull request as ready for review January 9, 2026 15:41
@jeroenrinzema jeroenrinzema merged commit 8781e1c into main Jan 9, 2026
4 checks passed
@jeroenrinzema jeroenrinzema deleted the copilot/remove-access-token-registry branch January 9, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove access token registry

2 participants