Opened 10 years ago

Closed 7 years ago

#830 closed defect (fixed)

Blog post and exif

Reported by: Jeremy Pope Owned by:
Priority: major Milestone:
Component: programming Keywords: blogpost, exif
Cc: Parent Tickets: 5438

Description

Looks like ./mediagoblin/templates/mediagoblin/user_pages/blog_media.html contains a reference to mediagoblin/utils/exif.html which causes blog posts to crash the server when being loaded if exif_visible = true is set in the config.

Removing the reference fixes the issue.

Anyone have insight as to why it may need to stay?

Change History (3)

comment:1 by ayleph, 8 years ago

Parent Tickets: 5438

comment:2 by ayleph, 8 years ago

I've got exif_visible = true in my config, and blog posts load without crashing. It seems like this was addressed at some point in the past. However, I don't see any reason at all for blog_media.html to pull in the exif page. I assume that blog_media.html was created as a copy of media.html and the exif code should not have been copied over. I'll create a patch which removes it.

comment:3 by ayleph, 7 years ago

Resolution: fixed
Status: newclosed

The exif reference has been removed from blog_media.html as of 6f5fcc5.

Note: See TracTickets for help on using tickets.