Skip to content

revert: restore SharePoint download URLs (revert #13)#18

Merged
kphunter merged 1 commit into
mainfrom
devin/1779238804-revert-azure-blob-urls
May 20, 2026
Merged

revert: restore SharePoint download URLs (revert #13)#18
kphunter merged 1 commit into
mainfrom
devin/1779238804-revert-azure-blob-urls

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Reverts the merge of PR #13, which changed dataset download URLs from SharePoint to direct Azure Blob Storage URLs. The blob URLs return 401 errors in browsers because Azure Blob Storage with private access doesn't have a browser-based SSO login flow.

This restores the original SharePoint download links so users can continue downloading datasets while the authenticated Function App download endpoint (#17) is set up and tested.

The URL cutover to the Function App endpoint will be done in a follow-up PR once the full stack is deployed and verified.

Review & Testing Checklist for Human

Low risk — this is a straightforward revert of a single JSON file change.

  • Verify the SharePoint download URLs are working after merge (click a dataset download link on the portal)

Notes

  • The Azure Blob Storage account and container are still provisioned and ready — this revert only affects the URLs in datasets.json
  • Once the Function App is deployed and Easy Auth is configured, a new PR will update the URLs to point to https://bcit-tlu-opendata-dl.azurewebsites.net/api/download?file=<name>.zip

Link to Devin session: https://app.devin.ai/sessions/96d3707e572d4c968d5bfa32f62562e8
Requested by: @kphunter

…lob-urls"

This reverts commit 4c99194, reversing
changes made to f11e8a7.
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@kphunter kphunter merged commit 2466178 into main May 20, 2026
5 checks passed
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 this pull request may close these issues.

1 participant