-
Notifications
You must be signed in to change notification settings - Fork 13
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
Getting Error while Creating Nodes #31
Comments
Just found the problem, the |
Hey there! I'm glad you figured this out. import * as pino from 'pino';
const logger = pino();
export const neogma = new Neogma(
{
url: process.env.NEO4J_URL || '',
username: process.env.NEO4J_USERNAME || '',
password: process.env.NEO4J_PASSWORD || '',
},
{
encrypted: false,
logger: (v: string) => logger.info(v),
},
); However, the format of the logs is not right, which is brought to my attention now. It's an easy fix though, but I'll wait to see if I'm missing something with Pino which can be implemented. |
What I was trying is actually, something like this, import * as pino from 'pino';
const logger = pino();
export const neogma = new Neogma(
{
url: process.env.NEO4J_URL || '',
username: process.env.NEO4J_USERNAME || '',
password: process.env.NEO4J_PASSWORD || '',
},
{
encrypted: false,
logger: logger,
},
); Which makes the error, because it tries to call Also a little out of topic here, but I think the docs should explain about how to handle cyclical dependencies more, or at least gave some example. Just a suggestion. |
I see, I hadn't considered this level of logging but it should come in handy for sure. I agree with the documentation issue, I've had more people look for it. It's actually mentioned in the documentation but hard to find, so a separate section for it would be much better. |
While running this code,
I found myself with this error,
I've tried using Neo4J Desktop, Neo4J Sandbox, and Neo4J Aura and nothing does works, it produces the same error.
The text was updated successfully, but these errors were encountered: