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

Issues - tested on Debian 8 #36

Closed
vasiler opened this issue Feb 10, 2019 · 8 comments
Closed

Issues - tested on Debian 8 #36

vasiler opened this issue Feb 10, 2019 · 8 comments
Labels

Comments

@vasiler
Copy link

vasiler commented Feb 10, 2019

Hi,

At the interactive installation, I remain stuck when I have to respond if I want an automatic update.

"Do you want to setup nginx-ee auto-update cronjob ? (y/n)
Select an option [y/n]: y
Select an option [y/n]: y"

When installing with parameters, I get errors.

For "bash <(wget -O - virtubox.net/nginx-ee || curl -sL virtubox.net/nginx-ee) --dynamic --naxsi --cron" I have that in /tmp/nginx-ee.log:
Hunk #6 succeeded at 469 (offset 2 lines).
patching file src/http/v2/ngx_http_v2_encode.c
patching file src/http/v2/ngx_http_v2_filter_module.c patching file src/http/v2/ngx_http_v2_table.c patching file src/event/ngx_event_openssl.c Hunk #1 succeeded at 355 with fuzz 1 (offset 25 lines).
make: *** No targets specified and no makefile found. Stop.
make: *** No rule to make target 'install'. Stop.

@VirtuBox VirtuBox added the bug label Feb 10, 2019
@VirtuBox
Copy link
Owner

Hello @vasiler,

have you tried to run the script since nginx-ee v3.5.1 was released ?
Because error in your logs seems to be related to the previous method used to download openssl.

Please, can you try to run the script again and confirm if the issue is fixed ?

@vasiler
Copy link
Author

vasiler commented Feb 16, 2019

The previous version I tested successfully is 3.3.1

I just tested the script and it does not work.
Command: bash <(wget -O - virtubox.net/nginx-ee || curl -sL virtubox.net/nginx-ee) --naxsi

Please see log: https://paie.ro/nginx-ee.zip

@VirtuBox
Copy link
Owner

Hello @vasiler,

Issue seems to be related to openssl, not to debian 8.
Can you try to compile nginx with nginx-ee develop branch ?

Here the command to run :

bash <(wget -O - virtubox.net/nginx-ee-dev || curl -sL virtubox.net/nginx-ee-dev) --naxsi

@vasiler
Copy link
Author

vasiler commented Feb 17, 2019

I tried, error again.

Error log https://paie.ro/nginx-ee2.zip

@VirtuBox
Copy link
Owner

Thanks, I will run few tests on a VM to see if it's related to the gcc version available on debian 8 or something else.

@VirtuBox
Copy link
Owner

Issue has been fixed with the last commit on develop branch.

Welcome to the nginx-ee bash script v3.5.2


##################################
 Compilation summary
##################################

 Detected OS : Debian GNU/Linux 8.11 (jessie)
 Detected Arch : x86_64

  - Nginx release : 1.15.8
  - Dynamic modules NO
  - Pagespeed : NO
  - Naxsi : YES
  - RTMP : NO

       Installing dependencies                [OK]
       Downloading additionals modules        [OK]
       Downloading zlib                       [OK]
       Downloading brotli                     [OK]
       Downloading openssl                    [OK]
       Downloading naxsi                      [OK]
       Downloading nginx                      [OK]
       Applying nginx patches                 [OK]
       Configuring nginx                      [OK]
       Compiling nginx                        [OK]
       Updating Nginx manual                  [OK]
       Performing final steps                 [OK]
       Checking nginx configuration           [OK]
       Nginx-ee was compiled successfully !

       Installation log : /tmp/nginx-ee.log

I will publish changes on master tomorrow.

@vasiler
Copy link
Author

vasiler commented Feb 18, 2019

Now is fine, but...

Tested on Chrome Mobile (Android) and does not work with ssl_ciphers from https://github.com/VirtuBox/nginx-ee/blob/master/etc/nginx/nginx.conf (ERR_SSL_VERSION_OR_CIPHER_MISMATCH)

@VirtuBox
Copy link
Owner

Hello @vasiler,
yes this ciphers suite is outdated. Proper TLS configuration is available in the Wiki

@vasiler vasiler closed this as completed Feb 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants