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

Deleting an org avatar breaks the image #2073

Closed
2 of 7 tasks
tboerger opened this issue Jun 28, 2017 · 8 comments
Closed
2 of 7 tasks

Deleting an org avatar breaks the image #2073

tboerger opened this issue Jun 28, 2017 · 8 comments
Labels
Milestone

Comments

@tboerger
Copy link
Member

  • Gitea version (or commit ref): 1.1.2
  • Git version: n/a
  • Operating system: docker
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
  • Log gist: n/a

Description

I have uploaded a custom avatar for my organization, afterwards I have deleted it again and than the org avatar is broken. No page is loading any avatar anymore. I would expect that I get an auto-generated Gravatar image again.

Screenshots

bildschirmfoto 2017-06-28 um 10 46 35

@tboerger tboerger added this to the 1.1.3 milestone Jun 28, 2017
@lunny
Copy link
Member

lunny commented Jun 28, 2017

This bug maybe reproduced on master?

@ethantkoenig
Copy link
Member

ethantkoenig commented Jun 29, 2017

Also, after the avatar breaks, the GET /orgs/:orgname API endpoint for the organization gives a 500.

I'm guessing this is because org.RelAvatarLink() is returning an empty string, so there is an out-of-index error at https://github.com/go-gitea/gitea/blob/master/models/user.go#L361

@tboerger
Copy link
Member Author

@lunny try.gitea.io is based on master.

@lunny
Copy link
Member

lunny commented Jun 29, 2017

So let's mark it as v1.2 and back port to v1.1.3

@ethantkoenig
Copy link
Member

@tboerger @lunny Fixed? (see https://try.gitea.io/foobar)

@lunny
Copy link
Member

lunny commented Jul 3, 2017

@ethantkoenig @tboerger seems yes.

@lunny
Copy link
Member

lunny commented Jul 11, 2017

@ethantkoenig please backport to v1.1

@lunny lunny modified the milestones: 1.2.0, 1.1.3 Jul 11, 2017
@lunny
Copy link
Member

lunny commented Jul 11, 2017

closed by #2083

@lunny lunny closed this as completed Jul 11, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants