Not creating another domain.

Question

Not creating another domain.

jackkelly-bayliss opened this issue 7 years ago · 8 comments

Hello,

I am currently having the issue where it will not detect a new domain name. It's only reading the very first line of proxydomains. Any suggestions to resolve this?

if nshield_proxy is 1 and dryrun is 0:
	print "Setting up nShield proxy for domains found in /etc/nshield/proxydomains\n"
        # Generates nginx proxy_pass from /etc/nshield/proxydomains and checks if already present in nginx conf
        with open("/etc/nshield/proxydomains") as f:
                content = f.readlines()
                content1 = content[0].split(' ')
                ip = content1[1].strip('\n')
                domain = content1[0]
                if domain not in os.popen("cat /etc/nginx/sites-enabled/dynamic-vhost.conf").read():
                    print "I Will generate proxy configuration for site "+domain+" on IP: "+ip
                    os.popen("""echo 'server {
        listen 80;

        root /var/www/html;
        index index.html index.htm index.nginx-debian.html;

        server_name """+domain+""";

        location / {
    proxy_pass       http://"""+ip+""";
    proxy_set_header Host      $host;
    proxy_set_header X-Real-IP $remote_addr;
}
}
' >> /etc/nginx/sites-enabled/dynamic-vhost.conf""")
                    os.popen('service nginx restart')
                else:
                    print "Domain already configured"

        print "Now you can test that your site is reachable via nShield proxy by changing the domain DNS or via your PC hosts file or directly DNS A record"

Kind Regards,
Jack K-B

Answer 1 · 2017-10-18T18:58:43.000Z

Hello Jack,
I fixed the code and now the proxydomain configuration is working with multiple domains

commit link

Sami

Answer 2 · 2017-10-18T20:43:49.000Z

Hello Sami,

That's perfect, thank you very much for resolving this issue for me!

Kind Regards,
Jack K-B

Answer 3 · 2017-10-18T20:53:07.000Z

Hello,

I'm sorry for re-opening this issue but upon executing the 'python nshield-main.py' command, I'm receiving this error http://prntscr.com/gz44pn - The file does exist and I've checked it's in the correct path.

Kind Regards,
Jack K-B

Answer 4 · 2017-10-24T17:31:27.000Z

Hello,
The error is most likely from the variable not a str, now i have added the str() conversion

Answer 5 · 2017-10-26T15:00:03.000Z

Hello,

That's perfect thank you. I've just executed git pull and it's now showing the error http://prntscr.com/h2bv4k

Kind Regards,
Jack K-B

Answer 6 · 2017-10-29T15:54:43.000Z

Hello,
Could you remove the /home/nShield/ folder and run again:
cd /home/ && git clone https://github.com/fnzv/nShield.git && bash nShield/install.sh
?
From my ubuntu 16 image works without issues

Answer 7 · 2017-10-29T21:27:07.000Z

Hello,

Thank you for replying. I've done as you said and it ran without issues upon default settings. As soon as I turned the variable 'nshield_proxy' to '1'. It's showing the same error as the above.

The machine that I'm receiving this error on is 'Ubuntu 16.04.3 LTS'.

Kind Regards,
Jack K-B.

Answer 8 · 2017-11-03T20:30:37.000Z

Hello,
Sorry for the late reply, here is the fix: commit 5af9a02eb63a00f18ec25cc98de4b8f5ff0ef611

Be sure there are no other blank lines on the /etc/nshield/proxydomains file.. example:

root@ubuntu16:/home# cat /etc/nshield/proxydomains
test1.com 123.123.123.123
test2.com 123.123.123.123
test3.com 123.123.123.123
test4.com 123.123.123.123
root@ubuntu16:/home#