back round 2 view raw

knotserver: set default git committer user via config #529

merged

opened by

boltless.me 3 months ago targeting master from boltless.me/core: push-skprtmnmwuqn

Introducing KNOT_GIT_USER_NAME and KNOT_GIT_USER_EMAIL environment variables.

This will prevent silently inheritting global gitconfig's user

Signed-off-by: Seongmin Lee boltlessengineer@proton.me

options

unified split

Changed files

+35 -19

knotserver

config

config.go

git

merge.go

routes.go

knotserver/config/config.go

···

       27
       27
        
       	Dev bool `env:"DEV, default=false"`

     

       28
       28
        
       }

     

       29
       29
        
       

     

       30
       30
       +
       type Git struct {

     

       31
       31
       +
       	// user name & email used as committer

     

       32
       32
       +
       	UserName     string `env:"USER_NAME, default=Tangled"`

     

       33
       33
       +
       	UserEmail    string `env:"USER_EMAIL, default=noreply@tangled.sh"`

     

       34
       34
       +
       }

     

       35
       35
       +
       

     

       30
       36
        
       func (s Server) Did() syntax.DID {

     

       31
       37
        
       	return syntax.DID(fmt.Sprintf("did:web:%s", s.Hostname))

     

       32
       38
        
       }

     
···

       34
       40
        
       type Config struct {

     

       35
       41
        
       	Repo            Repo   `env:",prefix=KNOT_REPO_"`

     

       36
       42
        
       	Server          Server `env:",prefix=KNOT_SERVER_"`

     

       43
       43
       +
       	Git             Git    `env:",prefix=KNOT_GIT_"`

     

       37
       44
        
       	AppViewEndpoint string `env:"APPVIEW_ENDPOINT, default=https://tangled.sh"`

     

       38
       45
        
       }

     

       39
       46

+22 -15

knotserver/git/merge.go

···

       85
       85
        
       

     

       86
       86
        
       // MergeOptions specifies the configuration for a merge operation

     

       87
       87
        
       type MergeOptions struct {

     

       88
       88
       -
       	CommitMessage string

     

       89
       89
       -
       	CommitBody    string

     

       90
       90
       -
       	AuthorName    string

     

       91
       91
       -
       	AuthorEmail   string

     

       92
       92
       -
       	FormatPatch   bool

     

       88
       88
       +
       	CommitMessage  string

     

       89
       89
       +
       	CommitBody     string

     

       90
       90
       +
       	AuthorName     string

     

       91
       91
       +
       	AuthorEmail    string

     

       92
       92
       +
       	CommitterName  string

     

       93
       93
       +
       	CommitterEmail string

     

       94
       94
       +
       	FormatPatch    bool

     

       93
       95
        
       }

     

       94
       96
        
       

     

       95
       97
        
       func (e ErrMerge) Error() string {

     
···

       164
       166
        
       	var stderr bytes.Buffer

     

       165
       167
        
       	var cmd *exec.Cmd

     

       166
       168
        
       

     

       167
       167
       -
       	exec.Command("git", "-C", tmpDir, "config", "advice.mergeConflict", "false").Run()

     

       169
       169
       +
       	// configure default git user before merge

     

       170
       170
       +
       	exec.Command(

     

       171
       171
       +
       		"git",

     

       172
       172
       +
       		"-C",

     

       173
       173
       +
       		tmpDir,

     

       174
       174
       +
       		"config",

     

       175
       175
       +
       		"user.name",

     

       176
       176
       +
       		opts.CommitterName,

     

       177
       177
       +
       		"user.email",

     

       178
       178
       +
       		opts.CommitterEmail,

     

       179
       179
       +
       		"advice.mergeConflict",

     

       180
       180
       +
       		"false",

     

       181
       181
       +
       	).Run()

     

       168
       182
        
       

     

       169
       183
        
       	// if patch is a format-patch, apply using 'git am'

     

       170
       184
        
       	if opts.FormatPatch {

     
···

       188
       202
        
       		authorName := opts.AuthorName

     

       189
       203
        
       		authorEmail := opts.AuthorEmail

     

       190
       204
        
       

     

       191
       191
       -
       		if authorEmail == "" {

     

       192
       192
       -
       			authorEmail = "noreply@tangled.sh"

     

       193
       193
       -
       		}

     

       194
       194
       -
       

     

       195
       195
       -
       		if authorName == "" {

     

       196
       196
       -
       			authorName = "Tangled"

     

       197
       197
       -
       		}

     

       198
       198
       -
       

     

       199
       199
       -
       		if authorName != "" {

     

       205
       205
       +
       		if authorName != "" && authorEmail != "" {

     

       200
       206
        
       			commitArgs = append(commitArgs, "--author", fmt.Sprintf("%s <%s>", authorName, authorEmail))

     

       201
       207
        
       		}

     

       208
       208
       +
       		// else, will default to knot's global user.name & user.email configured via `KNOT_GIT_USER_*` env variables

     

       202
       209
        
       

     

       203
       210
        
       		commitArgs = append(commitArgs, "-m", opts.CommitMessage)

     

       204
       211

+6 -4

knotserver/routes.go

···

       986
       986
        
       	}

     

       987
       987
        
       

     

       988
       988
        
       	mo := git.MergeOptions{

     

       989
       989
       -
       		AuthorName:    data.AuthorName,

     

       990
       990
       -
       		AuthorEmail:   data.AuthorEmail,

     

       991
       991
       -
       		CommitBody:    data.CommitBody,

     

       992
       992
       -
       		CommitMessage: data.CommitMessage,

     

       989
       989
       +
       		AuthorName:     data.AuthorName,

     

       990
       990
       +
       		AuthorEmail:    data.AuthorEmail,

     

       991
       991
       +
       		CommitterName:  h.c.Git.UserName,

     

       992
       992
       +
       		CommitterEmail: h.c.Git.UserEmail,

     

       993
       993
       +
       		CommitBody:     data.CommitBody,

     

       994
       994
       +
       		CommitMessage:  data.CommitMessage,

     

       993
       995
        
       		FormatPatch:   patchutil.IsFormatPatch(patch),

     

       994
       996
        
       	}

     

       995
       997