Skip to content

Add possibility to follow redirects at options request #76

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add possibility to follow redirects at options request #76

wants to merge 3 commits into from

Conversation

sunkup
Copy link
Member

@sunkup sunkup commented Jul 15, 2025 •
edited
Loading

Add possibility to enable redirect following to the options request method.

For bitfireAT/davx5-ose#1582

@sunkup sunkup self-assigned this Jul 15, 2025
@sunkup sunkup added the enhancement New feature or request label Jul 15, 2025
@sunkup sunkup mentioned this pull request Jul 15, 2025
Draft
4 tasks
@sunkup sunkup changed the title Implement WebDAV detection with redirect support Add WebDAV detection with redirect support to DavResource Jul 15, 2025
@sunkup sunkup requested a review from ArnyminerZ July 15, 2025 12:46
@sunkup sunkup moved this from Todo to In Review in DAVx⁵ Releases Jul 16, 2025
rfc2822
rfc2822 reviewed Jul 16, 2025
View reviewed changes
Copy link
Member

@rfc2822 rfc2822 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's already an options() method and the main difference seems to be the redirect handling.

Maybe we could add a followRedirects: Boolean argument to options() instead to avoid having multiple methods doing similar things?

@sunkup sunkup changed the title Add WebDAV detection with redirect support to DavResource Add possibility to follow redirects at options request Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rfc2822 rfc2822 rfc2822 left review comments

@ArnyminerZ ArnyminerZ Awaiting requested review from ArnyminerZ

At least 1 approving review is required to merge this pull request.

Assignees

@sunkup sunkup

Labels
enhancement New feature or request
Projects
DAVx⁵ Releases
Status: In Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sunkup @rfc2822