Skip to content
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

Generate test application when compose/testcontainers are selected for neo4j #1419

Closed
wants to merge 3 commits into from

Conversation

eddumelendez
Copy link
Contributor

It covers data-neo4j and spring-ai-vectordb-neo4j.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 28, 2024
@mhalbritter
Copy link
Contributor

Thanks for the PR! I had to set the password to notverysecret, otherwise the neo4j container would barge at me:

> docker compose up
[+] Running 1/0
 ✔ Container demo-neo4j-1  Created                                                                                                                                                                                                                                                                                 0.0s 
Attaching to neo4j-1
neo4j-1  | Invalid value for password. The minimum password length is 8 characters.
neo4j-1  | If Neo4j fails to start, you can:
neo4j-1  |   1) Use a stronger password.
neo4j-1  |   2) Set configuration dbms.security.auth_minimum_password_length to override the minimum password length requirement.
neo4j-1  |   3) Set environment variable NEO4J_dbms_security_auth__minimum__password__length to override the minimum password length requirement.
neo4j-1  | org.neo4j.commandline.admin.security.exception.InvalidPasswordException: A password must be at least 8 characters.
neo4j-1 exited with code 70

mhalbritter pushed a commit that referenced this pull request Feb 29, 2024
for neo4j

It covers data-neo4j and spring-ai-vectordb-neo4j.

See gh-1419
mhalbritter added a commit that referenced this pull request Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants