Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Invalid URL for blob has sniff #318

Closed
agubler opened this issue Apr 11, 2017 · 4 comments
Closed

Invalid URL for blob has sniff #318

agubler opened this issue Apr 11, 2017 · 4 comments
Assignees
@agubler
Labels
bug
Milestone
2017.04

Comments

@agubler
Copy link
Member

agubler commented Apr 11, 2017

Bug

The URI of / is not valid for the blob sniff in has.ts, which causes a failure running tests for projects built with webpack.

This should be updated to /foo (or another valid URI)
@agubler agubler self-assigned this Apr 11, 2017
@agubler agubler added this to the 2017.04 milestone Apr 11, 2017
@agubler agubler mentioned this issue Apr 11, 2017
Merged
3 tasks
@msssk
Copy link
Contributor

msssk commented Apr 11, 2017

This should use an absolute URI, see dojo/dojo#260

@agubler
Copy link
Member Author

agubler commented Apr 11, 2017 •
edited
Loading

@msssk I did see that fix afterwards as @dylans posted in dojo2, this is to make it work in web workers?

@agubler
Copy link
Member Author

agubler commented Apr 11, 2017

@msssk please can you open a new issue to update this for web workers?

@msssk
Copy link
Contributor

msssk commented Apr 11, 2017

@agubler Done: #320

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@agubler agubler

Labels
bug
Projects
None yet
Milestone
2017.04
Development

No branches or pull requests
2 participants
@msssk @agubler