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

The module at "/pad" does not exist. nginx, reverse proxy #515

Closed
mmehnert opened this issue Feb 27, 2012 · 18 comments
Closed

The module at "/pad" does not exist. nginx, reverse proxy #515

mmehnert opened this issue Feb 27, 2012 · 18 comments
Assignees

Comments

@mmehnert
Copy link

nginx is configured as described in the wiki.

stacktrace with commit
599c8c7

_designatedRequire()require-kernel.js (line 569)
path = "/pad"
continuation = undefined
designatedRequire()require-kernel.js (line 589)
path = "/pad"
continuation = undefined
requireRelative()require-kernel.js (line 595)
basePath = "/"
qualifiedPath = "/pad"
continuation = undefined
require()require-kernel.js (line 630)
qualifiedPath = "/pad"
continuation = undefined
(?)()RCtKLMMYOg (line 301)
RCtKLMMYOg()RCtKLMMYOg (line 298)

which comes from "require('/pad').init();" in the static pad.html

bisected:

dcc0740 is the first bad commit
commit dcc0740
Author: Chad Weider cweider@oofn.net
Date: Sun Jan 29 03:30:31 2012 -0800

Use Yajsml to combine files.

Minify only constructs individual files and optionally compresses them with UglifyJS.

:040000 040000 d97ce18d9fd67fadde681cf32b8ac29a5015acb0 88ad5af498b47d3f7b0fe5496ade364f6abf6e4d M node
:100644 100644 27b2125a1229e20422302b59732e3bba60f5fb7b e92905b7c1ab2d6cf0bab10e5ca46d8d15d029a9 M package.json
:040000 040000 3984e172e0ac28f57c52ebb6104eb4a1f6c12fc5 3dc99ec4c3bf07e1272e2e4480bafdba7714358d M static

@cweider
Copy link
Contributor

cweider commented Feb 28, 2012

Should be fixed by af4bd5c, could you check?

@cweider
Copy link
Contributor

cweider commented Feb 28, 2012

Also, thanks for the good error report.

@cweider cweider closed this as completed Feb 28, 2012
@mmehnert
Copy link
Author

No. not really. Still the same error.

@cweider cweider reopened this Feb 29, 2012
@cweider cweider mentioned this issue Feb 29, 2012
@mmehnert
Copy link
Author

Confirmed. This is fixed. Thanks for the great support and all the work! :-)

@cweider cweider closed this as completed Feb 29, 2012
@mmehnert
Copy link
Author

mmehnert commented Mar 3, 2012

Could you please hit me or something like that? I don't know what I did when I tested last time to get the idea that everything is fine. Pull Request #521 did not fix this. :-( I still get the same error with d868fd7 and 6fd73ec :-(
Everything works fine in b661ef5

@cweider cweider reopened this Mar 4, 2012
@ghost ghost assigned cweider Mar 4, 2012
@cweider cweider mentioned this issue Mar 4, 2012
@mmehnert
Copy link
Author

mmehnert commented Mar 8, 2012

With this merged into current master I still get the same error. :-(
Would it help if I send you a virtual machine with nginx configured for testing?

@cweider
Copy link
Contributor

cweider commented Mar 8, 2012

@mmehnert, possibly, but at least some HTTP data. Requests are made to nginx, requests made to the etherpad instance (from nginx), and their responses will help diagnose this issue. It must be something with redirects, but, without more data, I couldn’t say exactly what.

@mmehnert
Copy link
Author

mmehnert commented Mar 8, 2012

Do you prefer Vmware or Virtualbox?

@cweider
Copy link
Contributor

cweider commented Mar 8, 2012

@mmehnert, oh, just the config should be good enough. That said, it would also be helpful if you could create a minimal configuration the reproduces the error. In my tests, this works.

@mmehnert
Copy link
Author

mmehnert commented Mar 9, 2012

Found something. It works fine in standard configuration. The Problem appears as soon as I set httpAuth in settings.json. That somehow messes up everything.
Used a minimal configuration otherwise:

server {
        listen       443 ssl;
        server_name  pad.mydomain.org;
        access_log  /var/log/nginx/etherpad.access.log;
        error_log   /var/log/nginx/etherpad.error.log;

        ssl                  on; 
        ssl_session_timeout  5m; 

        location / { 
            proxy_pass             http://localhost:9001;
            proxy_buffering off;
        }   

}   
server {
    listen      80; 
    server_name pad.mydomain.org;
    rewrite     ^(.*)   https://$server_name$1 permanent;
}  

@cweider
Copy link
Contributor

cweider commented Mar 9, 2012

Ah, ok, then the fix should be pretty straight forward, I don’t see any reason for static assets to require authentication. Master is in limbo right now, but just move the block at server.js:127 beneath the static assets and that should fix it..

@mmehnert
Copy link
Author

mmehnert commented Mar 9, 2012

Hmm. I moved it to line 152 but somehow I could see content of pads without entering the password then. (restarted the browser but login was not requested.)

@jubro
Copy link

jubro commented Mar 9, 2012

the same error occurs to me and I had no http auth activated.

But I used the specific IP of the etherpad host in my network and not the loopback. So with 0.0.0.0 everything works fine to me.

@n3storm
Copy link

n3storm commented Mar 19, 2012

same here as jubro explains.

Just in case: Debian Wheezy

@marcelklehr
Copy link
Contributor

status?

@mmehnert
Copy link
Author

mmehnert commented May 9, 2012

still the same in

commit 309e3b09942389bd92205fc04d5e73088a30df60
Author: John McLear <john@mclear.co.uk>
Date:   Sat Mar 31 18:15:08 2012 +0100

    fix pitas fix

@mmehnert
Copy link
Author

mmehnert commented May 9, 2012

I mean, its possible to use nginx for authentication - then it's all fine. But the etherpad-lite authentication does not work through nginx.

@mmehnert
Copy link
Author

But it appears to work in the current development branch. 916e52a

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

No branches or pull requests

5 participants