Skip to content

Found a bug. #19

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

Merged
1 commit merged into from
Mar 8, 2011
Merged

Found a bug. #19

1 commit merged into from
Mar 8, 2011

Conversation

bjornblomqvist
Copy link

I couldn't fix it, but i did write a test that triggers it.

I hope this helps.

Best Regards Björn Blomqvist

…ug by making the query smaller (char count).
@brianc
Copy link
Owner

brianc commented Mar 7, 2011

Ah thanks so much! I'll get this looked at & fixed today.

@brianc
Copy link
Owner

brianc commented Mar 8, 2011

I merged your test in (can never have too many tests, right?!) and added another one to test the scenario even harder, but still I'm unable to reproduce on any of the following environments:

  • ubuntu with postgres 8.4, node 0.4.0, 0.4.1, 0.4.2
  • os x with postgres 9.x, node 0.2.5, 0.3.7, 0.4.1, 0.4.2

Which version of node, postgres, and which OS are you on?

@bjornblomqvist
Copy link
Author

PostgreSQL 9.0.1 on x86_64-apple-darwin10.5.0, compiled by GCC i686-apple-darwin10-gcc-4.2.1 (GCC) 4.2.1 (Apple Inc. build 5664), 64-bit

node v0.3.1

I only get the error every 4 - 8 time i run the test. I know its a bit crazy..

@bjornblomqvist
Copy link
Author

I will upgrade to latest node later today and try again.

@bjornblomqvist
Copy link
Author

I cant recreate it in 0.4.2.. So i guess it had something to do with node.

I hope you didn't spend to much time on this!

brianc added a commit that referenced this pull request Dec 18, 2019
Fix require for webpack compatibility
brianc pushed a commit that referenced this pull request Dec 27, 2019
* Added URL parsing example

* Typos and cleanup

* Last typo
This pull request was closed.
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