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

cleaner url, support vhosts couchdb config #11

Closed
mandric opened this issue Jan 3, 2012 · 4 comments
Closed

cleaner url, support vhosts couchdb config #11

mandric opened this issue Jan 3, 2012 · 4 comments
Labels
Won't fix: Ancient Too old to investigate

Comments

@mandric
Copy link
Contributor

mandric commented Jan 3, 2012

We should be able to use/change the domain name that points to the rewrite location.

Also improves UX, per Marc

@abbyad
Copy link
Contributor

abbyad commented Dec 23, 2013

Moving this issue to MMVA

@abbyad abbyad closed this as completed Dec 23, 2013
@ghost
Copy link

ghost commented Dec 23, 2013

Hey Marc -- it's not an MMVA issue; we need to make changes to the actual couchapp to support this, the MMVA-side changes are already done and easy to tweak.

Specifically: using a proxy rule to map / -> db/_design/db/_rewrite hides the actual database, which breaks the app.

Reopen please?

Thanks.

On Dec 23, 2013, at 8:41 AM, Marc Abbyad notifications@github.com wrote:

Moving this issue to MMVA


Reply to this email directly or view it on GitHub.

@abbyad abbyad reopened this Dec 23, 2013
@abbyad
Copy link
Contributor

abbyad commented Dec 23, 2013

I was thinking of this more as just the landing site URL, a bit.ly equivalent for those that will type out the address manually - not all URLs. A redirect to the long URL be sufficient for that. The reason I moved it to MMVA was because I imagine this is a nginx setting to include by default in MMVA

@mandric
Copy link
Contributor Author

mandric commented Dec 28, 2013

We might need a fix in the proxy config that causes this issue during replication https://gist.github.com/mandric/4d2051487552ed1a3c94.

secure_rewrites:false is already in MMVA and allows the rewrite to access the db via ../../ and is currently used in market https://github.com/garden20/garden-market/blob/master/lib/rewrites.js#L52. Similarly we should add something like this to Kujua to allow access to the db and design doc then update any reference to a http requests in the app not accessing a resource via /_rewrite/.

@ghost ghost added the Won't fix: Ancient Too old to investigate label Nov 25, 2014
@ghost ghost closed this as completed Nov 25, 2014
garethbowen pushed a commit that referenced this issue Oct 24, 2017
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Won't fix: Ancient Too old to investigate
Projects
None yet
Development

No branches or pull requests

2 participants