Page MenuHomePhorge
Differential D217

Add fallback endpoint for download
Changes PlannedPublicDraft
Actions

Authored by nannanko on Mon, May 26, 10:14 PM.
Tags
None
Referenced Files
F2576513: D217.1750628035.diff
Sat, Jun 21, 2:33 PM
F2575902: D217.1750591129.diff
Sat, Jun 21, 4:18 AM
F2575628: D217.1750580638.diff
Sat, Jun 21, 1:23 AM
F2575602: D217.1750579060.diff
Sat, Jun 21, 12:57 AM
Subscribers
None

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Maniphest Tasks
T164: Cannot download file from matrix.org
Summary

When the old Endpoint cannot be downloaded, fall back to the new endpoint.

Requires D216

Type: add

Test Plan

Log in to a matrix.org account (or other homeservers that do not support old endpoints), check the download function work fine.

Verify unit tests pass.

Diff Detail

Repository
rK kazv
Branch
nannanko/mxc-v1
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 387
Build 757: GitLab CI for kazv
Build 756: arc lint + arc unit

Event Timeline

nannanko created this revision.Mon, May 26, 10:14 PM
Owners added a reviewer: O1: the Kazv Project.Mon, May 26, 10:14 PM
nannanko added a parent revision: D216: Add GetContentJobV1 for the new download endpoint that needs authorization.Mon, May 26, 10:15 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Mon, May 26, 10:21 PM
Harbormaster failed remote builds in B373: Diff 608!
nannanko updated this revision to Diff 609.Mon, May 26, 10:50 PM

Add unit test

nannanko edited the test plan for this revision. (Show Details)Mon, May 26, 10:51 PM
nannanko edited the test plan for this revision. (Show Details)
Harbormaster returned this revision to the author for changes because remote builds failed.Mon, May 26, 10:57 PM
Harbormaster failed remote builds in B374: Diff 609!
nannanko added a task: T164: Cannot download file from matrix.org.Wed, May 28, 1:40 AM
nannanko edited the summary of this revision. (Show Details)Wed, May 28, 6:06 PM
nannanko updated this revision to Diff 620.Sat, Jun 7, 6:00 PM

Perfer new endpoint

Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Jun 7, 6:07 PM
Harbormaster failed remote builds in B382: Diff 620!
nannanko updated this revision to Diff 627.Sat, Jun 14, 6:47 PM

Automatically select the download endpoint according to the server version

Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Jun 14, 6:54 PM
Harbormaster failed remote builds in B387: Diff 627!

Revision Contents
Changeset List

Diff 627

View Options

src/contents/ui/FileHandler.qml

Loading...
View Options

src/kazv-io-job.hpp

Loading...
View Options

src/kazv-io-job.cpp

Loading...
View Options

src/kazv-io-manager.hpp

Loading...
View Options

src/kazv-io-manager.cpp

Loading...
View Options

src/matrix-sdk.hpp

Loading...
View Options

src/matrix-sdk.cpp

Loading...
View Options

src/tests/kazv-io-job-test.cpp

Loading...