Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull request error #2011

Closed
renewsorensen opened this issue Nov 21, 2015 · 7 comments
Closed

Pull request error #2011

renewsorensen opened this issue Nov 21, 2015 · 7 comments
Labels
status: needs feedback Tell me more about it
Milestone

Comments

@renewsorensen
Copy link

Application Version: 0.7.6.1112 Beta

I have experienced a few issues with PR.
The first thing i see, is when i select develop branch on my forked repository, and press the PR button, it automatically selects base:master, which in my case is a branch that is 6 month behind the develop, so that takes a very long time to compare.

Next thing is if i want to change that, i change the base:master to base:develop

gogs_problem1

The url is weird
gogs_problem2

If i change the url manually it seems to do the right thing.
gogs_problem3

If i now do the pull request i just get the 500 error.
gogs_problem4

from the logs
2015/11/21 12:12:24 [...routers/repo/pull.go:642 CompareAndPullRequestPost()] [E] NewPullRequest: SavePatch: WriteFile: open g:\Git\scada\vob.server.git\pulls\1.patch: The system cannot find the path specified.
2015/11/21 12:12:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:12:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:12:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token

The last thing i see is that my log is filled with thees lines
2015/11/21 12:15:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:15:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:15:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:16:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:16:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:16:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:16:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:16:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:17:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:17:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:17:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:18:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:18:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:18:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:18:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:18:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:19:29 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:19:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token
2015/11/21 12:19:41 [...routers/repo/http.go:119 HTTP()] [E] invalid token

@stonerl
Copy link

stonerl commented Nov 21, 2015

Try to upgrade to the latest master there have been some PR related fixes.

@unknwon
Copy link
Member

unknwon commented Nov 21, 2015

Thanks your feedback!

  1. The error produced by manually change URL is not quite valid since many background work can not apply.
  2. As @stonerl mentioned, there are a lot changes about PR after 0.7.6. please first test on https://try.gogs.io and then upgrade.

@unknwon unknwon added the status: needs feedback Tell me more about it label Nov 21, 2015
@unknwon unknwon modified the milestones: 0.7.x, 0.8.0 Nov 21, 2015
@renewsorensen
Copy link
Author

Ok fair enough, i just tried different approaches to see if i could somehow get around that error.

i can confirm that i am not able to reproduce that particular error on https:/try.gogs.io.

I am not able to build (don't know how) go apps, so i wait for your next release.

@unknwon
Copy link
Member

unknwon commented Nov 22, 2015

0.7.19 is already released.

@renewsorensen
Copy link
Author

nice 👍 thanks!!

@unknwon
Copy link
Member

unknwon commented Nov 22, 2015

I'm going to close this now, please reopen if problem still exits. 😷

@unknwon unknwon closed this as completed Nov 22, 2015
@renewsorensen
Copy link
Author

It was maybe unclear to me what was fixed in the current release, but after the upgrade to 0.7.19.1121 i still get the 400 if i change repo on one of the sides in PR

gogs_problem9

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: needs feedback Tell me more about it
Projects
None yet
Development

No branches or pull requests

3 participants