Add PAT scope foundation: request-context plugin, permission tagging, migrations, and models#7580
Add PAT scope foundation: request-context plugin, permission tagging, migrations, and models#7580cstns wants to merge 5 commits into
Conversation
…er for per-request context management
…or improved access control
|
@hardillb, @knolleary :this is the PR we discussed during planning. Setting the read/write flags on the existing permissions will need an extra set of eyes, which is why I've added you both as reviewers. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7580 +/- ##
==========================================
+ Coverage 76.95% 76.96% +0.01%
==========================================
Files 410 413 +3
Lines 21002 21024 +22
Branches 5117 5117
==========================================
+ Hits 16162 16182 +20
- Misses 4840 4842 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
WARNINGThere is another PR that includes a db migration #7577 |
hardillb
left a comment
There was a problem hiding this comment.
I've read this through and it all pretty much makes sense.
Only question is that I only see @fastify/request-context being added to the forge.js but not used anywhere yet. I assume this is just prep?
That is a correct assumption. the |
knolleary
left a comment
There was a problem hiding this comment.
Following up on @hardillb's warning about migrations.
#7577 is getting merged first. It includes a migration with the date 20260622-01-add-device-nodejs-ver.js - the same date as the two migrations in this PR.
The migrations in this PR will need renaming. Either increment to -02- and -03- or bump the date.
Description
Lays the groundwork for scoped PATs (#7411). No behavior changes for existing tokens or sessions.
Important
this PR introduces database changes through two new migration files
Related Issue(s)
closes #7445
Checklist
flowforge.yml?FlowFuse/helmto update ConfigMap TemplateFlowFuse/CloudProjectto update values for Staging/ProductionLabels
area:migrationlabel