microsoft / microsoft/AzureTRE
Add workspace level configurations to require double-review for airlock imports and/or exports
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 235
- Forks
- 192
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 13
Description
Is your feature request related to a problem? Please describe.
It was common for our data managers before using AzureTRE to do a double review of data being brought in our out of a secure research workspace. Currently with AzureTRE there is no way to require two airlock managers to review an import or export request.
Describe the solution you'd like
The best solution for our use case would be two workspace-level settings, enforcing double-review for airlock import and airlock export. AzureTRE would then require two airlock managers to approve the data before it can be brought in or taken out of a workspace.
Describe alternatives you've considered
Currently we tell our airlock managers to do this for certain projects, having two people look at the data on the same airlock VM. This works somewhat, but as we scale up the number of workspaces in our deployment that implicit requirement is likely to be forgotten.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the airlock import and export approval flows and how workspace-level settings are represented and loaded. Define how each workspace setting controls whether two airlock managers must approve, then verify that both workflows enforce the requirement before data movement proceeds.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, python
- Domain
- authorization, backend, security
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100