search

cem / gogs-sandstorm

Gogs for the Sandstorm platform
11 stars 8 forks source link

Merging same-repo pull request returns Gogs 500 error page #9

Closed DevonStrawn closed 8 years ago

DevonStrawn commented 8 years ago

First, thanks for writing gogs-sandstorm - it's the whole reason I looked into Sandstorm in the first place. It's wonderful!

Not sure if this is a Gogs bug or Gogs-sandstorm, but Google didn't turn up anything for the error below, and it seems like the sort of thing that might trip up Sandstorm / Gogs.

Repro steps:

  1. setup a Git repo with at least one branch in Gogs
  2. create a pull request from one of the branches into master
  3. in the pull request page, click "Merge Pull Request" button

Bug: Gogs 500 error pages appear:

image

Here's the relevant bit of debug log (something about "unable to look up current user in the passwd file: No such file or directory"):

image

(using the latest v0.9.13 Gogs Sandstorm app)

Note that this depends on Gogs' new feature of same-repository Pull Requests.

cem commented 8 years ago

First, thanks for writing gogs-sandstorm - it's the whole reason I looked into Sandstorm in the first place. It's wonderful!

I'm very happy to hear that!

I pushed a fix for this issue (v0.9.13.fix1). Please open another issue if this isn't resolved.