restivus breaks iron:router on deployed apps about meteor-restivus

[kahmali]

Alright, let me look into this. I have the latest version of Restivus in an app that uses Iron Router fairly extensively, and it's deployed on both meteor and digital ocean and both are working fine. There must be something else to the error besides simply having Restivus in the deployed app. I absolutely hate those Iron Router errors. I get them all the time and the strangest things seem to cause them. So frustrating! I'll check out your repro apps and keep you posted. Thanks for reporting the issue.

[hujhax]

Thanks! And let me know if there's anything I can do to help.

[kahmali]

I think you made your test repo private. Could you please add me as a collaborator or make it public so I can check it out?

[hujhax]

Derp! I am not smarting well today. It's public now.

[kahmali]

I was able to get it to run when deployed on meteor by making the call to Restivus.add() from within Meteor.startup(). So in lib/restivus.js:
Change:

Restivus.add('test', {
    get: {
        action: function() {
            return "Hello World!";
        }
    }
});

to this:

Meteor.startup(function () {
    Restivus.add('test', {
        get: {
            action: function() {
                return "Hello World!";
            }
        }
    });
});

Would you be able to test that on Heroku when you have a chance and let me know if it works? I will make sure I add that to the docs. I've always done it that way, but I need to say it explicitly in the docs. Sorry about that!

[hujhax]

We're good on heroku! Thanks so much!

[kahmali]

Alright, I updated the docs with that information. Sorry if you were alerted each time I rebased that commit. I have to push to test the GitHub markdown, and I was just making some minor cosmetic updates so I had to squash the commits.