FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

"Add Lightweight Tag" uses non-canonical sender address · Issue #173 · git-commit-notifier/git-commit-notifier · GitHub

Repository navigation

"Add Lightweight Tag" uses non-canonical sender address #173

Open

Description

After git tag foo && git push --tags the commit message produced by git-commit-notifier got stuck in mailman's moderation queue.

Closer inspection revealed that it was sent with a From: mg@fridge.pov.lt header instead of the expected From: Marius Gedminas <marius@pov.lt>.

If git-commit-notifier doesn't have enough metadata (this is a lightweight tag after all) and has to fall back to the Unix user account who ran the post-receive hook, it could at least use that user's ~/.gitconfig to get appropriate user.name and user.email.

Activity

  1. akzhan commented on Feb 25, 2013

    Member

    Interesting. Will review, but pull requests are also very apprepriated ;)

  2. mgedmin commented on Feb 26, 2013

    Author

    Pull requests would be easier if I knew Ruby :-)

    (I actually started a branch for adding support for nicer URLs that make use of gitweb's pathinfo feature, then stopped when I realized I've no idea how to run the test suite.)

  3. akzhan commented on Feb 26, 2013

    Member

    @mgedmin It's simple to test it in real life.

    Install required dependencies:

    cd $GCN_REPO # Here is your fork of git-commit-notifier repository
    bundle install

    Run for last commit in TEST_REPO.

    cd $TEST_REPO
    echo "HEAD^1 HEAD refs/heads/master" | $GCN_REPO/local-run.rb $PATH_TO_YML_CONFIG
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


      Back | FazBrowse Home | New Git URL