在Go语言中,如何在HTTP处理程序中正确使用pgx的上下文(context)?

3

更新1:似乎使用与HTTP请求相关联的上下文可能导致'上下文已取消'错误。然而,使用context.Background()作为父级似乎可以正常工作。

    // This works, no 'context canceled' errors
    ctx, cancel := context.WithTimeout(context.Background(), 100*time.Second)

    // However, this creates 'context canceled' errors under mild load
    // ctx, cancel := context.WithTimeout(r.Context(), 100*time.Second)

    defer cancel()
    app.Insert(ctx, record)

在Go中,我有一个HTTP handler,如下所示的代码。在第一次请求此端点时,我会收到一个 "context cancelled" 错误。然而,实际上数据已经插入到数据库中了。在此端点的后续请求中,不会出现这样的错误,并且数据也可以成功地插入到数据库中。

问题:我的设置和传递 context 到http handler 和 pgx QueryRow 方法是否正确?(如果不是,是否有更好的方法?)

如果将此代码复制到 main.go 中并运行 go run main.go,转到 localhost:4444/create 并按住 ctrl-R 产生轻微的负载,您应该会看到一些被取消的上下文错误。

package main

import (
    "context"
    "fmt"
    "log"
    "math/rand"
    "net/http"
    "time"

    "github.com/jackc/pgx/v4/pgxpool"
)

type application struct {
    DB *pgxpool.Pool
}

type Task struct {
    ID     string
    Name   string
    Status string
}

//HTTP GET /create
func (app *application) create(w http.ResponseWriter, r *http.Request) {
    fmt.Println(r.URL.Path, time.Now())
    task := &Task{Name: fmt.Sprintf("Task #%d", rand.Int()%1000), Status: "pending"}
    // -------- problem code here ----
    // This line works and does not generate any 'context canceled' errors
    //ctx, cancel := context.WithTimeout(context.Background(), 100*time.Second)
    // However, this linegenerates 'context canceled' errors under mild load
    ctx, cancel := context.WithTimeout(r.Context(), 100*time.Second)
    // -------- end -------
    defer cancel()
    err := app.insertTask(ctx, task)
    if err != nil {
        fmt.Println("insert error:", err)
        return
    }
    fmt.Fprintf(w, "%+v", task)
}
func (app *application) insertTask(ctx context.Context, t *Task) error {
    stmt := `INSERT INTO task (name, status) VALUES ($1, $2) RETURNING ID`
    row := app.DB.QueryRow(ctx, stmt, t.Name, t.Status)
    err := row.Scan(&t.ID)
    if err != nil {
        return err
    }
    return nil
}

func main() {
    rand.Seed(time.Now().UnixNano())
    db, err := pgxpool.Connect(context.Background(), "postgres://test:test123@localhost:5432/test")
    if err != nil {
        log.Fatal(err)
    }
    log.Println("db conn pool created")
    stmt := `CREATE TABLE IF NOT EXISTS public.task (
        id uuid NOT NULL DEFAULT gen_random_uuid(),
        name text NULL,
        status text NULL,
        PRIMARY KEY (id)
     ); `
    _, err = db.Exec(context.Background(), stmt)
    if err != nil {
        log.Fatal(err)
    }
    log.Println("task table created")
    defer db.Close()
    app := &application{
        DB: db,
    }
    mux := http.NewServeMux()
    mux.HandleFunc("/create", app.create)

    log.Println("http server up at localhost:4444")
    err = http.ListenAndServe(":4444", mux)
    if err != nil {
        log.Fatal(err)
    }
}



你使用了错误的上下文:app.Insert(r.Context()) -> app.Insert(ctx)。 - Dmitry Harnitski
似乎与使用 context.Background()r.Context() 作为父上下文有关。 - lifebythedrop
1个回答

2
TLDR:在生产中使用r.Context()没有问题,但在使用浏览器进行测试时会出现问题。
每个HTTP请求都有自己的上下文,当请求完成时,该上下文被取消。这是一种特性,而不是错误。开发人员应该使用它,并在请求被客户端或超时中断时优雅地关闭执行。例如,取消的请求可能意味着客户端永远看不到响应(交易结果),开发人员可以决定回滚该事务。
在生产中,对于通常设计/构建的API,请求取消并不经常发生。通常,流程由服务器控制,并且服务器在请求被取消之前返回结果。多个客户端请求不会相互影响,因为它们获得独立的Go协程和上下文。再次强调,我们正在谈论通常设计/构建应用程序的正常路径。您的示例应用程序看起来很好,应该能够正常工作。
问题在于我们如何测试应用程序。我们不是创建多个独立的请求,而是使用浏览器并刷新单个浏览器会话。我没有检查具体发生了什么,但假设当您单击ctrl-R时,浏览器终止了现有请求以运行新请求。服务器看到请求终止并将其作为上下文取消通信到您的代码中。
尝试使用curl或其他创建独立请求的脚本/实用程序测试您的代码。我相信在这种情况下,您不会看到取消。

感谢详细的解释。看起来这个方法很有效,即使在curl脚本的重负载下也不会出现问题。 - lifebythedrop
关于goroutine中上下文的用法如何呢?例如,我们在goroutine中仍有一些进程在工作时向客户端返回答案。在这种情况下,上下文是否会被取消? - Pavel

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接