- **status**: review --> closed
- **Comment**:

Looks good. Merged

I have removed slash from the forum icon path also:

~~~~
ForgeDiscussion/forgediscussion/templates/discussionforums/admin_forums.html
43:                <img src="{{forum.url()}}/icon" alt="Forum icon"/>
~~~~

I can't check how it works due to [#8008], but `Forum.url` adds `/` to the end 
of the url it returns (see 
`ForgeDiscussion/forgediscussion/model/forum.py:103`), so I'm pretty sure the 
change is right.



---

** [tickets:#8007] Broken icon images when running under gunicorn**

**Status:** closed
**Milestone:** unreleased
**Labels:** sf-1 sf-current 
**Created:** Tue Oct 20, 2015 08:52 PM UTC by Dave Brondsema
**Last Updated:** Wed Oct 21, 2015 11:12 AM UTC
**Owner:** Dave Brondsema


paster and mod_wsgi are forgiving with URLs like `/p/test//icon` but gunicorn 
doesn't help fix that double slash.


---

Sent from forge-allura.apache.org because [email protected] is subscribed 
to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.

Reply via email to