Page MenuHomePhorge
Differential D217

Add fallback endpoint for download
Needs ReviewPublic
Actions

Authored by nannanko on May 26 2025, 10:14 PM.
Tags
None
Referenced Files
F5705118: D217.1755011415.diff
Mon, Aug 11, 8:10 AM
F5705117: D217.1755011414.diff
Mon, Aug 11, 8:10 AM
F5705116: D217.1755011413.diff
Mon, Aug 11, 8:10 AM
F5705115: D217.1755011412.diff
Mon, Aug 11, 8:10 AM
F5705114: D217.1755011411.diff
Mon, Aug 11, 8:10 AM
F5705113: D217.1755011409.diff
Mon, Aug 11, 8:10 AM
F5705112: D217.1755011408.diff
Mon, Aug 11, 8:10 AM
F5694869: D217.1755005778.diff
Mon, Aug 11, 6:36 AM
View All 16 Files
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 374
Build 731: GitLab CI for kazv
Build 730: arc lint + arc unit

Event Timeline

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

Add unit test

nannanko edited the test plan for this revision. (Show Details)May 26 2025, 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.May 26 2025, 10:57 PM
Harbormaster failed remote builds in B374: Diff 609!
nannanko added a task: T164: Cannot download file from matrix.org.May 28 2025, 1:40 AM
nannanko edited the summary of this revision. (Show Details)May 28 2025, 6:06 PM
nannanko updated this revision to Diff 620.Jun 7 2025, 6:00 PM

Perfer new endpoint

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 7 2025, 6:07 PM
Harbormaster failed remote builds in B382: Diff 620!
nannanko updated this revision to Diff 627.Jun 14 2025, 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.Jun 14 2025, 6:54 PM
Harbormaster failed remote builds in B387: Diff 627!
nannanko updated this revision to Diff 635.Jun 24 2025, 9:47 PM

Add tests

nannanko published this revision for review.Jun 24 2025, 9:48 PM
nannanko planned changes to this revision.
Harbormaster failed remote builds in B393: Diff 635!Jun 24 2025, 11:15 PM
nannanko requested review of this revision.Jun 28 2025, 7:01 PM
nannanko updated this revision to Diff 644.Jul 5 2025, 6:23 PM

Rebase to servant

Harbormaster failed remote builds in B400: Diff 644!Jul 5 2025, 6:38 PM
nannanko planned changes to this revision.Jul 5 2025, 7:18 PM
nannanko updated this revision to Diff 646.Jul 5 2025, 7:56 PM

Fixed CI

Harbormaster failed remote builds in B401: Diff 646!Jul 5 2025, 8:06 PM
nannanko planned changes to this revision.Jul 5 2025, 8:28 PM
nannanko updated this revision to Diff 653.Sat, Aug 9, 9:58 AM

Fixed tests.

Harbormaster failed remote builds in B406: Diff 653!Sat, Aug 9, 10:10 AM
nannanko updated this revision to Diff 654.Sat, Aug 9, 10:32 AM

Fixed qt quick tests

Harbormaster completed remote builds in B407: Diff 654.Sat, Aug 9, 10:50 AM

Revision Contents
Changeset List

Diff 609

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...