Skip to content

Commit

Permalink
Merge pull request #93 from thedadams/remove-disable-server
Browse files Browse the repository at this point in the history
fix: remove the disable server environment variable
  • Loading branch information
thedadams authored Sep 26, 2024
2 parents a2753a6 + f8b4e72 commit 523aa08
Showing 1 changed file with 16 additions and 3 deletions.
19 changes: 16 additions & 3 deletions src/gptscript.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,9 +83,13 @@ export class GPTScript {
this.ready = false
GPTScript.instanceCount++
if (!GPTScript.serverURL) {
GPTScript.serverURL = "http://" + (process.env.GPTSCRIPT_URL || "127.0.0.1:0")
GPTScript.serverURL = process.env.GPTSCRIPT_URL ?? "http://127.0.0.1:0"
if (!GPTScript.serverURL.startsWith("http://") && !GPTScript.serverURL.startsWith("https://")) {
GPTScript.serverURL = "http://" + GPTScript.serverURL
}
}
if (GPTScript.instanceCount === 1 && process.env.GPTSCRIPT_DISABLE_SERVER !== "true") {

if (GPTScript.instanceCount === 1 && !process.env.GPTSCRIPT_URL) {
let env = process.env
if (this.opts.Env) {
env = {
Expand Down Expand Up @@ -121,16 +125,25 @@ export class GPTScript {
}

GPTScript.serverURL = `http://${url}`
if (!this.opts.Env) {
this.opts.Env = []
}
this.opts.Env.push(`GPTSCRIPT_URL=${GPTScript.serverURL}`)

GPTScript.serverProcess.stderr?.removeAllListeners()
})
} else {
if (!this.opts.Env) {
this.opts.Env = []
}
this.opts.Env.push("GPTSCRIPT_URL=" + GPTScript.serverURL)
}
}

close(): void {
GPTScript.instanceCount--
if (GPTScript.instanceCount === 0 && GPTScript.serverProcess) {
GPTScript.serverURL = "http://" + (process.env.GPTSCRIPT_URL || "127.0.0.1:0")
GPTScript.serverURL = process.env.GPTSCRIPT_URL ?? "http://127.0.0.1:0"
GPTScript.serverProcess.kill("SIGTERM")
GPTScript.serverProcess.stdin?.end()
}
Expand Down

0 comments on commit 523aa08

Please sign in to comment.