Skip to content
This repository has been archived by the owner on Mar 27, 2019. It is now read-only.

Remove trailing slashes in vault address before sending request #104

Merged

Conversation

msessa
Copy link
Collaborator

@msessa msessa commented Apr 26, 2017

Fixes #103.

I opted for a one-liner fix to the specific issue instead of including a whole new dependency to handle URL joining.

Copy link
Owner

@djenriquez djenriquez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful, didn't test tbh but looks simple enough =P

@djenriquez
Copy link
Owner

Let's release a hotfix, 2.1.1, for this.

@msessa
Copy link
Collaborator Author

msessa commented Apr 26, 2017

Sure, but before you do give me a couple of hours to look into that UI bug where the icon for a RaisedButton doesn't correctly show. I'd like to get that one into the hotfix release too

Ex:

image

@djenriquez
Copy link
Owner

Of course man, no rush. Such is the beauty of an open-source project =P.

@djenriquez
Copy link
Owner

@msessa going to merge this in for 2.2.0, feel free to open up a new PR for the weird UI bug you mentioned above.

@djenriquez djenriquez merged commit 13a41dc into djenriquez:master Jun 14, 2017
@msessa
Copy link
Collaborator Author

msessa commented Jun 14, 2017

Nah that was a glitch on my dev station probably because of some stale cache. all good 👍

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

Successfully merging this pull request may close these issues.

Wrong /v1/sys/capabilities-self HTTP method
2 participants