-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Labels
kind/bugOutline of a bug - must adhere to the bug report template.Outline of a bug - must adhere to the bug report template.severity/P2Has a minor but important impact to the usage or development of the system.Has a minor but important impact to the usage or development of the system.
Description
In the class org.eclipse.che.ide.rest.UrlBuilder, there are methods to get the path, host, etc. They all compute their value something like so:
return o.get("path").toString();
I believe this to in an unintended way, because if the path is, effectively "/foo/bar", the path value returned is indeed ""/foo/bar"". The "toString()" method on JsonString adds extra quotadion marks around the value. The methods don't seem to be used anywhere currently.
I would not expect this to happen, and the behaviour is not mentioned in the Javadoc. Also, it's easier to add the quotation marks if necessary than to remove them afterwards.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
kind/bugOutline of a bug - must adhere to the bug report template.Outline of a bug - must adhere to the bug report template.severity/P2Has a minor but important impact to the usage or development of the system.Has a minor but important impact to the usage or development of the system.