feat: auth support with dockerfile

[vinitparekh17]

This PR ensures that dicedb will be used with proper username and password to ensure security

[vinitparekh17]

@lucifercr07, have a look

[rishavvajpayee]

added a comment @vinitparekh17 .

Hey @vinitparekh17 can you also go ahead and add the docker-compose file that builds the BE depending on DiceDB image: https://hub.docker.com/r/dicedb/dicedb as DB dependency. You already have Dockerfile in place that LGTM! 👍

[vinitparekh17]

added a comment @vinitparekh17 .

Hey @vinitparekh17 can you also go ahead and add the docker-compose file that builds the BE depending on DiceDB image: https://hub.docker.com/r/dicedb/dicedb as DB dependency. You already have Dockerfile in place that LGTM! 👍

done

[lucifercr07]

Let's use DICEDB everywhere.

[vinitparekh17]

pardon?

[rishavvajpayee]

he means the naming convention is dicedb and not dice.
like below:
environment:
- DICEDB_ADDR=dicedb:7379

[lucifercr07]

Please update the README with details around docker compose.

[vinitparekh17]

Please update the README with details around docker compose.

it's just a basic cmd docker compose up what to write about that?

[rishavvajpayee]

@vinitparekh17 we need documention around docker-compose as oppose to the people setting up this repo can just read the ReadMe.md file and do it. let me know if you need any help around it.

[vinitparekh17]

@vinitparekh17 we need documention around docker-compose as oppose to the people setting up this repo can just read the ReadMe.md file and do it. let me know if you need any help around it.

done

[vinitparekh17]

ready for review

[vinitparekh17]

ready for review

@lucifercr07 @rishavvajpayee

[lucifercr07]

@vinitparekh17 apologies for delay, will review it by tomorrow EOD.

[lucifercr07]

LGTM, minor comments.
Please resolve conflicts.

[lucifercr07]

Also the backend container is failing on spawn, please check once.

backend-1  | Warning: .env file not found, falling back to system environment variables.
dicedb-1   | 2024/10/09 17:43:52 INFO DiceDB 0.0.4 running on port 7379
dicedb-1   | 2024/10/09 17:43:52 INFO HTTP Server running on Port:8082
dicedb-1   | 2024/10/09 17:43:52 WARN WARNING: DiceDB is running without authentication. Consider setting a password.
backend-1  | 2024/10/09 17:43:55 ERROR Failed to initialize DiceDB client: %v err="dial tcp 127.0.0.1:7379: connect: connection refused"
backend-1 exited with code 1

[vinitparekh17]

LGTM, minor comments. Please resolve conflicts.

done

[vinitparekh17]

issue ref

[lucifercr07]

issue ref

@vinitparekh1 can you add the detailed steps in README, would be helpful for folks working on project.

[vinitparekh17]

issue ref

@vinitparekh1 can you add the detailed steps in README, would be helpful for folks working on project.

done

[lucifercr07]

LGTM, please fix minor comments.

[lucifercr07]

Shall we rename it to .env.sample same as README?

[vinitparekh17]

done

[vinitparekh17]

@lucifercr07 can you perform HGETALL test cases manually into dicedb coz i think db has some issues with that cmd

[lucifercr07]

@vinitparekh17 had fixed it as part of my earlier commit, please check master build it's green.