feat(content): new endpoint for content retrieval PD-169 by Avantol13 · Pull Request #1348 · uc-cdis/fence

Avantol13 · 2026-05-21T19:17:31Z

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/PD-169

New Features

Bulk Content Retrieval support in Fence

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

coveralls · 2026-05-21T19:22:33Z

Coverage Report for CI Build 29041659008

Coverage increased (+0.08%) to 75.49%

Details

Coverage increased (+0.08%) from the base build.
Patch coverage: No coverable lines changed in this PR.
102 coverage regressions across 4 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

102 previously-covered lines in 4 files lost coverage.

File	Lines Losing Coverage	Coverage
blueprints/data/indexd.py	32	94.29%
blueprints/data/blueprint.py	28	85.02%
init.py	26	89.49%
config.py	16	68.06%

Coverage Stats


Relevant Lines:	11522
Covered Lines:	8698
Line Coverage:	75.49%
Coverage Strength:	0.75 hits per line

💛 - Coveralls

github-actions · 2026-05-21T20:07:16Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_presigned_url.py	7	0	7
tests/test_dbgap.py	4	1	5
tests/test_drs_endpoint.py	4	0	4
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_fence_admin.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_register_user.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	72	9	81

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-05-26T20:02:27Z

Integration Tests

filepath	passed	error	skipped	SUBTOTAL
tests/test_user_login_activation.py	0	5	0	5
tests/test_oauth2.py	15	0	0	15
tests/test_centralized_auth.py	16	0	0	16
tests/test_audit_service.py	3	0	3	6
tests/test_data_upload.py	8	0	1	9
tests/test_presigned_url.py	7	0	0	7
tests/test_dbgap.py	4	0	1	5
tests/test_drs_endpoint.py	4	0	0	4
tests/test_oidc_client.py	2	0	0	2
tests/test_user_token.py	5	0	0	5
tests/test_fence_admin.py	2	0	0	2
tests/test_google_data_access.py	1	0	0	1
tests/test_client_credentials.py	1	0	0	1
tests/test_register_user.py	2	0	0	2
tests/test_ras_authn.py	0	0	3	3
TOTAL	70	5	8	83

Please find the detailed integration test report here

Please find the Github Action logs here

tianj7 · 2026-05-29T15:30:01Z

+@blueprint.route("/content", methods=["POST"])
+def get_file_content():
+    """
+    Get content of a files referenced by GUIDs provided.


files instead of a files

tianj7 · 2026-05-29T19:15:20Z

+                        break
+
+                if any(
+                    file_location.url.startswith(allowed_ai_url)


we just did this check in the previous block L893-898, can we perhaps combine the logic?

or use if any allowed_ai_url_root

good call, I refactored a bit

tianj7

left 2 comments

github-actions · 2026-06-05T15:43:03Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_dbgap.py	4	1	5
tests/test_presigned_url.py	7	0	7
tests/test_drs_endpoint.py	4	0	4
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_fence_admin.py	2	0	2
tests/test_register_user.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	72	9	81

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-06-08T17:06:56Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_dbgap.py	4	1	5
tests/test_drs_endpoint.py	4	0	4
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_presigned_url.py	7	0	7
tests/test_register_user.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_fence_admin.py	2	0	2
tests/test_client_credentials.py	1	0	1
tests/test_google_data_access.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	72	9	81

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-06-08T17:30:27Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_dbgap.py	4	1	5
tests/test_drs_endpoint.py	4	0	4
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_presigned_url.py	7	0	7
tests/test_register_user.py	2	0	2
tests/test_client_credentials.py	1	0	1
tests/test_fence_admin.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	72	9	81

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-06-17T21:51:16Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_dbgap.py	4	1	5
tests/test_presigned_url.py	7	0	7
tests/test_drs_endpoint.py	5	0	5
tests/test_user_token.py	5	0	5
tests/test_fence_admin.py	2	0	2
tests/test_user_login_activation.py	2	1	3
tests/test_register_user.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	73	9	82

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-06-18T15:13:37Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_dbgap.py	4	1	5
tests/test_drs_endpoint.py	5	0	5
tests/test_presigned_url.py	7	0	7
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_fence_admin.py	2	0	2
tests/test_register_user.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_ras_authn.py	0	3	3
TOTAL	73	9	82

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-07-08T16:23:14Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_drs_endpoint.py	6	0	6
tests/test_dbgap.py	4	1	5
tests/test_user_token.py	5	0	5
tests/test_user_login_activation.py	2	1	3
tests/test_presigned_url.py	7	0	7
tests/test_register_user.py	2	0	2
tests/test_fence_admin.py	2	0	2
tests/test_oidc_client.py	2	0	2
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_ras_authn.py	0	3	3
tests/test_ras_passport.py	0	3	3
TOTAL	74	12	86

Please find the detailed integration test report here

Please find the Github Action logs here

github-actions · 2026-07-09T19:44:21Z

Integration Tests

Test summary after running integration tests

filepath	passed	failed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	0	15
tests/test_centralized_auth.py	16	0	0	16
tests/test_audit_service.py	2	1	3	6
tests/test_data_upload.py	8	0	1	9
tests/test_drs_endpoint.py	6	0	0	6
tests/test_dbgap.py	4	0	1	5
tests/test_user_token.py	5	0	0	5
tests/test_user_login_activation.py	2	0	1	3
tests/test_register_user.py	2	0	0	2
tests/test_fence_admin.py	2	0	0	2
tests/test_oidc_client.py	2	0	0	2
tests/test_client_credentials.py	1	0	0	1
tests/test_google_data_access.py	1	0	0	1
tests/test_presigned_url.py	7	0	0	7
tests/test_ras_authn.py	0	0	3	3
tests/test_ras_passport.py	0	0	3	3
TOTAL	73	1	12	86

Test summary after rerunning failed integration tests

filepath	failed	SUBTOTAL
tests/test_audit_service.py	1	1
TOTAL	1	1

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

github-actions · 2026-07-09T20:40:27Z

Integration Tests

filepath	passed	skipped	SUBTOTAL
tests/test_audit_service.py	3	3	6
TOTAL	3	3	6

Please find the detailed integration test report here

Please find the Github Action logs here

feat(content): new endpoint for content retrieval

a6dbf96

feat(bulk): unit tests, bug fixes, openapi docs

6921ed3

tianj7 reviewed May 29, 2026

View reviewed changes

fix(comments): address comments, more debug lines

132a7d3

Avantol13 and others added 3 commits June 8, 2026 11:43

fix(index): serialize the json

fea0b2d

Merge branch 'master' into feat/download-content

27cae35

fix(content): use json serializer

4c27dc4

fix(tests): use json serializer

e922a1a

chore(content): debug line

44db0c4

Merge branch 'master' into feat/download-content

6a9dda6

data/content only

d5684b8

markxiao previously approved these changes Jul 9, 2026

View reviewed changes

Avantol13 and others added 2 commits July 9, 2026 13:41

Merge branch 'master' into feat/download-content

98d9def

chore(data): use safer quoating in f string

467f071

Avantol13 dismissed markxiao’s stale review via 467f071 July 9, 2026 18:42

markxiao approved these changes Jul 9, 2026

View reviewed changes

PlanXCyborg added the TestAuditService label Jul 9, 2026

Avantol13 merged commit 86c2f9c into master Jul 9, 2026
11 of 12 checks passed

Avantol13 deleted the feat/download-content branch July 9, 2026 21:17

Uh oh!

Conversation

Avantol13 commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New Features

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

Uh oh!

coveralls commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report for CI Build 29041659008

Coverage increased (+0.08%) to 75.49%

Details

Uncovered Changes

Coverage Regressions

Coverage Stats

💛 - Coveralls

Uh oh!

github-actions Bot commented May 21, 2026

Integration Tests

Uh oh!

github-actions Bot commented May 26, 2026

Integration Tests

Uh oh!

tianj7 May 29, 2026

Choose a reason for hiding this comment

Uh oh!

tianj7 May 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Avantol13 Jun 5, 2026

Choose a reason for hiding this comment

Uh oh!

tianj7 left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jun 5, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jun 8, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jun 8, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jun 17, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jun 18, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jul 8, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jul 9, 2026

Integration Tests

Uh oh!

github-actions Bot commented Jul 9, 2026

Integration Tests

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Avantol13 commented May 21, 2026 •

edited

Loading

coveralls commented May 21, 2026 •

edited

Loading

tianj7 May 29, 2026 •

edited

Loading