-
Notifications
You must be signed in to change notification settings - Fork 53
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
Questions about EurekaConnection.generateURL() #9
Comments
These changes make sense to me. I'm not sure if @ryansb has any objections, though. |
Oops, I read and (thought) I'd 👍'd this already. In any case, sounds great. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems like generateURL() could be use to completely assemble the URL from its slug parts just by passing the parts as varargs. But none of the calling code actually uses this feature, and instead assembles most of the URL with a couple different approaches.
E.g. in RegisterInstance(), two approaches in combination are used, Sprintf and later string cat:
And it seems that using generateURL() to do the work would be simpler:
Second question: what purpose does the EurekaURLSlugs[] lookup serve if it only ever maps a string to a nearly identical string. Maybe we can just remove it? Then the above would be:
I can test this and prepare a PR if it makes sense to you.
The text was updated successfully, but these errors were encountered: