Use generated id in the returned url, if in use

[jonasfj]

Seems like new_client may overwrite the req_id with a generated value, if the id already is in use by another client.

This causes an inconsistency between id and url. This is just a quick fix to make it consistent.

I suspect a proper fix would be some sort of secret_token returned the client that reserves the id. So that other clients can't use it. But that might be overkill, keeping the service simple seems like a good choice :)

[defunctzombie]

The fix here is to actually return the url that corresponds to the id assigned to the client (in the case where they requested an id which was not available?)

[jonasfj]

Yeah, that what this PR does... It returns the randomly generated id, if the id requested is in use... Den 18. mar. 2017 5.23 PM skrev "Roman Shtylman" <notifications@github.com>:

…

The fix here is to actually return the url that corresponds to the id assigned to the client (in the case where they requested an id which was not available?) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#59 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAJI5BZdFOfqF7f6Zl5dQRjfiY93WS6mks5rnHV_gaJpZM4Mhj8F> .