Skip to content
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.

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

basic WebDAV test in place now #1481

Merged
merged 5 commits into from
Feb 9, 2013
Merged

basic WebDAV test in place now #1481

merged 5 commits into from
Feb 9, 2013

Conversation

DeepDiver1975
Copy link
Member

Very first attempt to fulfill the requirements defined in #1354.

/**
 * we test if webDAV is working properly
 *
 * The basic assumption is that if the server returns 401/Not Authenticated for an unauthenticated PROPFIND
 * the web server it self is setup properly.
 *
 * Why not an authenticated PROFIND and other verbs?
 *  - We don't have the password available
 *  - We have no idea about other auth methods implemented (e.g. OAuth with Bearer header)
 *
 */

@danimo @bartv2 @karlitschek @icewind1991 @tanghus THX

@icewind1991
Copy link
Contributor

Seems reasonable

👍

@bartv2
Copy link
Contributor

bartv2 commented Feb 6, 2013

Can we do this in lib/setup? that would be a better place for this check.

@DeepDiver1975
Copy link
Member Author

sorry for the missing info - as described here: #1354 (comment)

the webdav interface is only functional after installation - that's why I did choose this solution.

@bartv2
Copy link
Contributor

bartv2 commented Feb 6, 2013

isn't the install finished at https://github.com/owncloud/core/blob/master/core/setup.php#L46? we can do the check there

@DeepDiver1975
Copy link
Member Author

but if you reload it will no longer popup

@bartv2
Copy link
Contributor

bartv2 commented Feb 6, 2013

@DeepDiver1975 what about this?

@DeepDiver1975
Copy link
Member Author

nice solution with the routing - I need to get into this ;-)

but still nothing prevents the user from navigating to the index.php or root url and the check is gone forever

@bartv2
Copy link
Contributor

bartv2 commented Feb 6, 2013

true, but they say the message, and chose to continue. But we could add a check to the admin screen. Maybe they don't need/want the webdav access

@DeepDiver1975
Copy link
Member Author

seems legit - I'll add the warning to the admin section

@DeepDiver1975
Copy link
Member Author

@bartv2 warning in admin section in place and rebased - wana check it out? THX

@bartv2
Copy link
Contributor

bartv2 commented Feb 7, 2013

I just noticed the hardcoded doc url, we need to centralize that and make the version dependend on the version in OC_Util. But I think a new PR for that is better.

+1 for this version

@danimo
Copy link
Contributor

danimo commented Feb 7, 2013

Yes, the doc situation needs to be sorted out. @karlitschek was working on integrating the docs for 5.0. Mabye he has a plan. Anyway, the rest looks good, can't test it right now though. Has anyone else except @DeepDiver1975? Otherwise I'd reserve some time later. Which servers were tested? Only IIS? Does Apache have this problem at all?

@karlitschek
Copy link
Contributor

That's tricky. The shipped docu is in an iframe so we can't deeplink to it properly. But we could link to the online documentation of course.

@DeepDiver1975
Copy link
Member Author

@danimo curl test included

@karlitschek
Copy link
Contributor

👍

DeepDiver1975 added a commit that referenced this pull request Feb 9, 2013
@DeepDiver1975 DeepDiver1975 merged commit 279cbeb into master Feb 9, 2013
@DeepDiver1975 DeepDiver1975 deleted the fixing-1354-master branch February 9, 2013 13:13
@lock lock bot locked as resolved and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants