When url ends with / overwrite dbname with empty string #40

Merged
merged 2 commits into from Sep 9, 2012

Conversation

Projects
None yet
2 participants
@carlo-colombo
Contributor

carlo-colombo commented Jun 27, 2012

This error due to "/".split("/") == ["",""] so pathname[1] == "" and overwrite dbname even if it is configured

@kcbanner

This comment has been minimized.

Show comment Hide comment
@kcbanner

kcbanner Sep 9, 2012

Collaborator

Thanks for this!

Collaborator

kcbanner commented Sep 9, 2012

Thanks for this!

kcbanner added a commit that referenced this pull request Sep 9, 2012

Merge pull request #40 from carlo-colombo/master
When url ends with / overwrite dbname with empty string

@kcbanner kcbanner merged commit fc5f32a into jdesboeufs:master Sep 9, 2012

kcbanner added a commit that referenced this pull request Sep 9, 2012

Merge pull request #40 from carlo-colombo/master
When url ends with / overwrite dbname with empty string
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment