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

Config.js possible Regex issue #113

Closed
ilanbiala opened this issue Jul 13, 2014 · 2 comments
Closed

Config.js possible Regex issue #113

ilanbiala opened this issue Jul 13, 2014 · 2 comments

Comments

@ilanbiala
Copy link
Member

Shouldn't the regex on line 26 of config.js be ^(?:[a-z]+:)?\/ instead of ^(?:[a-z]+:)?//? If it's supposed to be a real URL like google.com it should probably be ^(?:[a-z]+:)?\/\/ to match ://.
@ilanbiala
Copy link
Member Author

Bump.

@ilanbiala
Copy link
Member Author

This was fixed at some point, but @roieki @amoshaviv it would be nice if I got a response at some point....

@ilanbiala ilanbiala reopened this Oct 26, 2014
@tejas002 tejas002 mentioned this issue Oct 10, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ilanbiala