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

500 in commit graph when commit description contain "DATA:" #4467

Closed
2 of 7 tasks
m0xf opened this issue Jul 18, 2018 · 1 comment · Fixed by #4487
Closed
2 of 7 tasks

500 in commit graph when commit description contain "DATA:" #4467

m0xf opened this issue Jul 18, 2018 · 1 comment · Fixed by #4487
Labels
Milestone

Comments

@m0xf
Copy link

m0xf commented Jul 18, 2018

2018/07/18 19:56:15 [...uters/repo/commit.go:91 Graph()] [E] GetCommitGraph: Failed parsing grap line:* DATA:| (HEAD -> master)|8fbe54a333ecc27e55e137c8194d14fcdfd131ef|2018-07-18 19:56:10 +0300|m0xf|m0xf@yaru|8fbe54a|test DATA:. Expect 1 or two fields

Description

If commit description contain "DATA:", gitea show 500 error on commit graph.

@kjellkvinge
Copy link
Contributor

I think I have a fix for this. I could also make som tests for these kinds of issues (parsing commit messages in graphItemFromString() in models/graph.go)

This is kind of related to #1189.

@lafriks lafriks added this to the 1.6.0 milestone Jul 21, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants