Skip to content

Fix: Simpler approach to get rid of undefined method `scheme' for nil:Ni...#3

Open
johvet wants to merge 9 commits intoRISCfuture:masterfrom
johvet:master
Open

Fix: Simpler approach to get rid of undefined method `scheme' for nil:Ni...#3
johvet wants to merge 9 commits intoRISCfuture:masterfrom
johvet:master

Conversation

@johvet
Copy link

@johvet johvet commented Mar 19, 2015

...lClass

I came across this problem when trying to validate an URL like ://example.com/or/this.rss.
An Addressable::URI::InvalidURIError is properly raised but when figuring out what error message to create, it dies because the lack of an URI instance.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants