Merge pull request #5 from j4ys0n/nginx-config

take domains as input obtainCerts

Author: j4ys0n

package.json

@@ -1,6 +1,6 @@
 {
   "name": "llm-proxy",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Manages Nginx for reverse proxy to multiple LLMs, with TLS & Bearer Auth tokens",
   "main": "dist/index.js",
   "scripts": {

src/controllers/nginx.ts

@@ -18,7 +18,7 @@ export class NginxController {
     this.app.get('/nginx/config/get', ...this.requestHandlers, this.getConfig.bind(this))
     this.app.get('/nginx/config/get-default', ...this.requestHandlers, this.getDefaultConfig.bind(this))
     this.app.get('/nginx/config/write-default', ...this.requestHandlers, this.writeDefaultConfig.bind(this))
-    this.app.get('/nginx/certificates/obtain', ...this.requestHandlers, this.obtainCertificates.bind(this))
+    this.app.post('/nginx/certificates/obtain', ...this.requestHandlers, this.obtainCertificates.bind(this))
     this.app.get('/nginx/certificates/renew', ...this.requestHandlers, this.renewCertificates.bind(this))
     log('info', 'NginxController initialized')
   }
@@ -64,9 +64,14 @@ export class NginxController {
   }
 
   private async obtainCertificates(req: Request, res: Response): Promise<void> {
-    const { success, message } = await this.nginxManager.obtainCertificates(true)
-    const status = success ? 200 : 500
-    res.status(status).json({ success, message })
+    if (req.body != null && req.body.domains != null && Array.isArray(req.body.domains)) {
+      const domains = req.body.domains
+      const { success, message } = await this.nginxManager.obtainCertificates(domains, true)
+      const status = success ? 200 : 500
+      res.status(status).json({ success, message })
+    } else {
+      res.status(400).json({ success: false, message: 'Invalid request body' })
+    }
   }
 
   private async renewCertificates(req: Request, res: Response): Promise<void> {

src/utils/nginx.ts

@@ -21,11 +21,11 @@ const CONFIG_TEMPLATE_PATH = join(__dirname, '../static/nginx-server-template.co
 
 export class NginxManager {
   private configPath: string
-  private domains: string[]
+  // private domains: string[]
 
   constructor(configPath: string = '/etc/nginx/conf.d/server.conf', domains: string[] = []) {
     this.configPath = configPath
-    this.domains = domains
+    // this.domains = domains
   }
 
   async start(): Promise<NginxResponse> {
@@ -105,8 +105,8 @@ export class NginxManager {
     return { success, config, message }
   }
 
-  async obtainCertificates(cloudflare?: boolean): Promise<NginxResponse> {
-    const domainArgs = this.domains.map((domain) => `-d ${domain}`).join(' ')
+  async obtainCertificates(domains: string[] = [], cloudflare?: boolean): Promise<NginxResponse> {
+    const domainArgs = domains.map((domain) => `-d ${domain}`).join(' ')
     let message: string | undefined
     let success = false
     const cloudflareFlags = cloudflare