Cloud Files contains a FileStreamWrapper for Rackspace Cloud Files. It is similar to the Media: Amazon module, only for Rackspace Cloud Files instead of AWS S3.

I did not find any other module that uses Rackspace Cloud Files, other then the URL rewriting modules, but those dont actually upload the files to Rackspace Cloud. This module uses the new support for FileStreamWrappers in Drupal 7.

Here is the sandbox project: http://drupal.org/sandbox/mr.moses/1157816

I had planned on adding support to other cloud storage services (Google Storage), but might create separate modules instead.

Comments

mr.moses’s picture

mr.moses commented

whoops, heres the Media: Amazon module

mr.moses’s picture

mr.moses commented

needs review

berkas1’s picture

berkas1 commented
Priority: Normal » Critical

subscribing

looking forward to release:)

mr.moses’s picture

mr.moses commented
Priority: Critical » Normal

review please?

mr.moses’s picture

mr.moses commented
Priority: Normal » Critical
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Priority: Critical » Normal
Status: Needs review » Needs work

On review this looks really well put together. I can't really see any security risks, and while it would be nice to have someone who actually uses Cloud Files test this out, I don't feel its necessary.

On your thought about adding support for other stream wrappers, I would suggest against that and make them separate modules for each service, especially if they're not related.

The only thing I see is an issue I've already filed: #1309056: The 'access administration pages' is not secure enough for the module administration page. Once that is fixed I think this is ready to be approved.

mr.moses’s picture

mr.moses commented

Thanks for the review!

I've changed the access parameter for the config page, per your suggestion.

I've also changed the library folder from 'rackspacecloudfiles' to 'rackspacecloud'
My thinking here is that I know Rackspace has a few php libraries, so a 'rackspacecloud' folder would make sense to contain them all (if other modules were to use them).

I've already started working on another stream wrapper for another service, so be sure to watch for...Google Storage!

Thanks again for the review!

mr.moses’s picture

mr.moses commented
Status: Needs work » Needs review
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Reviewed & tested by the community

Great job! There's a few minor things like using proper branches like 7.x-1.x rather than using 'master', but otherwise this looks ready to go. Very nice job again (this coming from someone who's written a couple stream wrappers already!)

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Thanks for your contribution, mr.moses! Welcome to the community of project contributors on drupal.org.

I've granted you the git vetted user role which will let you promote this to a full project and also create new projects as either sandbox or "full" projects depending on which you feel is best.

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

As you continue to work on your module, keep in minde: Commit messages - providing history and credit and Release naming conventions.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

oops

mr.moses’s picture

mr.moses commented

@David Reid, thanks again for the review!

I've created a 7.x-1.x branch, but apparently its not possible to remove the master branch (#1074960: Let maintainers set a default branch on git repositories)

So I followed these instructions (http://drupal.org/node/1127732) to empty the master branch and point to the other branches:

@greggles, thanks for the full project permission and for the links/tips!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.