Fix non-get redirects
ClosedPublic
Actions

Authored by mollekopf on Sep 10 2025, 2:15 PM.

Project Tags

None

Referenced Files

	F117758344: D5583.1775209454.diff
	Thu, Apr 2, 9:44 AM

Subscribers

machniak

Details

Reviewers

machniak

Group Reviewers

Restricted Project

Commits

rK1ffe6cfd245b: Fix non-get redirects

Summary

If we do e.g. a PROPFIND that is redirected,
we want the redirect to also use PROPFIND, not GET.

I ran into this sometime during some dav migrations.

Diff Detail

Repository

rK kolab

Lint

Lint Skipped

Unit

No Test Coverage

Build Status

Buildable 55125
Build 19593: arc lint + arc unit

Event Timeline

mollekopf requested review of this revision.Sep 10 2025, 2:15 PM

mollekopf created this revision.

Harbormaster completed remote builds in B55125: Diff 15999.Sep 10 2025, 2:15 PM

mollekopf added a reviewer: Restricted Project.Sep 10 2025, 2:24 PM

mollekopf edited the summary of this revision. (Show Details)

machniak accepted this revision.Sep 11 2025, 8:22 AM

machniak subscribed.

machniak added inline comments.

src/app/Providers/AppServiceProvider.php
284	I'd just move this block up, so it's together with other Http related code.

This revision is now accepted and ready to land.Sep 11 2025, 8:22 AM

Closed by commit rK1ffe6cfd245b: Fix non-get redirects. · Explain WhySep 11 2025, 12:05 PM

This revision was automatically updated to reflect the committed changes.

mollekopf added a commit: rK1ffe6cfd245b: Fix non-get redirects.

Revision Contents
Changeset List

Path

Size

src/

app/

Providers/

AppServiceProvider.php

lines

Diff 15999

View Options

src/app/Providers/AppServiceProvider.php

Fix non-get redirectsClosedPublicActions