Opened 13 years ago
Closed 8 years ago
#1568 closed enhancement (invalid)
dojo.uri.Uri does not clean up path in first argument
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | low | Milestone: | future |
| Component: | General | Version: | 0.3 |
| Keywords: | Cc: | eingb@… | |
| Blocked By: | Blocking: |
Description (last modified by )
If dojo.uri.Uri is only passed one argument, no cleanup of the path is done:
var uri = new dojo.uri.Uri("/foo/../bar"); dojo.debug(uri.toString()) prints /foo/../bar instead of /bar
Possible solutions
- Document that dojo.uri.Uri should always be called with at least a period as the second argument in order to enforce a path cleanup ;)
- Put the code form lines 58 - 70 into a separate helper function (say cleanPath) and call that in line 91 like so
this.path = cleanPath(r[5]);
Change History (7)
comment:1 Changed 13 years ago by
| Milestone: | → 0.6 |
|---|
comment:2 Changed 12 years ago by
| Owner: | changed from anonymous to James Burke |
|---|
comment:3 Changed 12 years ago by
| Milestone: | 1.0 → 1.1 |
|---|
comment:4 Changed 12 years ago by
| Milestone: | 1.1 → 1.2 |
|---|
comment:5 Changed 11 years ago by
| Description: | modified (diff) |
|---|---|
| Milestone: | 1.2 → future |
comment:6 Changed 8 years ago by
| Owner: | changed from James Burke to eingb@… |
|---|---|
| Status: | new → pending |
is this ticket still relevant? it will automatically close in 14 days if there is no response.
comment:7 Changed 8 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
is this a bug?