Skip to content

Comments

Fixups to allow running out of subdirectory#2518

Closed
r0ps3c wants to merge 11 commits intoRocketChat:developfrom
r0ps3c:develop
Closed

Fixups to allow running out of subdirectory#2518
r0ps3c wants to merge 11 commits intoRocketChat:developfrom
r0ps3c:develop

Conversation

@r0ps3c
Copy link

@r0ps3c r0ps3c commented Mar 16, 2016

Currently, there are a number of issues with running rocket.chat out of a subdirectory of the webroot (e.g. /chat). These seem to be primarily around resources/assets that are referenced, such as fonts, images and so on. I have also opened a number of issues on modules being used (e.g. sandstorm-io/meteor-accounts-sandstorm#17, Meteor-Community-Packages/meteor-timesync#36) and will issue another PR once those are wrapped up. In the meantime this PR should fix the assumption in rocket.chat itself, and help to close a number of issues (e.g. #1696, #335, #2401)

@sampaiodiego
Copy link
Member

what happens when your first hit is in a URL like this https://demo.rocket.chat/channel/general ?
are the assets loaded correctly?

@r0ps3c
Copy link
Author

r0ps3c commented Mar 18, 2016

Hi,

all assets (fonts, sounds, avatar images, etc.) do not load correctly (i.e. 404s) without those updates for me. The issue is fairly easy to replicate - setting ROOT_URL to http://localhost:3000/chat should do it.

Thanks

@sampaiodiego
Copy link
Member

I'm sorry.. I mean, with your fix, what happens when your first hit is in a URL like this https://demo.rocket.chat/channel/general ?

@r0ps3c
Copy link
Author

r0ps3c commented Mar 20, 2016

Hi,

on the instance I am testing, where ROOT_URL is set to /chat/, going to /chat/channel/general for the first time returns the login screen. Are you seeing a regression for this?

@engelgabriel engelgabriel modified the milestone: 0.24.0 Mar 21, 2016
@engelgabriel
Copy link
Member

Hi @r0p0s3c we will merge this into develop tomorrow to give enough time for testers, can you just please resolve the conflicts?

type: 'warning'
title: t('Warning')
text: t("The_setting_s_is_configured_to_s_and_you_are_accessing_from_s", t('Site_Url'), siteUrl, location.origin) + '<br/><br/>' + t("Do_you_want_to_change_to_s_question", location.origin)
text: t("The_setting_s_is_configured_to_s_and_you_are_accessing_from_s", t('Site_Url'), siteUrl, Meteor.absoluteUrl()) + '<br/><br/>' + t("Do_you_want_to_change_to_s_question", Meteor.absoluteUrl())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

siteUrl and Meteor.absoluteUrl() will always be the same.
This verification exists to warn users when they access from an URL different from the siteUrl

@rodrigok
Copy link
Member

Closed in favor of #2624

@rodrigok rodrigok closed this Mar 23, 2016
@r0ps3c
Copy link
Author

r0ps3c commented Mar 23, 2016

Thanks, was not able to get around to responding to the comments/questions in time due to other commitments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants