#6982 SCM views should parse user/email pairs better

v1.1.0
closed
nobody
42cc (432)
General
Cory Johns
2015-08-20
2013-12-12
Dave Brondsema
No

ForgeHg uses re_hg_user to split the name & email portion of the "user" field on a commit. But this is simplistic and doesn't work if just an email address is set, for example. We should use a real email header parsing method.

Also happening on git views, for example: https://forge-allura.apache.org/p/allura/git/ci/a113345dc4e79ca40c29726475f241c8b63379d3/log/?path=

Related

Tickets: #6995
Tickets: #7019

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-12-16
    • summary: ForgeHg should parse user/email pairs better --> SCM views should parse user/email pairs better
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,3 @@
 ForgeHg uses `re_hg_user` to split the name & email portion of the "user" field on a commit.  But this is simplistic and doesn't work if just an email address is set, for example.  We should use a real email header parsing method.
+
+Also happening on git views, for example: https://forge-allura.apache.org/p/allura/git/ci/a113345dc4e79ca40c29726475f241c8b63379d3/log/?path=
     

  • Dave Brondsema

    Dave Brondsema - 2014-01-06
    • Labels: --> 42cc
    • Status: open --> in-progress
     

  • Igor Bondarenko - 2014-01-23
    • status: in-progress --> code-review
     

  • Igor Bondarenko - 2014-01-23

    Closed #520. je/42cc_6982

    Also fixes [#6995] and [#7019]

     

    Related

    Tickets: #6995
    Tickets: #7019

  • Cory Johns - 2014-01-23
    • QA: Cory Johns
     

  • Cory Johns - 2014-01-23
    • status: code-review --> validation
    • Milestone: forge-backlog --> forge-jan-24
     

  • Dave Brondsema

    Dave Brondsema - 2014-01-24
    • status: validation --> closed
     

Log in to post a comment.