Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update run_access_request to include data uses and data category metadata #3266

Closed
TheAndrewJackson opened this issue May 10, 2023 · 1 comment · Fixed by #3674
Closed

Update run_access_request to include data uses and data category metadata #3266

TheAndrewJackson opened this issue May 10, 2023 · 1 comment · Fixed by #3674
Assignees

Comments

@TheAndrewJackson
Copy link
Contributor

Description

The upcoming DSR export package is going to use each connectors data uses and data categories to help organize the data subjects data with the package. Currently, access requests don't add this data to the access package. Access Requests need to be updated to include in data in the package.

access_results_post_processing is the function that will updated to include the additional metadata.

Things to consider

The access package needs to use the metadata that was in the datasets at the time of graph generation. They cannot be looked up at runtime. This is because the execution will likely not match the current datasets exactly. This is because access requests are long running jobs. DSR's already cache the graph to fix this drift issue.

Make sure that all of the required package metadata is included at graph creation time. If it's not already included in the cached graph then add it.

@Roger-Ethyca
Copy link
Contributor

moving to done

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 a pull request may close this issue.

4 participants