Moodle

Add features to provide easier WebDAV access

Details

  • Type: New Feature New Feature
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.0.9
  • Fix Version/s: None
  • Component/s: General
  • Labels:
    None
  • Environment:
    All
  • Affected Branches:
    MOODLE_10_STABLE

Description

  • Change the directory name from the current course ID number to the course shortname.
  • add better checks and balances so that the teacher can change the course shortname and the directory will change to.
  • keep internal file references using the course ID (translate to course shortname on the fly)
  • add some mechanism to safely expose the course directories from within the moodle web site (ie every course has a very restrictive .htaccess file, which is upgraded on demand with one that prompts for the teacher's username and password)
  1. 0001-WebDAV-add-proper-security-checks-to-the-PROPFIND-m.patch
    09/Jun/09 3:57 PM
    1.0 kB
    Ashley Holman
  2. webdav.patch
    27/Feb/08 1:40 PM
    48 kB
    Martín Langhoff
  3. webdav.patch
    27/Feb/08 1:23 PM
    0.7 kB
    Martín Langhoff
  4. webdav-01.patch
    27/Feb/08 2:06 PM
    45 kB
    Martín Langhoff
  5. webdav-applyto19.patch
    19/Mar/08 12:12 AM
    74 kB
    Dan Poltawski

Issue Links

Activity

Hide
Martin Dougiamas added a comment -

From Martin Dougiamas (martin at moodle.com) Friday, 11 July 2003, 04:46 PM:

Moving this out to version 2.0 ... fits with the whole resource management rethink.

Show
Martin Dougiamas added a comment - From Martin Dougiamas (martin at moodle.com) Friday, 11 July 2003, 04:46 PM: Moving this out to version 2.0 ... fits with the whole resource management rethink.
Hide
Slobodan Jovcic added a comment -

Is this, and WebDAV support in general, still on track for 2.0?

Show
Slobodan Jovcic added a comment - Is this, and WebDAV support in general, still on track for 2.0?
Hide
Martín Langhoff added a comment -

We have a good patch, as seen in this thread http://moodle.org/mod/forum/discuss.php?d=89123

Show
Martín Langhoff added a comment - We have a good patch, as seen in this thread http://moodle.org/mod/forum/discuss.php?d=89123
Hide
Martín Langhoff added a comment -

The plan is as follows:

  • Merge into 1.9 the bits needed to support WebDAV, but not the /webdav directory which implements the feature (perhaps this should be delayed until 1.9.1)
  • Merge into HEAD all of it
Show
Martín Langhoff added a comment - The plan is as follows:
  • Merge into 1.9 the bits needed to support WebDAV, but not the /webdav directory which implements the feature (perhaps this should be delayed until 1.9.1)
  • Merge into HEAD all of it
Hide
Martín Langhoff added a comment -

This is the patch for 1.9 on top of the patches merged today.

Show
Martín Langhoff added a comment - This is the patch for 1.9 on top of the patches merged today.
Hide
Martín Langhoff added a comment -

Here is a cleaner patch. With the parts I already landed in 19_STABLE and HEAD out of the way, and MD putting the locks table in lib/db, this is what we'll land in HEAD.

Show
Martín Langhoff added a comment - Here is a cleaner patch. With the parts I already landed in 19_STABLE and HEAD out of the way, and MD putting the locks table in lib/db, this is what we'll land in HEAD.
Hide
Petr Škoda (skodak) added a comment -

please use xmldb editor to produce valid install.xml - it broke install, going to fix it now

Show
Petr Škoda (skodak) added a comment - please use xmldb editor to produce valid install.xml - it broke install, going to fix it now
Hide
Petr Škoda (skodak) added a comment -

another blocker - we must include our pear libs first because we rely on specific version of quickforms

Show
Petr Škoda (skodak) added a comment - another blocker - we must include our pear libs first because we rely on specific version of quickforms
Hide
Petr Škoda (skodak) added a comment -

pear include fixed in cvs

Show
Petr Škoda (skodak) added a comment - pear include fixed in cvs
Hide
Martín Langhoff added a comment -

Petr

  • sorry about the install.xml breakage! - I did generate them with XMLDB, but it may have gotten broken along the way (ah! I see, I changed the field name by hand!)
  • PEAR inclusion order - sorry again! - probably depends on having (or not) local PEAR libs – I have been using/testing this for awhile, and I didn't see any forms break.

Thanks for the fixes!

Show
Martín Langhoff added a comment - Petr
  • sorry about the install.xml breakage! - I did generate them with XMLDB, but it may have gotten broken along the way (ah! I see, I changed the field name by hand!)
  • PEAR inclusion order - sorry again! - probably depends on having (or not) local PEAR libs – I have been using/testing this for awhile, and I didn't see any forms break.
Thanks for the fixes!
Hide
Dan Poltawski added a comment -

Martín,

Have you packaged up your 1.9-compatible 'drop-in' webdav folder anywhere?

Show
Dan Poltawski added a comment - Martín, Have you packaged up your 1.9-compatible 'drop-in' webdav folder anywhere?
Hide
Dan Poltawski added a comment -

Here is a patch which I think is what is required for applying straight onto 1.9

Show
Dan Poltawski added a comment - Here is a patch which I think is what is required for applying straight onto 1.9
Hide
Matt Gibson added a comment -

just tried the patch on 1.9.1+ from CVS and got an error saying that /lib/pear/console/getopt.php already existed. I chose no for using -R and then said apply anyway. Seems to have installed cleanly otherwise and the settings came up when I went to the notifications page.

However, when I go to http://mymoodle.com/webdav/moodledata-server.php in firefox, I just get a blank page and I also the same in konqueror. Debug is set to max, but still no error.

Does this mean its broken or have I missed something?

Show
Matt Gibson added a comment - just tried the patch on 1.9.1+ from CVS and got an error saying that /lib/pear/console/getopt.php already existed. I chose no for using -R and then said apply anyway. Seems to have installed cleanly otherwise and the settings came up when I went to the notifications page. However, when I go to http://mymoodle.com/webdav/moodledata-server.php in firefox, I just get a blank page and I also the same in konqueror. Debug is set to max, but still no error. Does this mean its broken or have I missed something?
Hide
Matt Gibson added a comment -

Trying webdav://mymoodle.com/webdav/moodledata-server.php in konqueror gives me a 'does not exist' error, even though the file is clearly present.

Show
Matt Gibson added a comment - Trying webdav://mymoodle.com/webdav/moodledata-server.php in konqueror gives me a 'does not exist' error, even though the file is clearly present.
Hide
Matt Gibson added a comment -

oops, it was magic_quotes_gpc set to on. works fine now!

Show
Matt Gibson added a comment - oops, it was magic_quotes_gpc set to on. works fine now!
Hide
Dan Poltawski added a comment -

Cool, for others info, you must check your apache error log to find errors, since inline errors are not supported

Show
Dan Poltawski added a comment - Cool, for others info, you must check your apache error log to find errors, since inline errors are not supported
Hide
Wojciech Cieszynski added a comment -

Hi everyone,

please take apologise for my question but could you please confirm that this patch works with windows Moodle (1.9+) and if could you please advise how to implement it?

Many thanks in advance
Wojciech

Show
Wojciech Cieszynski added a comment - Hi everyone, please take apologise for my question but could you please confirm that this patch works with windows Moodle (1.9+) and if could you please advise how to implement it? Many thanks in advance Wojciech
Hide
Lael... added a comment -

Hi,

I've tried to apply the patch using git on windows, but get some complaints about trailing whitespaces, and then errors stating 'patch does not apply'. (for serve.php, index.php, admin.php, moodlelib.php and then that Getopt.php already exists.

Is it possible to get a fresh patch based off of catalyst's repo? (I am assuming the problem is that there have been changes to these files since the patch was created?)

Lael

Show
Lael... added a comment - Hi, I've tried to apply the patch using git on windows, but get some complaints about trailing whitespaces, and then errors stating 'patch does not apply'. (for serve.php, index.php, admin.php, moodlelib.php and then that Getopt.php already exists. Is it possible to get a fresh patch based off of catalyst's repo? (I am assuming the problem is that there have been changes to these files since the patch was created?) Lael
Hide
Ashley Holman added a comment -

Here's a patch which adds proper access control to the PROPFIND method.

We found that people using a DAV client were able to manually type in the course ID of a course they do not teach, and were able to view the files (but not read or modify them).

Thanks

Show
Ashley Holman added a comment - Here's a patch which adds proper access control to the PROPFIND method. We found that people using a DAV client were able to manually type in the course ID of a course they do not teach, and were able to view the files (but not read or modify them). Thanks

Dates

  • Created:
    Updated: