8000 Case insensitivity for command names · Issue #15 · mongolab/dex · GitHub
[go: up one dir, main page]

Skip to content

Case insensitivity for command names #15

Open

Description

@md5

I tried running dex on a Mongo shard log I've got and noticed that it missed a bunch of findAndModify lines. It turns out that the log has "findandmodify" instead of "findAndModify" for the command name. I hacked my copy of parsers.py locally to account for that one special case, but I'm not sure if something more comprehensive should be done. I can open a PR with my changes if it helps, but I'm not sure they're exactly what they should be (in particular, my installed version doesn't have SUPPORTED_COMMANDS in analyzer.py).

Activity

  1. esedor commented on Oct 4, 2013

    @esedor
    Contributor

    A great find. I'll take a look. Preliminary, I don't see a problem with supporting case-insensitive command names, so please do submit your tweak as a pull request, and I will incorporate it differently than as-written if it becomes necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      0