Differential D24315

REPORT also supports the Depth header
ClosedPublic
Actions

Authored by vkrause on Sep 30 2019, 4:57 PM.

Details

Reviewers
dfaure
Commits
R241:ac0462180c0d: REPORT also supports the Depth header
Summary

See RFC 3253 Section 3.6.

Setting the depth for REPORT currently requires dirty workarounds in
application code, as can be seen in KDAV::DavManager::createReportJob.

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
vkrause created this revision.Sep 30 2019, 4:57 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptSep 30 2019, 4:57 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
vkrause requested review of this revision.Sep 30 2019, 4:57 PM
dfaure accepted this revision.Sep 30 2019, 6:31 PM

This is exactly why we release KF5 every month these days, so people fix issues here instead of working around them in app code like in 2010 :-)

This revision is now accepted and ready to land.Sep 30 2019, 6:31 PM
Closed by commit R241:ac0462180c0d: REPORT also supports the Depth header (authored by vkrause). · Explain WhyOct 1 2019, 4:46 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/ioslaves/http/http.cpp (2 lines)

Diff 67141

View Options

src/ioslaves/http/http.cpp

Loading...