Request URLs created with superfluous trailing slash

[z5h]

When a URL is created a trailing slash will be appended even if one is not required by the spec. e.g. A delete push. The culprit code is here.

Any suggestions on the preferred way to fix this?

[anthonator]

It doesn't look like it even has a purpose. Would you mind removing it, testing and then creating a PR if it works?

[z5h]

To stick to the API exactly (which I think is preferred to guarantee compatibility & safety) we have to honour some calls which need the trailing slash, and others that don't.
e.g.
http://docs.urbanairship.com/api/ua.html#list-schedules vs. http://docs.urbanairship.com/api/ua.html#validate

So having the slash is a function of the specific call being made. It should be part of the path parameter passed in. I can change all calls to properly include or exclude as required by Urban Airship.

For example, this line

dirigible/lib/dirigible/schedule.rb

Line 22 in 829b265

Dirigible.post('/schedules', params)

would become
Dirigible.post('/schedules/', params)

If that sounds OK I'll proceed with the work.

[anthonator]

Ah, gotcha.

Go ahead if you have time. Thanks for the help!

[z5h]

I can likely get to this next week (May 25-29).

[anthonator]

Awesome. It doesn't seem to be breaking anything so no rush. But I agree, should probably stick to the spec.