neondatabase / serverless

Connect to Neon PostgreSQL from serverless/worker/edge functions
https://www.npmjs.com/package/@neondatabase/serverless
MIT License
318 stars 11 forks source link

Broken links pointing to non-existing parts of the README #49

Closed Jolg42 closed 10 months ago

Jolg42 commented 10 months ago

In the source code, you will find these 2 links in error messages:

They point to the README, but the headline they are pointing to doesn't seem to exist anymore?

(at Prisma we use a URL shortener where we can update the link it points to for that use-case)

jawj commented 10 months ago

Thanks @Jolg42.

The first error message won't often be thrown any more (because a default proxy address is always set), but the second one will be. I've updated the links, so they'll be in the next package version.

janpio commented 9 months ago

It looks to me that the All attempts... message is also thrown when no neonConfig.webSocketConstructor is set when using Node. (Latest published version still uses https://github.com/neondatabase/serverless#run-on-node)

jawj commented 9 months ago

Thanks Jan. I wasn't sure this was important enough to make a release for, so I was just going to include this update in the next release.