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

Not working with latest version of sinatra #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

devstator82
Copy link

Hi guys,

Don't know if you noticed or wether this happens to be something specific for me, but using the latest version of sinatra (1.2.6) cloud-crowd was not running due to deprecation of Sinatra::Default

Here is a fix if it helps.

Waseem

@forgotpw1
Copy link

I saw and implemented some of these cloud-crowd commits from back in the day. Thanks for putting those out there. They helped me get CloudCrowd live on an instance of mine, and most importantly, use https to transfer the files. Ruby 1.8.7 has an exception when trying to use the https.

You probably are aware that it messes up the "operation center," even though it does process jobs (I think).

On the front end the operation center, the javascript fails here.

this._nodes[i].workers.length;

Is there some reason why workers is getting set to nil on the backend?

I'll be looking into this a little more. Maybe there's a nil being set instead of an empty set that would probably not break the op center.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants