When Discourse is hosted from a subfolder (e.g.
/forum) it seems that the cookie is still set to the root folder
To me this sounds like a potential security issue. For example, if a forum is hosted in
/forum and a WordPress blog is hosted in
/blog, the cookie with Discourse session is sent also to the WordPress site.
This should be pretty easy to fix by changing
Discourse::Application.config.session_store :cookie_store, key: '_forum_session'
path: (Rails.application.config.relative_url_root.nil?) ? '/' : Rails.application.config.relative_url_root
/ work also for subdirectory
/forum. So the migration path should be ok.
Does this make sense? I could make a PR if you think this makes sense.
One addition. In this guide
Discourse in a subfolder, multiple servers sharing a domain it says that
Note: This won’t work for serving multiple Discourse instances from different folders on the same domain. You need to use different subdomains so that each site can have different cookies.
So with this change you could remove that note. There’s no need for different subdomain if the cookies are scoped by the path. Or are there any other reasons why one couldn’t host two Discourse instances from different folders?
It would be nice if Discourse set the correct cookie path, but it’s definitely not a security issue.
@neil we should look at doing this if it is not too risky. Feels risky to me for live sites though…
Yes please submit a PR.
I’ll test it out myself to see how it impacts live sites after a deploy.
I tested the change and it’s safe. Sessions aren’t invalidated. I committed the change today.
Serving multiple Discourse instances from different paths on the same domain will most likely work now, but note that the instances can’t be served from one
multisite setup. Multisite is still keyed off of domain, not path.
I have 2 discourse setups.
setup 1 running from
setup 2 running from
www.site.com/d2/ Problems noticed
I cannot remain logged into both the discourse setups at the same time
If I am logged into d1 and I refresh d2 I am logged out of d1.
More information about the setup:
Both are running in their own docker containers using their own DB
installed using discourse_docker
version is 1.9 beta2
Yeah we have a bug here:
We need to explicitly add the path there, just like session has the path.
Unfortunately fixing this will log everyone off on all subfolder installs, but still, it is probably worth fixing.
@neil maybe look at adding path there?
@neil let me know once you commit the fix … I can check it out if it works …
@vikaskedia I pushed a fix so please try it out.
Ok I am following the steps here to upgrade:
If you self-host Discourse, you occasionally need to run a manual update via the command line to get the latest security releases newest libraries. These updates are not picked up in admin/upgrade, which is why you’ll occasionally need to do this additional step.
Discourse itself should be updated about twice a month, by clicking the “Update to Latest Version” button in your admin dashboard ( admin/upgrade). We do beta releases roughly once every week.
Every two months we recommend SSH’ing…
and will let you know soon
@neil damm man … it works !!
I am successfully logged into
Very good …
@neil I just started getting “internal server error 500” can this be related to the above fix …
The error auto corrects itself in a couple of minutes. So it seems to indicate it is the “rate limiting” issue.
But this is just me using the site and its a beefy server …
and i am seeing it for the first time after i upgraded.
Looks like your subfolder is missing? “/subfolder/faq”
If the subfolder was missing the error will come all the time.
This error is a transient error …
It sounds like another problem we saw today.
@sam seems like this code is being hit in this case too? The cookie is invalid now?
I tested on local and it tries to reset the cookie, I guess it is failing cause it can no longer reset the cookie from the top level path
Let me know when you release a fix … I will check it out …
Try deleting cookies, does the problem go away?