New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to delete malformed DSN (empty) server from list #108

Closed
andrewryno opened this Issue May 1, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@andrewryno

It will delete but upon refresh it will reappear.

Possibly disallow empty values?

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow May 1, 2013

Owner

Thanks for reporting this. I'll get a fix into the next release.

Owner

bobthecow commented May 1, 2013

Thanks for reporting this. I'll get a fix into the next release.

bobthecow added a commit that referenced this issue May 2, 2013

Add failing tests for #108
Adding malformed or empty server DSNs should return 400.

bobthecow added a commit that referenced this issue May 2, 2013

bobthecow added a commit that referenced this issue May 2, 2013

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow May 2, 2013

Owner

Okay. This has been fixed in develop. And it'll even clean up your old broken entry while it's at it :)

Owner

bobthecow commented May 2, 2013

Okay. This has been fixed in develop. And it'll even clean up your old broken entry while it's at it :)

@bobthecow bobthecow closed this in 97f7168 May 2, 2013

@bobthecow

This comment has been minimized.

Show comment
Hide comment
@bobthecow

bobthecow May 2, 2013

Owner

And released. Thanks again!

Owner

bobthecow commented May 2, 2013

And released. Thanks again!

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