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

Provide access for external services to embargoed content #2066

Open
yarikoptic opened this issue Oct 31, 2024 · 2 comments
Open

Provide access for external services to embargoed content #2066

yarikoptic opened this issue Oct 31, 2024 · 2 comments
Labels
embargo Issues around embargo functionality UX Affects usability of the system

Comments

@yarikoptic
Copy link
Member

Inspired by @aaronkanzer pointer to docs on how integration is happening for zarrs in LINC in

but I guess in many cases for simple .nwb we would be fine if we just point to "minted on demand" (the /download/ end point at the time of external service might already be lacking session/context to authenticate seems since https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/dd8e4fed-743b-497e-b4da-db0ba71ff5d8/download/&dandisetId=001169&dandisetVersion=draft does not work for me even though I can download that asset in the same browser... attn @magland )
Thus we would just annotate which services in general would have ability to access embargoed assets, and trigger minting URLs upon click instead of providing ready to use URLs.

@yarikoptic yarikoptic added UX Affects usability of the system embargo Issues around embargo functionality labels Oct 31, 2024
@aaronkanzer
Copy link
Member

aaronkanzer commented Oct 31, 2024

@yarikoptic -- thanks for filing this -- just tagging some working code we have for calling any operation upon button click in the FileBrowser here (this is from the LINC side when we want to issue CloudFront cookies prior to proceeding to neuroglancer)

I'm assuming this this nwb use case you'd also want to ping an endpoint rather than including a potentially minted presigned URL in every asset.metadata upfront -- see code reference here and here

Happy to share additional unsolicited advice 😉 -- (e.g. make sure to leverage aioboto3 here instead of boto3 -- boto is blocking for expensive presigned url construction!)

@magland
Copy link
Contributor

magland commented Oct 31, 2024

See #2065 (comment)

Neurosift supports viewing of embargoed assets (if it's working properly)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embargo Issues around embargo functionality UX Affects usability of the system
Projects
None yet
Development

No branches or pull requests

3 participants