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

feat: add MongoDB support #248

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

ATheCoder
Copy link

@ATheCoder ATheCoder commented Sep 21, 2020

This will add MongoDB support.

You can test this by changing the following files:

dejavu.cnf.SAMPLE

{
    "database": {
        "database": "dejavu"
    },
    "database_type": "mongodb"
}

docker-compose.yaml

version: '3'
services:
  db:
    build:
      context: ./docker/postgres
    environment:
      - POSTGRES_DB=dejavu
      - POSTGRES_USER=postgres
      - POSTGRES_PASSWORD=password
    networks:
      - db_network
  mongodb:
    image: mongo
    ports:
        - "27020:27017"
    networks:
      - db_network
  python:
    build:
      context: ./docker/python
    volumes:
      - .:/code
    depends_on:
      - db
    networks:
      - db_network
networks:
  db_network:

Then you can run docker-compose up -d

and then run docker-compose run python /bin/bash

You'll also need to make something like example_docker_postgres.py to start the fingerprinting process and populate the database.

Then you can use python dejavu.py -r file ... to recognize a file.

@ATheCoder
Copy link
Author

@worldveil please check this out.

@steveouma
Copy link

Hi @ATheCoder, what does the performance look like on MongoDB compared to either Postgre or Mysql?

@heechuls
Copy link

heechuls commented Aug 3, 2021

Hi @ATheCoder, what does the performance look like on MongoDB compared to either Postgre or Mysql?

+1, Has anyone tested the performance?

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.

3 participants