App\Support\Nginx\SiteConfBuilder

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

81.82%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	9	1
A	destroy()	0	3	1

<?php

namespace App\Support\Nginx;

use App\Models\Site;

class SiteConfBuilder
{
    /**
     * Build the nginx.conf file for a given site
     *
     * @param \App\Models\Site $site
     * @throws \Throwable
     */
    public function build(Site $site)
    {
        file_put_contents(
            $site->nginx_conf_path,
            view($site->nginx_conf_template)->with([
                'site' => $site->url,
                'name' => $site->name,
                'version' => $site->php_version->safe
            ])->render()
        );
    }

    /**
     * Destroy the nginx.conf conf for a given site
     *
     * @param \App\Models\Site $site
     */
    public function destroy(Site $site)
    {
        @unlink($site->nginx_conf_path);

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

    }
}