Skip to content

Fetch inspection record data as sas URL#421

Open
andchiind wants to merge 2 commits into
mainfrom
419-sas
Open

Fetch inspection record data as sas URL#421
andchiind wants to merge 2 commits into
mainfrom
419-sas

Conversation

@andchiind

@andchiind andchiind commented Jun 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #419

Blob data can now be fetched using a URL. Example:

image

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like logging, TODO, etc.
  • The PR has been tested locally
  • A test has been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that requires new issues
  • The changes do not introduce dead code as unused imports, functions etc.

@andchiind andchiind self-assigned this Jun 23, 2026
@andchiind andchiind added the breaking-change A breaking change which introduces changes to the public APIs label Jun 23, 2026
@andchiind andchiind marked this pull request as draft June 23, 2026 10:28
@andchiind andchiind force-pushed the 419-sas branch 2 times, most recently from 2f60bcd to 3de8822 Compare June 25, 2026 10:46
@andchiind andchiind marked this pull request as ready for review June 25, 2026 10:50
@andchiind andchiind force-pushed the 419-sas branch 2 times, most recently from 2d6cee3 to f165de2 Compare June 25, 2026 13:02
@andchiind andchiind added improvement Improvement to existing functionality security Issue concerning a security vulnerability labels Jun 25, 2026
Christdej
Christdej approved these changes Jun 26, 2026
View reviewed changes

@Christdej Christdej left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Christdej Christdej Christdej approved these changes

Assignees

@andchiind andchiind

Labels

breaking-change A breaking change which introduces changes to the public APIs improvement Improvement to existing functionality security Issue concerning a security vulnerability

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add endpoint for getting SAS token for analysis results

2 participants

@andchiind @Christdej