fix bug 1295949 for public artifacts (#143)

* fix bug 1295949 for public artifacts

* Addressed feedback from @dustin @eliperelman

src/lib/ui/artifactlist.jsx

@@ -140,19 +140,20 @@ export default React.createClass({
  let icon;
 
  if (/^public\//.test(artifact.name)) {
+ url = 'https://queue.taskcluster.net/v1/task/';
  if (this.props.runId != null) {
- url = this.queue.buildUrl(
- this.queue.getArtifact,
- this.props.taskId,
- this.props.runId,
- artifact.name
- );
+ url += `${this.props.taskId}/runs/${this.props.runId}/artifacts/${artifact.name}`;
+ // We could use queue.buildUrl as follows, but this creates URLs where artifact name
+ // has slashes encoded. For artifacts we specifically allow slashes in the name not
+ // to be encoded, as this make things like: $ wget <url> create files with nice names.
+ // url = this.queue.buildUrl(
+ // this.queue.getArtifact,
+ // this.props.taskId,
+ // this.props.runId,
+ // artifact.name
+ // );
  } else {
- url = this.queue.buildUrl(
- this.queue.getLatestArtifact,
- this.props.taskId,
- artifact.name
- );
+ url += `${this.props.taskId}/artifacts/${artifact.name}`;
  }
 
  icon = getIconFromMime(artifact.contentType);