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

Patch 1 #543

Closed
wants to merge 2 commits into from
Closed

Patch 1 #543

wants to merge 2 commits into from

Conversation

oiLvAcciNe
Copy link
Contributor

No description provided.

Will xxxx get updates via cron, ssh, or http(s) [c|s|h]? h

  --> Detecting the web server...apache2
  --> Detecting the web server configuration file.../etc/apache2/apache2.conf
  --> Detecting the web server document root...
  
  
	@at function getParameterValue($param, $file, $delimiter = '=')
  
		grep '^' '/etc/apache2/apache2.conf' 2>/dev/null | grep '"'

			#ServerRoot "/etc/apache2"
			# one request per connection). Set to "Off" to deactivate.
			# "LogLevel info ssl:warn"

	  
		since docroot_parameter is null we are getting the first value from the output above (this case ServerRoot "/etc/apache2")
			

	Web server is detected correct (apache2) but there is no case for web server in case switch:
	Tested with adding apache2 to case and it worked ...  

	@at funciton.php ~line 1678
	
       switch ($web_server['app']) {
                case 'httpd':
                case 'apache2':
                        $docroot_parameter = 'DocumentRoot';
                        break;
                case 'lighttpd':
                        $docroot_parameter = 'server.document-root';
                        break;
                case 'nginx':
                        $docroot_parameter = 'document root';
                        break;
        }


  

  --> Creating /etc/apache2/fM link.
@oiLvAcciNe oiLvAcciNe marked this pull request as ready for review November 26, 2021 11:11
WillyXJ added a commit that referenced this pull request Dec 7, 2021
@WillyXJ
Copy link
Owner

WillyXJ commented Dec 7, 2021

Thanks for the fix!

@WillyXJ WillyXJ closed this Dec 7, 2021
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.

None yet

2 participants