| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546254725482549255025512552255325542555255625572558255925602561256225632564256525662567256825692570257125722573257425752576257725782579258025812582258325842585258625872588258925902591259225932594259525962597259825992600260126022603260426052606260726082609261026112612261326142615261626172618261926202621262226232624262526262627262826292630263126322633263426352636263726382639264026412642264326442645264626472648264926502651265226532654265526562657265826592660266126622663266426652666266726682669267026712672267326742675267626772678267926802681268226832684268526862687268826892690269126922693269426952696269726982699270027012702270327042705270627072708270927102711271227132714271527162717271827192720272127222723272427252726272727282729273027312732273327342735273627372738273927402741274227432744274527462747274827492750275127522753275427552756275727582759276027612762276327642765276627672768276927702771 |
- package service
- import (
- "bytes"
- "context"
- "errors"
- "fmt"
- "io"
- "net/http"
- "net/http/httptest"
- "strings"
- "sync"
- "testing"
- "time"
- "github.com/Wei-Shaw/sub2api/internal/config"
- "github.com/Wei-Shaw/sub2api/internal/pkg/apicompat"
- "github.com/Wei-Shaw/sub2api/internal/pkg/ctxkey"
- "github.com/Wei-Shaw/sub2api/internal/pkg/logger"
- "github.com/Wei-Shaw/sub2api/internal/pkg/openai"
- "github.com/Wei-Shaw/sub2api/internal/pkg/openai_compat"
- "github.com/Wei-Shaw/sub2api/internal/pkg/tlsfingerprint"
- "github.com/gin-gonic/gin"
- "github.com/stretchr/testify/require"
- "github.com/tidwall/gjson"
- )
- func f64p(v float64) *float64 { return &v }
- type httpUpstreamRecorder struct {
- lastReq *http.Request
- lastBody []byte
- lastProxyURL string
- requests []*http.Request
- bodies [][]byte
- resp *http.Response
- responses []*http.Response
- err error
- }
- type passthroughErrReadCloser struct {
- err error
- }
- type passthroughCloseTrackingReadCloser struct {
- io.Reader
- closed bool
- }
- func (r *passthroughCloseTrackingReadCloser) Close() error {
- r.closed = true
- return nil
- }
- func (r passthroughErrReadCloser) Read(_ []byte) (int, error) {
- if r.err != nil {
- return 0, r.err
- }
- return 0, io.ErrUnexpectedEOF
- }
- func (r passthroughErrReadCloser) Close() error {
- return nil
- }
- func (u *httpUpstreamRecorder) Do(req *http.Request, proxyURL string, accountID int64, accountConcurrency int) (*http.Response, error) {
- u.lastReq = req
- u.lastProxyURL = proxyURL
- if req != nil && req.Body != nil {
- b, _ := io.ReadAll(req.Body)
- u.lastBody = b
- u.bodies = append(u.bodies, append([]byte(nil), b...))
- _ = req.Body.Close()
- req.Body = io.NopCloser(bytes.NewReader(b))
- }
- u.requests = append(u.requests, req)
- if u.err != nil {
- return nil, u.err
- }
- if len(u.responses) > 0 {
- resp := u.responses[0]
- u.responses = u.responses[1:]
- return resp, nil
- }
- return u.resp, nil
- }
- func (u *httpUpstreamRecorder) DoWithTLS(req *http.Request, proxyURL string, accountID int64, accountConcurrency int, profile *tlsfingerprint.Profile) (*http.Response, error) {
- return u.Do(req, proxyURL, accountID, accountConcurrency)
- }
- func TestOpenAIGatewayService_ResponsesUnknownModelDoesNotFallbackToGPT54(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- originalBody := []byte(`{"model":"gpt6","stream":false,"instructions":"local-test-instructions","input":[{"type":"text","text":"hi"}]}`)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(originalBody))
- c.Request.Header.Set("Content-Type", "application/json")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid_unknown_model"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"type":"invalid_request_error","message":"model not found"}}`)),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- "chatgpt_account_id": "chatgpt-acc",
- },
- Status: StatusActive,
- Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.Error(t, err)
- require.Nil(t, result)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, "https://chatgpt.com/backend-api/codex/responses", upstream.lastReq.URL.String())
- require.Equal(t, "gpt6", gjson.GetBytes(upstream.lastBody, "model").String())
- require.NotEqual(t, "gpt-5.4", gjson.GetBytes(upstream.lastBody, "model").String())
- require.True(t, rec.Code >= http.StatusBadRequest)
- }
- func TestOpenAIGatewayService_OAuthResponsesPromotesSystemMessageWithoutDuplication(t *testing.T) {
- gin.SetMode(gin.TestMode)
- const systemPrompt = "Unique system prefix for Responses token accounting."
- const existingInstructions = "Existing instructions."
- body := []byte(`{"model":"gpt-5.4","stream":false,"instructions":"` + existingInstructions + `","input":[{"role":"system","content":"` + systemPrompt + `"},{"role":"user","content":"hello"}]}`)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(body))
- c.Request.Header.Set("Content-Type", "application/json")
- upstream := &httpUpstreamRecorder{err: errors.New("stop after capture")}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 124,
- Name: "openai-oauth",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- "chatgpt_account_id": "chatgpt-acc",
- },
- Status: StatusActive,
- Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, body)
- require.Error(t, err)
- require.Nil(t, result)
- require.NotEmpty(t, upstream.lastBody)
- require.Equal(t, systemPrompt+"\n\n"+existingInstructions, gjson.GetBytes(upstream.lastBody, "instructions").String())
- require.Equal(t, int64(1), gjson.GetBytes(upstream.lastBody, "input.#").Int())
- require.Equal(t, "user", gjson.GetBytes(upstream.lastBody, "input.0.role").String())
- require.Equal(t, 1, strings.Count(string(upstream.lastBody), systemPrompt))
- }
- func TestOpenAIGatewayService_NativeResponsesBodyModificationPreservesHTMLChars(t *testing.T) {
- gin.SetMode(gin.TestMode)
- payloadText := strings.Repeat(`<tag>&value</tag>`, 128)
- originalBody := []byte(fmt.Sprintf(`{"model":"gpt-5.5","stream":false,"max_output_tokens":100,"previous_response_id":"resp_prev","input":[{"type":"message","role":"user","content":[{"type":"input_text","text":%q}]}]}`, payloadText))
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(originalBody))
- c.Request.Header.Set("Content-Type", "application/json")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid_native_reencode"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"type":"invalid_request_error","message":"stop after capture"}}`)),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Security: config.SecurityConfig{URLAllowlist: config.URLAllowlistConfig{
- Enabled: false,
- AllowInsecureHTTP: true,
- }}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 456,
- Name: "openai-apikey",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "base_url": "http://upstream.example",
- },
- Extra: map[string]any{
- openai_compat.ExtraKeyResponsesMode: string(openai_compat.ResponsesSupportModeAuto),
- openai_compat.ExtraKeyResponsesSupported: true,
- },
- Status: StatusActive,
- Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.Error(t, err)
- require.Nil(t, result)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, "http://upstream.example/v1/responses", upstream.lastReq.URL.String())
- require.Contains(t, string(upstream.lastBody), payloadText)
- require.NotContains(t, string(upstream.lastBody), `\\u003c`)
- require.NotContains(t, string(upstream.lastBody), `\\u003e`)
- require.NotContains(t, string(upstream.lastBody), `\\u0026`)
- }
- func TestOpenAIGatewayService_OAuthMessagesBridgeDoesNotInjectDefaultInstructions(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- originalBody := []byte(`{"model":"gpt-5.5","stream":true,"prompt_cache_key":"anthropic-metadata-session-1","input":[{"type":"message","role":"developer","content":[{"type":"input_text","text":"<sub2api-claude-code-todo-guard>"}]},{"type":"message","role":"user","content":"hello"}]}`)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(originalBody))
- c.Request.Header.Set("Content-Type", "application/json")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid_bridge"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"type":"invalid_request_error","message":"bridge stop"}}`)),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- "chatgpt_account_id": "chatgpt-acc",
- },
- Status: StatusActive,
- Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.Error(t, err)
- require.Nil(t, result)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, "", gjson.GetBytes(upstream.lastBody, "instructions").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "prompt_cache_key").Exists())
- require.NotEmpty(t, upstream.lastReq.Header.Get("Session_Id"))
- require.Empty(t, upstream.lastReq.Header.Get("Conversation_Id"))
- require.Empty(t, upstream.lastReq.Header.Get("OpenAI-Beta"))
- require.Empty(t, upstream.lastReq.Header.Get("originator"))
- }
- type openAIPassthroughFailoverRepo struct {
- stubOpenAIAccountRepo
- rateLimitCalls []time.Time
- overloadCalls []time.Time
- }
- func (r *openAIPassthroughFailoverRepo) SetRateLimited(_ context.Context, _ int64, resetAt time.Time) error {
- r.rateLimitCalls = append(r.rateLimitCalls, resetAt)
- return nil
- }
- func (r *openAIPassthroughFailoverRepo) SetOverloaded(_ context.Context, _ int64, until time.Time) error {
- r.overloadCalls = append(r.overloadCalls, until)
- return nil
- }
- var structuredLogCaptureMu sync.Mutex
- type inMemoryLogSink struct {
- mu sync.Mutex
- events []*logger.LogEvent
- }
- func (s *inMemoryLogSink) WriteLogEvent(event *logger.LogEvent) {
- if event == nil {
- return
- }
- cloned := *event
- if event.Fields != nil {
- cloned.Fields = make(map[string]any, len(event.Fields))
- for k, v := range event.Fields {
- cloned.Fields[k] = v
- }
- }
- s.mu.Lock()
- s.events = append(s.events, &cloned)
- s.mu.Unlock()
- }
- func (s *inMemoryLogSink) ContainsMessage(substr string) bool {
- s.mu.Lock()
- defer s.mu.Unlock()
- for _, ev := range s.events {
- if ev != nil && strings.Contains(ev.Message, substr) {
- return true
- }
- }
- return false
- }
- func (s *inMemoryLogSink) ContainsMessageAtLevel(substr, level string) bool {
- s.mu.Lock()
- defer s.mu.Unlock()
- wantLevel := strings.ToLower(strings.TrimSpace(level))
- for _, ev := range s.events {
- if ev == nil {
- continue
- }
- if strings.Contains(ev.Message, substr) && strings.ToLower(strings.TrimSpace(ev.Level)) == wantLevel {
- return true
- }
- }
- return false
- }
- func (s *inMemoryLogSink) ContainsFieldValue(field, substr string) bool {
- s.mu.Lock()
- defer s.mu.Unlock()
- for _, ev := range s.events {
- if ev == nil || ev.Fields == nil {
- continue
- }
- if v, ok := ev.Fields[field]; ok && strings.Contains(fmt.Sprint(v), substr) {
- return true
- }
- }
- return false
- }
- func (s *inMemoryLogSink) ContainsField(field string) bool {
- s.mu.Lock()
- defer s.mu.Unlock()
- for _, ev := range s.events {
- if ev == nil || ev.Fields == nil {
- continue
- }
- if _, ok := ev.Fields[field]; ok {
- return true
- }
- }
- return false
- }
- func captureStructuredLog(t *testing.T) (*inMemoryLogSink, func()) {
- t.Helper()
- structuredLogCaptureMu.Lock()
- err := logger.Init(logger.InitOptions{
- Level: "debug",
- Format: "json",
- ServiceName: "sub2api",
- Environment: "test",
- Output: logger.OutputOptions{
- ToStdout: true,
- ToFile: false,
- },
- Sampling: logger.SamplingOptions{Enabled: false},
- })
- require.NoError(t, err)
- sink := &inMemoryLogSink{}
- logger.SetSink(sink)
- return sink, func() {
- logger.SetSink(nil)
- structuredLogCaptureMu.Unlock()
- }
- }
- func TestOpenAIGatewayService_OAuthPassthrough_StreamKeepsToolNameAndBodyNormalized(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- c.Request.Header.Set("Authorization", "Bearer inbound-should-not-forward")
- c.Request.Header.Set("Cookie", "secret=1")
- c.Request.Header.Set("X-Api-Key", "sk-inbound")
- c.Request.Header.Set("X-Goog-Api-Key", "goog-inbound")
- c.Request.Header.Set("Accept-Encoding", "gzip")
- c.Request.Header.Set("Proxy-Authorization", "Basic abc")
- c.Request.Header.Set("X-Test", "keep")
- c.Request.Header.Set("x-codex-beta-features", "remote_compaction_v2")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"store":true,"instructions":"local-test-instructions","input":[{"type":"text","text":"hi"}]}`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_item.added","item":{"type":"tool_call","tool_calls":[{"function":{"name":"apply_patch"}}]}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- openAITokenProvider: &OpenAITokenProvider{ // minimal: will be bypassed by nil cache/service, but GetAccessToken uses provider only if non-nil
- accountRepo: nil,
- },
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- // Use the gateway method that reads token from credentials when provider is nil.
- svc.openAITokenProvider = nil
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.True(t, result.Stream)
- // 1) 透传 OAuth 请求体与旧链路关键行为保持一致:store=false + stream=true。
- require.Equal(t, false, gjson.GetBytes(upstream.lastBody, "store").Bool())
- require.Equal(t, true, gjson.GetBytes(upstream.lastBody, "stream").Bool())
- require.Equal(t, "local-test-instructions", strings.TrimSpace(gjson.GetBytes(upstream.lastBody, "instructions").String()))
- // 其余关键字段保持原值。
- require.Equal(t, "gpt-5.2", gjson.GetBytes(upstream.lastBody, "model").String())
- require.Equal(t, "hi", gjson.GetBytes(upstream.lastBody, "input.0.text").String())
- // 2) only auth is replaced; inbound auth/cookie are not forwarded
- require.Equal(t, "Bearer oauth-token", upstream.lastReq.Header.Get("Authorization"))
- // 强制统一出口:客户端自报的 codex_cli_rs/0.1.0 不会到达上游。
- require.Equal(t, codexCLIUserAgent, upstream.lastReq.Header.Get("User-Agent"))
- require.Empty(t, upstream.lastReq.Header.Get("Cookie"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Api-Key"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Goog-Api-Key"))
- require.Empty(t, upstream.lastReq.Header.Get("Accept-Encoding"))
- require.Empty(t, upstream.lastReq.Header.Get("Proxy-Authorization"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Test"))
- require.Equal(t, "remote_compaction_v2", upstream.lastReq.Header.Get("x-codex-beta-features"))
- // 3) required OAuth headers are present
- require.Equal(t, "chatgpt.com", upstream.lastReq.Host)
- require.Equal(t, "chatgpt-acc", upstream.lastReq.Header.Get("chatgpt-account-id"))
- // 4) downstream SSE keeps tool name (no toolCorrector)
- body := rec.Body.String()
- require.Contains(t, body, "apply_patch")
- require.NotContains(t, body, "\"name\":\"edit\"")
- }
- func TestOpenAIGatewayService_OAuthPassthrough_GroupForceOpenAIFastInjectsMissingTier(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- body := []byte(`{"model":"gpt-5.6-sol","stream":true,"instructions":"test","input":"hi"}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 123, Name: "oauth", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- RateMultiplier: f64p(1),
- }
- ctx := context.WithValue(context.Background(), ctxkey.Group, &Group{
- ID: 7, Platform: PlatformOpenAI, Status: StatusActive, Hydrated: true, ForceOpenAIFast: true,
- })
- result, err := svc.Forward(ctx, c, account, body)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Equal(t, OpenAIFastTierPriority, gjson.GetBytes(upstream.lastBody, "service_tier").String())
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, OpenAIFastTierPriority, *result.ServiceTier)
- }
- // 「自动透传(仅替换认证)」的默认行为必须真的只替换认证:namespace 声明、
- // namespace 形态的 tool_choice、历史调用项上的 namespace 都原样转发,只清掉
- // 非调用项上的残留 namespace(Codex 协议里只有调用项会带该字段)。
- func TestOpenAIGatewayService_OAuthPassthrough_PreservesNamespaceRequest(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- originalBody := []byte(`{
- "model":"gpt-5.5",
- "stream":true,
- "instructions":"local-test-instructions",
- "tools":[
- {"type":"function","name":"plain","description":"keep","parameters":{"type":"object"}},
- {"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent","description":"spawn","parameters":{"type":"object"}}]}
- ],
- "tool_choice":{"type":"function","name":"spawn_agent","namespace":"collaboration"},
- "input":[
- {"type":"function_call","call_id":"call_old","name":"spawn_agent","namespace":"collaboration","arguments":"{}"},
- {"type":"message","role":"user","namespace":"residual","content":[{"type":"input_text","text":"keep","namespace":"nested"}]}
- ]
- }`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_item.done","output_index":0,"item":{"type":"function_call","id":"fc_1","call_id":"call_1","name":"spawn_agent","namespace":"collaboration","arguments":"{}"}}`,
- "",
- `data: {"type":"response.completed","response":{"id":"resp_1","status":"completed","output":[],"usage":{"input_tokens":2,"output_tokens":1,"total_tokens":3}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_preserve"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 125, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Len(t, gjson.GetBytes(upstream.lastBody, "tools").Array(), 2)
- require.Equal(t, "namespace", gjson.GetBytes(upstream.lastBody, "tools.1.type").String())
- require.Equal(t, "collaboration", gjson.GetBytes(upstream.lastBody, "tools.1.name").String())
- require.Equal(t, "spawn_agent", gjson.GetBytes(upstream.lastBody, "tools.1.tools.0.name").String())
- require.Equal(t, "collaboration", gjson.GetBytes(upstream.lastBody, "tool_choice.namespace").String())
- require.Equal(t, "spawn_agent", gjson.GetBytes(upstream.lastBody, "tool_choice.name").String())
- require.Equal(t, "collaboration", gjson.GetBytes(upstream.lastBody, "input.0.namespace").String())
- require.Equal(t, "spawn_agent", gjson.GetBytes(upstream.lastBody, "input.0.name").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "input.1.namespace").Exists())
- require.Equal(t, "nested", gjson.GetBytes(upstream.lastBody, "input.1.content.0.namespace").String())
- require.NotContains(t, string(upstream.lastBody), "collaboration__spawn_agent")
- // 未摊平即无需回程还原,上游事件原样下发。
- downstream := rec.Body.String()
- require.Contains(t, downstream, `"name":"spawn_agent"`)
- require.Contains(t, downstream, `"namespace":"collaboration"`)
- }
- // 兼容开关打开时的旧行为:摊平请求、回程还原。默认路径见
- // TestOpenAIGatewayService_OAuthPassthrough_PreservesNamespaceRequest。
- func TestOpenAIGatewayService_OAuthPassthrough_FlattenEnabledNamespaceRequestAndStreamResponse(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- originalBody := []byte(`{
- "model":"gpt-5.5",
- "stream":true,
- "instructions":"local-test-instructions",
- "tools":[
- {"type":"function","name":"plain","description":"keep","parameters":{"type":"object"}},
- {"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent","description":"spawn","parameters":{"type":"object"}}]}
- ],
- "tool_choice":{"type":"function","name":"spawn_agent","namespace":"collaboration"},
- "input":[
- {"type":"function_call","call_id":"call_old","name":"spawn_agent","namespace":"collaboration","arguments":"{}"},
- {"type":"message","role":"user","namespace":"residual","content":[{"type":"input_text","text":"keep","namespace":"nested"}]}
- ]
- }`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_item.added","output_index":0,"item":{"type":"function_call","id":"fc_1","call_id":"call_1","name":"collaboration__spawn_agent","arguments":""}}`,
- "",
- `data: {"type":"response.output_item.done","output_index":0,"item":{"type":"function_call","id":"fc_1","call_id":"call_1","name":"collaboration__spawn_agent","arguments":"{}"}}`,
- "",
- `data: {"type":"response.completed","response":{"id":"resp_1","status":"completed","output":[{"type":"function_call","id":"fc_1","call_id":"call_1","name":"collaboration__spawn_agent","arguments":"{}"}],"usage":{"input_tokens":2,"output_tokens":1,"total_tokens":3}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_namespace"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{
- "openai_passthrough": true,
- "openai_responses_flatten_namespaces": true,
- },
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Len(t, gjson.GetBytes(upstream.lastBody, "tools").Array(), 2)
- require.Equal(t, "plain", gjson.GetBytes(upstream.lastBody, "tools.0.name").String())
- require.Equal(t, "function", gjson.GetBytes(upstream.lastBody, "tools.1.type").String())
- require.Equal(t, "collaboration__spawn_agent", gjson.GetBytes(upstream.lastBody, "tools.1.name").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "tools.1.tools").Exists())
- require.Equal(t, "collaboration__spawn_agent", gjson.GetBytes(upstream.lastBody, "tool_choice.name").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "tool_choice.namespace").Exists())
- require.Equal(t, "collaboration__spawn_agent", gjson.GetBytes(upstream.lastBody, "input.0.name").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "input.0.namespace").Exists())
- require.False(t, gjson.GetBytes(upstream.lastBody, "input.1.namespace").Exists())
- require.Equal(t, "nested", gjson.GetBytes(upstream.lastBody, "input.1.content.0.namespace").String())
- require.Len(t, upstream.bodies, 1)
- downstream := rec.Body.String()
- require.NotContains(t, downstream, "collaboration__spawn_agent")
- require.Contains(t, downstream, `"name":"spawn_agent"`)
- require.Contains(t, downstream, `"namespace":"collaboration"`)
- }
- // 兼容开关打开时的旧行为;默认保留路径见
- // TestOpenAIGatewayService_OAuthPreservesCodexNamespaceTools。
- func TestOpenAIGatewayService_NativeOAuth_FlattenEnabledNamespaceRequestAndStreamResponse(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- body := []byte(`{
- "model":"gpt-5.5","stream":true,"instructions":"test",
- "tools":[{"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent","parameters":{"type":"object"}}]}],
- "input":[{"type":"function_call","call_id":"call_old","name":"spawn_agent","namespace":"collaboration","arguments":"{}"}]
- }`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_item.added","output_index":0,"item":{"type":"function_call","id":"fc_1","call_id":"call_1","name":"collaboration__spawn_agent","arguments":""}}`,
- "",
- `data: {"type":"response.completed","response":{"id":"resp_1","status":"completed","output":[{"type":"function_call","id":"fc_1","call_id":"call_1","name":"collaboration__spawn_agent","arguments":"{}"}],"usage":{"input_tokens":2,"output_tokens":1,"total_tokens":3}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_native_namespace"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 124, Name: "native", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_responses_flatten_namespaces": true},
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, body)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Equal(t, "function", gjson.GetBytes(upstream.lastBody, "tools.0.type").String())
- require.Equal(t, "collaboration__spawn_agent", gjson.GetBytes(upstream.lastBody, "tools.0.name").String())
- require.Equal(t, "collaboration__spawn_agent", gjson.GetBytes(upstream.lastBody, "input.0.name").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "input.0.namespace").Exists())
- require.NotContains(t, rec.Body.String(), "collaboration__spawn_agent")
- require.Contains(t, rec.Body.String(), `"name":"spawn_agent"`)
- require.Contains(t, rec.Body.String(), `"namespace":"collaboration"`)
- }
- func TestOpenAIGatewayService_NativeOAuth_NamespaceNonStreamingResponse(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- setOpenAIResponsesNamespaceNames(c, map[string]apicompat.ResponsesNamespaceName{
- "collaboration__spawn_agent": {Namespace: "collaboration", Name: "spawn_agent"},
- })
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(`{
- "id":"resp_1","output":[{"type":"function_call","name":"collaboration__spawn_agent","call_id":"call_1","arguments":"{}"}],
- "usage":{"input_tokens":1,"output_tokens":1,"total_tokens":2}
- }`)),
- }
- result, err := (&OpenAIGatewayService{cfg: &config.Config{}}).handleNonStreamingResponse(
- context.Background(), resp, c, &Account{Type: AccountTypeOAuth}, "gpt-5.5", "gpt-5.5",
- )
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotContains(t, rec.Body.String(), "collaboration__spawn_agent")
- require.Contains(t, rec.Body.String(), `"name":"spawn_agent"`)
- require.Contains(t, rec.Body.String(), `"namespace":"collaboration"`)
- }
- func TestOpenAIGatewayService_OAuthPassthrough_NamespaceNonStreamingResponse(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(`{"id":"resp_1","output":[{"type":"function_call","name":"collaboration__spawn_agent","call_id":"call_1","arguments":"{}"}],"usage":{"input_tokens":1,"output_tokens":1}}`)),
- }
- names := map[string]apicompat.ResponsesNamespaceName{
- "collaboration__spawn_agent": {Namespace: "collaboration", Name: "spawn_agent"},
- }
- setOpenAIResponsesNamespaceNames(c, names)
- result, err := (&OpenAIGatewayService{cfg: &config.Config{}}).handleNonStreamingResponsePassthrough(
- context.Background(), resp, c, &Account{ID: 91}, "gpt-5.5", "",
- )
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotContains(t, rec.Body.String(), "collaboration__spawn_agent")
- require.Contains(t, rec.Body.String(), `"name":"spawn_agent"`)
- require.Contains(t, rec.Body.String(), `"namespace":"collaboration"`)
- }
- // 摊平名冲突只在兼容开关打开时才可能发生:默认保留 namespace,不存在平名冲突。
- func TestOpenAIGatewayService_OAuthPassthrough_FlattenEnabledNamespaceCollisionReturnsBadRequest(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- body := []byte(`{
- "model":"gpt-5.5","stream":true,"instructions":"test",
- "tools":[
- {"type":"function","name":"collaboration__spawn_agent","parameters":{"type":"object"}},
- {"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent","parameters":{"type":"object"}}]}
- ],"input":"hi"
- }`)
- upstream := &httpUpstreamRecorder{}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{
- "openai_passthrough": true,
- "openai_responses_flatten_namespaces": true,
- },
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, body)
- require.Error(t, err)
- require.Nil(t, result)
- require.Nil(t, upstream.lastReq)
- require.Equal(t, http.StatusBadRequest, rec.Code)
- require.Equal(t, "invalid_request_error", gjson.Get(rec.Body.String(), "error.type").String())
- require.Equal(t, "tools", gjson.Get(rec.Body.String(), "error.param").String())
- require.Contains(t, gjson.Get(rec.Body.String(), "error.message").String(), "conflicts with a top-level tool")
- }
- func TestOpenAIGatewayService_OAuthPassthrough_CompactUsesJSONAndKeepsNonStreaming(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses/compact", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- c.Request.Header.Set("Content-Type", "application/json")
- originalBody := []byte(`{"model":"gpt-5.1-codex","stream":true,"store":true,"instructions":"local-test-instructions","input":[{"type":"text","text":"compact me"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid-compact"}},
- Body: io.NopCloser(strings.NewReader(`{"id":"cmp_123","usage":{"input_tokens":11,"output_tokens":22}}`)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- "chatgpt_account_id": "chatgpt-acc",
- "model_mapping": map[string]any{"gpt-5.1-codex": "gpt-5.1-account"},
- "compact_model_mapping": map[string]any{"gpt-5.1-codex": "gpt-5.1-compact"},
- },
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.False(t, result.Stream)
- require.False(t, gjson.GetBytes(upstream.lastBody, "store").Exists())
- require.False(t, gjson.GetBytes(upstream.lastBody, "stream").Exists())
- require.Equal(t, "gpt-5.1-compact", gjson.GetBytes(upstream.lastBody, "model").String())
- require.Equal(t, "compact me", gjson.GetBytes(upstream.lastBody, "input.0.text").String())
- require.Equal(t, "local-test-instructions", strings.TrimSpace(gjson.GetBytes(upstream.lastBody, "instructions").String()))
- require.Equal(t, "application/json", upstream.lastReq.Header.Get("Accept"))
- require.Equal(t, codexCLIVersion, upstream.lastReq.Header.Get("Version"))
- require.NotEmpty(t, upstream.lastReq.Header.Get("Session_Id"))
- require.Equal(t, "chatgpt.com", upstream.lastReq.Host)
- require.Equal(t, "chatgpt-acc", upstream.lastReq.Header.Get("chatgpt-account-id"))
- require.Contains(t, rec.Body.String(), `"id":"cmp_123"`)
- }
- func TestOpenAIGatewayService_OAuthPassthrough_UpstreamRequestIgnoresClientCancel(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- reqCtx, cancel := context.WithCancel(context.Background())
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil)).WithContext(reqCtx)
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- cancel()
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"store":true,"instructions":"local-test-instructions","input":[{"type":"text","text":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_passthrough_ctx"}},
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":2,"output_tokens":1}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n"))),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true, "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(reqCtx, c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotNil(t, upstream.lastReq)
- require.NoError(t, upstream.lastReq.Context().Err())
- }
- func TestOpenAIGatewayService_OAuthPassthrough_CodexMissingInstructionsGetsDefault(t *testing.T) {
- gin.SetMode(gin.TestMode)
- for _, stream := range []bool{false, true} {
- t.Run(fmt.Sprintf("stream=%t", stream), func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- path := "/v1/responses"
- responseBody := strings.Join([]string{
- `data: {"type":"response.completed","response":{"id":"resp_1","status":"completed","output":[],"usage":{"input_tokens":1,"output_tokens":1}}}`,
- "", "data: [DONE]", "",
- }, "\n")
- responseContentType := "text/event-stream"
- if !stream {
- path = "/v1/responses/compact"
- responseBody = `{"id":"resp_1","status":"completed","output":[],"usage":{"input_tokens":1,"output_tokens":1}}`
- responseContentType = "application/json"
- }
- c.Request = httptest.NewRequest(http.MethodPost, path, bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.98.0")
- originalBody := []byte(fmt.Sprintf(`{"model":"gpt-5.1-codex-max","stream":%t,"store":true,"input":[{"type":"text","text":"hi"}]}`, stream))
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{responseContentType}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(responseBody)),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true, "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff},
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotNil(t, upstream.lastReq)
- if stream {
- require.True(t, gjson.GetBytes(upstream.lastBody, "stream").Bool())
- } else {
- require.False(t, gjson.GetBytes(upstream.lastBody, "stream").Exists())
- }
- require.Equal(t, strings.TrimSpace(defaultCodexSynthInstructions("gpt-5.1-codex-max")), strings.TrimSpace(gjson.GetBytes(upstream.lastBody, "instructions").String()))
- })
- }
- }
- func TestOpenAIGatewayService_Forward_MissingInstructionsUsesMappedModelTemplate(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.98.0")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_mapped_astra"}},
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.completed","response":{"id":"resp_1","status":"completed","output":[],"usage":{"input_tokens":1,"output_tokens":1}}}`,
- "", "data: [DONE]", "",
- }, "\n"))),
- }}
- svc := &OpenAIGatewayService{cfg: &config.Config{}, httpUpstream: upstream}
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- "chatgpt_account_id": "chatgpt-acc",
- "model_mapping": map[string]any{"astra-public": "gpt-6-astra"},
- },
- Extra: map[string]any{
- "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff,
- },
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"astra-public","stream":true,"store":true,"input":[{"type":"text","text":"hi"}]}`))
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Equal(t, "gpt-6-astra", gjson.GetBytes(upstream.lastBody, "model").String())
- instructions := gjson.GetBytes(upstream.lastBody, "instructions").String()
- require.True(t, strings.HasPrefix(strings.TrimSpace(instructions), "You are Codex, an agent based on GPT-6."))
- require.NotContains(t, instructions, "You are Codex, a coding agent based on GPT-5.")
- }
- func TestOpenAIGatewayService_OAuthPassthrough_DisabledUsesLegacyTransform(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- // store=true + stream=false should be forced to store=false + stream=true by applyCodexOAuthTransform (OAuth legacy path)
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": false},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- // legacy path rewrites request body (not byte-equal)
- require.NotEqual(t, inputBody, upstream.lastBody)
- require.Contains(t, string(upstream.lastBody), `"store":false`)
- require.Contains(t, string(upstream.lastBody), `"stream":true`)
- }
- func TestOpenAIGatewayService_OAuthLegacy_GroupForceOpenAIFastInjectsMissingTier(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- inputBody := []byte(`{"model":"gpt-5.6-sol","stream":false,"input":[{"type":"text","text":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_group_fast_legacy"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := newOpenAIGatewayServiceWithSettings(t, DefaultOpenAIFastPolicySettings())
- svc.cfg = &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}}
- svc.httpUpstream = upstream
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": false, "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff},
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- ctx := context.WithValue(context.Background(), ctxkey.Group, &Group{
- ID: 7, Platform: PlatformOpenAI, Status: StatusActive, Hydrated: true, ForceOpenAIFast: true,
- })
- result, err := svc.Forward(ctx, c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.Equal(t, OpenAIFastTierPriority, gjson.GetBytes(upstream.lastBody, "service_tier").String())
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, OpenAIFastTierPriority, *result.ServiceTier)
- }
- func TestOpenAIGatewayService_OAuthLegacy_GroupForceStillHonorsGlobalFilter(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- inputBody := []byte(`{"model":"gpt-5.6-sol","stream":false,"input":[{"type":"text","text":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_group_fast_filter"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := newOpenAIGatewayServiceWithSettings(t, openAIFastFilterPriorityPolicy())
- svc.cfg = &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}}
- svc.httpUpstream = upstream
- account := &Account{
- ID: 123, Name: "acc", Platform: PlatformOpenAI, Type: AccountTypeOAuth, Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": false, "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff},
- Status: StatusActive, Schedulable: true, RateMultiplier: f64p(1),
- }
- ctx := context.WithValue(context.Background(), ctxkey.Group, &Group{
- ID: 7, Platform: PlatformOpenAI, Status: StatusActive, Hydrated: true, ForceOpenAIFast: true,
- })
- result, err := svc.Forward(ctx, c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.False(t, gjson.GetBytes(upstream.lastBody, "service_tier").Exists())
- require.Nil(t, result.ServiceTier)
- }
- func TestOpenAIGatewayService_OAuthLegacy_UpstreamRequestIgnoresClientCancel(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- reqCtx, cancel := context.WithCancel(context.Background())
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil)).WithContext(reqCtx)
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- cancel()
- originalBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_legacy_ctx"}},
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":1,"output_tokens":1}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n"))),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": false, "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModeOff},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(reqCtx, c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotNil(t, upstream.lastReq)
- require.NoError(t, upstream.lastReq.Context().Err())
- }
- func TestOpenAIGatewayService_OAuthLegacy_CompositeCodexUAUsesCodexOriginator(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- // 复合 UA(前缀不是 codex_cli_rs),历史实现会误判为非 Codex 并走 opencode。
- c.Request.Header.Set("User-Agent", "Mozilla/5.0 codex_cli_rs/0.1.0")
- inputBody := []byte(`{"model":"gpt-5.2","stream":true,"store":false,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": false},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- // 浏览器型复合 UA 被替换为默认 Codex TUI UA,
- // originator 随最终 UA 配套(issue #3901)。
- require.Equal(t, DefaultOpenAICodexUserAgent, upstream.lastReq.Header.Get("User-Agent"))
- require.Equal(t, openai.CodexDefaultOriginator, upstream.lastReq.Header.Get("originator"))
- require.NotEqual(t, "opencode", upstream.lastReq.Header.Get("originator"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_ResponseHeadersAllowXCodex(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- headers := make(http.Header)
- headers.Set("Content-Type", "application/json")
- headers.Set("x-request-id", "rid")
- headers.Set("x-codex-primary-used-percent", "12")
- headers.Set("x-codex-secondary-used-percent", "34")
- headers.Set("x-codex-primary-window-minutes", "300")
- headers.Set("x-codex-secondary-window-minutes", "10080")
- headers.Set("x-codex-primary-reset-after-seconds", "1")
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: headers,
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.output_text.delta","delta":"h"}`,
- "",
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":1,"output_tokens":1,"input_tokens_details":{"cached_tokens":0}}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n"))),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.Equal(t, "12", rec.Header().Get("x-codex-primary-used-percent"))
- require.Equal(t, "34", rec.Header().Get("x-codex-secondary-used-percent"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_UpstreamErrorIncludesPassthroughFlag(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- originalBody := []byte(`{"model":"gpt-5.2","stream":false,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"message":"bad"}}`)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.Error(t, err)
- require.True(t, c.Writer.Written(), "非 429/529 的 passthrough 错误应直接写回客户端")
- require.Equal(t, http.StatusBadRequest, rec.Code)
- // should append an upstream error event with passthrough=true
- v, ok := c.Get(OpsUpstreamErrorsKey)
- require.True(t, ok)
- arr, ok := v.([]*OpsUpstreamErrorEvent)
- require.True(t, ok)
- require.NotEmpty(t, arr)
- require.True(t, arr[len(arr)-1].Passthrough)
- require.Equal(t, "http_error", arr[len(arr)-1].Kind)
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_RebuildsUpstreamErrors(t *testing.T) {
- gin.SetMode(gin.TestMode)
- tests := []struct {
- name string
- statusCode int
- contentType string
- responseBody string
- retryAfter string
- wantStatus int
- wantMessage string
- wantRetryAfter string
- }{
- {
- name: "upstream forbidden is reported as gateway failure",
- statusCode: http.StatusForbidden,
- contentType: "text/html; charset=UTF-8",
- responseBody: `<!DOCTYPE html><title>secret-upstream.example denied the request</title>`,
- retryAfter: "17",
- wantStatus: http.StatusBadGateway,
- wantMessage: "Upstream access denied",
- wantRetryAfter: "17",
- },
- {
- name: "upstream unauthorized is reported as gateway failure",
- statusCode: http.StatusUnauthorized,
- contentType: "application/json",
- responseBody: `{"error":{"message":"invalid secret-upstream.example token","type":"authentication_error","code":"invalid_api_key","param":"api_key"},"rate_limit":{"remaining":0}}`,
- wantStatus: http.StatusBadGateway,
- wantMessage: "Upstream authentication failed",
- },
- // 瞬时 5xx(500/502/503/504/520-524)对 API-key 账号已改走多账号
- // failover(见 APIKeyPassthrough_Transient5xxTriggersFailover),此处
- // 改用非瞬时 5xx 状态码,继续覆盖净化重建路径。
- {
- name: "html 5xx",
- statusCode: 530,
- contentType: "text/html; charset=UTF-8",
- responseBody: `<!DOCTYPE html><title>secret-upstream.example | 530: Origin DNS error</title>`,
- wantStatus: 530,
- wantMessage: "Upstream service temporarily unavailable",
- },
- {
- name: "structured 5xx",
- statusCode: http.StatusNotImplemented,
- contentType: "application/json",
- responseBody: `{"error":{"message":"secret-upstream.example internal failure"}}`,
- wantStatus: http.StatusNotImplemented,
- wantMessage: "Upstream service temporarily unavailable",
- },
- {
- name: "unstructured 4xx",
- statusCode: http.StatusBadRequest,
- contentType: "text/plain",
- responseBody: `proxy secret-upstream.example rejected the request`,
- wantStatus: http.StatusBadRequest,
- wantMessage: "Upstream request failed",
- },
- {
- name: "malicious valid json 4xx",
- statusCode: http.StatusBadRequest,
- contentType: "application/json",
- responseBody: `{"error":{"message":"secret-upstream.example invalid parameter","type":"invalid_request_error","code":"upstream_secret_code","param":"private_field","internal_token":"sk-upstream-secret"},"rate_limit":{"remaining":0,"reset":"internal-window"},"debug":{"admin":"root"},"redirect":"https://secret-upstream.example/admin"}`,
- retryAfter: "not-a-valid-delay",
- wantStatus: http.StatusBadRequest,
- wantMessage: "Upstream request failed",
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: tt.statusCode,
- Header: http.Header{
- "Content-Type": []string{tt.contentType},
- "Location": []string{"https://secret-upstream.example/admin"},
- "Retry-After": []string{tt.retryAfter},
- "Server": []string{"secret-upstream-proxy"},
- "Set-Cookie": []string{"admin_token=secret"},
- "WWW-Authenticate": []string{`Bearer realm="secret-upstream.example"`},
- "X-Admin-Debug": []string{"internal-route=secret-upstream.example"},
- "X-Codex-Primary-Used-Percent": []string{"99"},
- "x-request-id": []string{"rid-sensitive-upstream"},
- },
- Body: io.NopCloser(strings.NewReader(tt.responseBody)),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 124,
- Name: "sensitive-upstream",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "base_url": "https://secret-upstream.example",
- },
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- }
- requestBody := []byte(`{"model":"gpt-5.2","stream":false,"input":"hello"}`)
- _, err := svc.Forward(context.Background(), c, account, requestBody)
- require.Error(t, err)
- require.Equal(t, tt.wantStatus, rec.Code)
- opsValue, ok := c.Get(OpsUpstreamErrorsKey)
- require.True(t, ok)
- opsEvents, ok := opsValue.([]*OpsUpstreamErrorEvent)
- require.True(t, ok)
- require.NotEmpty(t, opsEvents)
- require.Equal(t, tt.statusCode, opsEvents[len(opsEvents)-1].UpstreamStatusCode)
- require.Contains(t, rec.Header().Get("Content-Type"), "application/json")
- require.Equal(t, tt.wantRetryAfter, rec.Header().Get("Retry-After"))
- for _, key := range []string{
- "Location",
- "Server",
- "Set-Cookie",
- "WWW-Authenticate",
- "X-Admin-Debug",
- "X-Codex-Primary-Used-Percent",
- "X-Request-Id",
- } {
- require.Empty(t, rec.Header().Values(key), "sensitive upstream header %s must be dropped", key)
- }
- require.Equal(t, "upstream_error", gjson.Get(rec.Body.String(), "error.type").String())
- require.Equal(t, tt.wantMessage, gjson.Get(rec.Body.String(), "error.message").String())
- require.False(t, gjson.Get(rec.Body.String(), "error.code").Exists())
- require.False(t, gjson.Get(rec.Body.String(), "error.param").Exists())
- require.False(t, gjson.Get(rec.Body.String(), "rate_limit").Exists())
- require.NotContains(t, rec.Body.String(), "secret-upstream.example")
- require.NotContains(t, rec.Body.String(), "sk-upstream-secret")
- require.NotContains(t, err.Error(), "secret-upstream.example")
- })
- }
- }
- func TestWriteOpenAIPassthroughErrorHeaders_StrictRetryAfter(t *testing.T) {
- now := time.Now().UTC()
- tests := []struct {
- name string
- raw string
- want bool
- }{
- {name: "positive delay seconds", raw: "17", want: true},
- {name: "fractional delay", raw: "1.5"},
- {name: "scientific notation", raw: "1e3"},
- {name: "explicit plus sign", raw: "+17"},
- {name: "zero", raw: "0"},
- {name: "negative delay", raw: "-1"},
- {name: "uint64 overflow", raw: "18446744073709551616"},
- {name: "future http date", raw: now.Add(time.Hour).Format(http.TimeFormat), want: true},
- {name: "past http date", raw: now.Add(-time.Hour).Format(http.TimeFormat)},
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- dst := http.Header{"Retry-After": []string{"stale"}}
- writeOpenAIPassthroughErrorHeaders(dst, http.Header{"Retry-After": []string{tt.raw}})
- if tt.want {
- require.Equal(t, tt.raw, dst.Get("Retry-After"))
- } else {
- require.Empty(t, dst.Get("Retry-After"))
- }
- })
- }
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_CompactErrorBeforeKeepaliveIsSingleJSON(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses/compact", bytes.NewReader(nil))
- MarkOpenAICompactClientStream(c)
- stop := StartOpenAICompactSSEKeepalive(c, time.Hour)
- defer stop()
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"message":"secret-upstream.example invalid request"}}`)),
- }},
- }
- account := &Account{
- ID: 125, Platform: PlatformOpenAI, Type: AccountTypeAPIKey, Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test", "base_url": "https://secret-upstream.example"},
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- }
- _, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"gpt-5.2","input":"hello"}`))
- require.Error(t, err)
- require.Equal(t, http.StatusBadRequest, rec.Code)
- require.True(t, gjson.Valid(rec.Body.String()))
- require.Equal(t, "upstream_error", gjson.Get(rec.Body.String(), "error.type").String())
- require.NotContains(t, rec.Body.String(), "event:")
- require.NotContains(t, rec.Body.String(), ": keepalive")
- require.NotContains(t, rec.Body.String(), "secret-upstream.example")
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_CompactErrorAfterKeepaliveIsFailedSSE(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses/compact", bytes.NewReader(nil))
- MarkOpenAICompactClientStream(c)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadRequest,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"message":"secret-upstream.example invalid request"}}`)),
- }},
- }
- account := &Account{
- ID: 126, Platform: PlatformOpenAI, Type: AccountTypeAPIKey, Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test", "base_url": "https://secret-upstream.example"},
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- }
- _, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"gpt-5.2","input":"hello"}`))
- require.Error(t, err)
- require.Equal(t, http.StatusOK, rec.Code)
- require.Contains(t, rec.Result().Header.Get("Content-Type"), "text/event-stream")
- events := parseCompactBridgeSSE(t, stripKeepaliveComments(rec.Body.String()))
- require.Len(t, events, 1)
- require.Equal(t, "response.failed", events[0][0])
- require.Equal(t, "failed", gjson.Get(events[0][1], "response.status").String())
- require.Equal(t, "upstream_error", gjson.Get(events[0][1], "response.error.code").String())
- require.Equal(t, "Upstream request failed", gjson.Get(events[0][1], "response.error.message").String())
- require.NotContains(t, rec.Body.String(), "secret-upstream.example")
- }
- func TestOpenAIGatewayService_OpenAIPassthrough_RetryableStatusesTriggerFailover(t *testing.T) {
- gin.SetMode(gin.TestMode)
- originalBody := []byte(`{"model":"gpt-5.2","stream":false,"instructions":"local-test-instructions","input":[{"type":"text","text":"hi"}]}`)
- newAccount := func(accountType string) *Account {
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: accountType,
- Concurrency: 1,
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- switch accountType {
- case AccountTypeOAuth:
- account.Credentials = map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"}
- case AccountTypeAPIKey:
- account.Credentials = map[string]any{"api_key": "sk-test"}
- }
- return account
- }
- testCases := []struct {
- name string
- accountType string
- statusCode int
- body string
- expectFailover bool
- assertRepo func(t *testing.T, repo *openAIPassthroughFailoverRepo, start time.Time)
- }{
- {
- name: "oauth_429_rate_limit",
- accountType: AccountTypeOAuth,
- statusCode: http.StatusTooManyRequests,
- body: func() string {
- resetAt := time.Now().Add(7 * 24 * time.Hour).Unix()
- return fmt.Sprintf(`{"error":{"message":"The usage limit has been reached","type":"usage_limit_reached","resets_at":%d}}`, resetAt)
- }(),
- expectFailover: true,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, _ time.Time) {
- require.Len(t, repo.rateLimitCalls, 1)
- require.Empty(t, repo.overloadCalls)
- require.True(t, time.Until(repo.rateLimitCalls[0]) > 24*time.Hour)
- },
- },
- {
- name: "oauth_529_overload",
- accountType: AccountTypeOAuth,
- statusCode: 529,
- body: `{"error":{"message":"server overloaded","type":"server_error"}}`,
- expectFailover: true,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, start time.Time) {
- require.Empty(t, repo.rateLimitCalls)
- require.Len(t, repo.overloadCalls, 1)
- require.WithinDuration(t, start.Add(10*time.Minute), repo.overloadCalls[0], 5*time.Second)
- },
- },
- {
- name: "oauth_502_bad_gateway",
- accountType: AccountTypeOAuth,
- statusCode: http.StatusBadGateway,
- body: `{"error":{"message":"bad gateway","type":"server_error"}}`,
- expectFailover: false,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, _ time.Time) {
- require.Empty(t, repo.rateLimitCalls)
- require.Empty(t, repo.overloadCalls)
- },
- },
- {
- name: "oauth_503_unavailable",
- accountType: AccountTypeOAuth,
- statusCode: http.StatusServiceUnavailable,
- body: `{"error":{"message":"service unavailable","type":"server_error"}}`,
- expectFailover: false,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, _ time.Time) {
- require.Empty(t, repo.rateLimitCalls)
- require.Empty(t, repo.overloadCalls)
- },
- },
- {
- name: "oauth_504_gateway_timeout",
- accountType: AccountTypeOAuth,
- statusCode: http.StatusGatewayTimeout,
- body: `{"error":{"message":"gateway timeout","type":"server_error"}}`,
- expectFailover: false,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, _ time.Time) {
- require.Empty(t, repo.rateLimitCalls)
- require.Empty(t, repo.overloadCalls)
- },
- },
- {
- name: "apikey_429_rate_limit",
- accountType: AccountTypeAPIKey,
- statusCode: http.StatusTooManyRequests,
- body: func() string {
- resetAt := time.Now().Add(7 * 24 * time.Hour).Unix()
- return fmt.Sprintf(`{"error":{"message":"The usage limit has been reached","type":"usage_limit_reached","resets_at":%d}}`, resetAt)
- }(),
- expectFailover: true,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, _ time.Time) {
- require.Len(t, repo.rateLimitCalls, 1)
- require.Empty(t, repo.overloadCalls)
- require.True(t, time.Until(repo.rateLimitCalls[0]) > 24*time.Hour)
- },
- },
- {
- name: "apikey_529_overload",
- accountType: AccountTypeAPIKey,
- statusCode: 529,
- body: `{"error":{"message":"server overloaded","type":"server_error"}}`,
- expectFailover: true,
- assertRepo: func(t *testing.T, repo *openAIPassthroughFailoverRepo, start time.Time) {
- require.Empty(t, repo.rateLimitCalls)
- require.Len(t, repo.overloadCalls, 1)
- require.WithinDuration(t, start.Add(10*time.Minute), repo.overloadCalls[0], 5*time.Second)
- },
- },
- }
- for _, tc := range testCases {
- t.Run(tc.name, func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- resp := &http.Response{
- StatusCode: tc.statusCode,
- Header: http.Header{
- "Content-Type": []string{"application/json"},
- "x-request-id": []string{"rid-failover"},
- },
- Body: io.NopCloser(strings.NewReader(tc.body)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- repo := &openAIPassthroughFailoverRepo{}
- rateSvc := &RateLimitService{
- accountRepo: repo,
- cfg: &config.Config{
- RateLimit: config.RateLimitConfig{OverloadCooldownMinutes: 10},
- },
- }
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- rateLimitService: rateSvc,
- }
- account := newAccount(tc.accountType)
- start := time.Now()
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.Error(t, err)
- var failoverErr *UpstreamFailoverError
- if tc.expectFailover {
- require.ErrorAs(t, err, &failoverErr)
- require.Equal(t, tc.statusCode, failoverErr.StatusCode)
- require.False(t, c.Writer.Written(), "retryable passthrough 错误应返回 failover 错误给上层换号,而不是直接向客户端写响应")
- } else {
- require.False(t, errors.As(err, &failoverErr))
- require.True(t, c.Writer.Written(), "非 failover 的 passthrough http 错误应直接写回客户端")
- require.Equal(t, tc.statusCode, rec.Code)
- }
- v, ok := c.Get(OpsUpstreamErrorsKey)
- require.True(t, ok)
- arr, ok := v.([]*OpsUpstreamErrorEvent)
- require.True(t, ok)
- require.NotEmpty(t, arr)
- require.True(t, arr[len(arr)-1].Passthrough)
- if tc.expectFailover {
- require.Equal(t, "failover", arr[len(arr)-1].Kind)
- } else {
- require.Equal(t, "http_error", arr[len(arr)-1].Kind)
- }
- require.Equal(t, tc.statusCode, arr[len(arr)-1].UpstreamStatusCode)
- tc.assertRepo(t, repo, start)
- })
- }
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_Transient5xxTriggersFailover(t *testing.T) {
- gin.SetMode(gin.TestMode)
- requestBody := []byte(`{"model":"gpt-5.2","stream":false,"input":"hello"}`)
- for _, statusCode := range []int{
- http.StatusInternalServerError,
- http.StatusBadGateway,
- http.StatusServiceUnavailable,
- http.StatusGatewayTimeout,
- 520, 521, 522, 523, 524,
- } {
- t.Run(fmt.Sprintf("status_%d", statusCode), func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- upstreamBody := fmt.Sprintf(`{"error":{"message":"temporary upstream failure","status":%d}}`, statusCode)
- body := &passthroughCloseTrackingReadCloser{Reader: strings.NewReader(upstreamBody)}
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: statusCode,
- Header: http.Header{
- "Content-Type": []string{"application/json"},
- "X-Request-Id": []string{"rid-api-key-5xx"},
- },
- Body: body,
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 124,
- Name: "api-key-transient-5xx",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "base_url": "https://api.example.test",
- },
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, requestBody)
- require.Nil(t, result, "failed attempts must not report usage or success metadata")
- var failoverErr *UpstreamFailoverError
- require.ErrorAs(t, err, &failoverErr)
- require.Equal(t, statusCode, failoverErr.StatusCode)
- require.JSONEq(t, upstreamBody, string(failoverErr.ResponseBody))
- require.Equal(t, "rid-api-key-5xx", failoverErr.ResponseHeaders.Get("x-request-id"))
- require.False(t, c.Writer.Written(), "failover must happen before downstream output is committed")
- require.True(t, body.closed, "the failed upstream response body must be closed")
- require.Equal(t, requestBody, upstream.lastBody, "the request body remains available for the outer account retry")
- value, ok := c.Get(OpsUpstreamErrorsKey)
- require.True(t, ok)
- events, ok := value.([]*OpsUpstreamErrorEvent)
- require.True(t, ok)
- require.NotEmpty(t, events)
- require.Equal(t, "failover", events[len(events)-1].Kind)
- require.Equal(t, account.ID, events[len(events)-1].AccountID)
- })
- }
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_ContextWindow502DoesNotFailover(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- const upstreamBody = `{"error":{"message":"Your input exceeds the context window of this model. Please adjust your input and try again.","type":"upstream_error"}}`
- body := &passthroughCloseTrackingReadCloser{Reader: strings.NewReader(upstreamBody)}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadGateway,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: body,
- }},
- }
- account := &Account{
- ID: 127, Platform: PlatformOpenAI, Type: AccountTypeAPIKey, Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test", "base_url": "https://api.example.test"},
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- }
- result, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"gpt-5.2","input":"hello"}`))
- require.Nil(t, result)
- require.Error(t, err)
- var failoverErr *UpstreamFailoverError
- require.False(t, errors.As(err, &failoverErr), "context-window errors are deterministic request failures")
- require.True(t, c.Writer.Written())
- require.Equal(t, http.StatusBadGateway, rec.Code)
- require.Contains(t, rec.Body.String(), "exceeds the context window")
- require.True(t, body.closed)
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_PoolModeConfigured5xxRetriesSameAccount(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusBadGateway,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(`{"error":{"message":"temporary upstream failure"}}`)),
- }},
- }
- account := &Account{
- ID: 128, Platform: PlatformOpenAI, Type: AccountTypeAPIKey, Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "base_url": "https://api.example.test",
- "pool_mode": true,
- "pool_mode_retry_status_codes": []any{float64(http.StatusBadGateway)},
- },
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- }
- _, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"gpt-5.2","input":"hello"}`))
- var failoverErr *UpstreamFailoverError
- require.ErrorAs(t, err, &failoverErr)
- require.True(t, failoverErr.RetryableOnSameAccount)
- require.False(t, c.Writer.Written())
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_PoolModeAuthErrorsTriggerFailover(t *testing.T) {
- gin.SetMode(gin.TestMode)
- tests := []struct {
- name string
- statusCode int
- credentials map[string]any
- }{
- {
- name: "configured_401",
- statusCode: http.StatusUnauthorized,
- credentials: map[string]any{
- "pool_mode_retry_status_codes": []any{float64(http.StatusUnauthorized)},
- },
- },
- {
- name: "default_403",
- statusCode: http.StatusForbidden,
- credentials: map[string]any{},
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- upstreamBody := `{"error":{"message":"upstream credential rejected"}}`
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- rateLimitService: NewRateLimitService(transientCooldownAccountRepo{}, nil, &config.Config{}, nil, nil),
- httpUpstream: &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: tt.statusCode,
- Header: http.Header{"Content-Type": []string{"application/json"}},
- Body: io.NopCloser(strings.NewReader(upstreamBody)),
- }},
- }
- credentials := map[string]any{
- "api_key": "sk-test",
- "base_url": "https://api.example.test",
- "pool_mode": true,
- }
- for key, value := range tt.credentials {
- credentials[key] = value
- }
- account := &Account{
- ID: 129, Platform: PlatformOpenAI, Type: AccountTypeAPIKey, Concurrency: 1,
- Credentials: credentials,
- Extra: map[string]any{"openai_passthrough": true}, Status: StatusActive, Schedulable: true,
- }
- _, err := svc.Forward(context.Background(), c, account, []byte(`{"model":"gpt-5.2","input":"hello"}`))
- var failoverErr *UpstreamFailoverError
- require.ErrorAs(t, err, &failoverErr)
- require.Equal(t, tt.statusCode, failoverErr.StatusCode)
- require.True(t, failoverErr.RetryableOnSameAccount)
- require.False(t, c.Writer.Written(), "pool-mode auth failure must fail over before committing a response")
- require.False(t, IsResponseCommitted(c))
- })
- }
- }
- func TestOpenAIGatewayService_OpenAIPassthrough_CompactNetworkErrorsTriggerFailover(t *testing.T) {
- gin.SetMode(gin.TestMode)
- tests := []struct {
- name string
- resp *http.Response
- err error
- expectFailover bool
- }{
- {
- name: "request_error",
- err: errors.New("stream disconnected before completion"),
- expectFailover: true,
- },
- {
- name: "read_error",
- resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid-compact"}},
- Body: passthroughErrReadCloser{err: io.ErrUnexpectedEOF},
- },
- expectFailover: false,
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses/compact", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- upstream := &httpUpstreamRecorder{resp: tt.resp, err: tt.err}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- body := []byte(`{"model":"gpt-5.5","instructions":"local-test-instructions","input":[{"type":"text","text":"compact me"}]}`)
- _, err := svc.Forward(context.Background(), c, account, body)
- require.Error(t, err)
- var failoverErr *UpstreamFailoverError
- if tt.expectFailover {
- require.ErrorAs(t, err, &failoverErr)
- require.Equal(t, http.StatusBadGateway, failoverErr.StatusCode)
- require.False(t, c.Writer.Written(), "compact 网络错误应交给外层 failover,而不是直接写回客户端")
- } else {
- require.False(t, errors.As(err, &failoverErr))
- require.ErrorIs(t, err, io.ErrUnexpectedEOF)
- require.False(t, c.Writer.Written())
- }
- })
- }
- }
- func TestOpenAIGatewayService_OAuthPassthrough_NonCodexUAFallbackToCodexUA(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- // Non-Codex UA
- c.Request.Header.Set("User-Agent", "curl/8.0")
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- require.Equal(t, false, gjson.GetBytes(upstream.lastBody, "store").Bool())
- require.Equal(t, true, gjson.GetBytes(upstream.lastBody, "stream").Bool())
- require.Equal(t, codexCLIUserAgent, upstream.lastReq.Header.Get("User-Agent"))
- }
- // 透传模式的 OAuth 与非透传一致:官方客户端身份同样被强制统一为网关规范身份,
- // originator 与 UA 首段天然配套,不会出现历史上 originator/UA 错配被上游 404 的形态
- // (issue #3901)。
- func TestOpenAIGatewayService_OAuthPassthrough_OfficialIdentityUnified(t *testing.T) {
- gin.SetMode(gin.TestMode)
- const tuiUA = "codex_vscode/0.140.2 (Mac OS X 14.0; arm64) vscode (codex_vscode; 0.140.2)"
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", tuiUA)
- // 客户端携带错配的 originator,也必须按最终 UA 重配。
- c.Request.Header.Set("originator", "codex_cli_rs")
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, codexCLIUserAgent, upstream.lastReq.Header.Get("User-Agent"))
- require.Equal(t, openai.CodexDefaultOriginator, upstream.lastReq.Header.Get("originator"))
- require.Equal(t, codexCLIVersion, upstream.lastReq.Header.Get("version"))
- }
- // 透传模式下真实 TUI 客户端的身份同样被统一:被优先降载的身份不会带到上游。
- func TestOpenAIGatewayService_OAuthPassthrough_CodexTuiIdentityUnified(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex-tui/0.140.2 (Mac OS X 14.0; arm64) iTerm (codex-tui; 0.140.2)")
- c.Request.Header.Set("originator", "codex-tui")
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, codexCLIUserAgent, upstream.lastReq.Header.Get("User-Agent"))
- require.Equal(t, openai.CodexDefaultOriginator, upstream.lastReq.Header.Get("originator"))
- require.Equal(t, codexCLIVersion, upstream.lastReq.Header.Get("version"))
- }
- func TestOpenAIGatewayService_CodexFingerprintHTTPTransformedHeaderBodyParityAndDefaultCacheKey(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- c.Request.Header.Set("originator", "codex_cli_rs")
- c.Request.Header.Set("session-id", "header-session")
- c.Request.Header.Set("x-codex-turn-metadata", `{"installation_id":"header-install","session_id":"header-session","thread_id":"header-thread","turn_id":"header-turn","window_id":"header-window","sandbox":"seatbelt"}`)
- body := []byte(`{"model":"gpt-5.2","stream":false,"prompt_cache_key":"body-session","client_metadata":{"session_id":"body-session","x-codex-turn-metadata":"{\"installation_id\":\"body-install\",\"session_id\":\"body-session\",\"thread_id\":\"body-thread\",\"turn_id\":\"body-turn\",\"window_id\":\"body-window\",\"sandbox\":\"seatbelt\"}"},"input":[{"type":"message","role":"user","content":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- toolCorrector: NewCodexToolCorrector(),
- }
- account := newTestOAuthAccount(4401, map[string]any{codexFingerprintModeExtraKey: "session"})
- account.Name = "oauth-transformed"
- account.Status = StatusActive
- account.Schedulable = true
- account.Concurrency = 1
- account.Credentials = map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"}
- _, err := svc.Forward(context.Background(), c, account, body)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- seed, ok := codexFingerprintSeed(account.Extra)
- require.True(t, ok)
- wantInstall := resolveConvergedInstallationID(account, seed)
- wantSession := resolveConvergedSessionID(seed)
- wantThread := resolveConvergedThreadID(seed, "header-session")
- require.Equal(t, wantInstall, upstream.lastReq.Header.Get("x-codex-installation-id"))
- require.Equal(t, wantSession, upstream.lastReq.Header.Get("session-id"))
- require.Equal(t, wantSession, upstream.lastReq.Header.Get("session_id"))
- require.Equal(t, wantThread, upstream.lastReq.Header.Get("thread-id"))
- require.Equal(t, wantThread, upstream.lastReq.Header.Get("x-client-request-id"))
- require.Equal(t, wantThread+":0", upstream.lastReq.Header.Get("x-codex-window-id"))
- require.Equal(t, wantSession, gjson.GetBytes(upstream.lastBody, "prompt_cache_key").String())
- require.Equal(t, wantInstall, gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-installation-id").String())
- require.Equal(t, wantSession, gjson.GetBytes(upstream.lastBody, "client_metadata.session_id").String())
- require.Equal(t, wantThread, gjson.GetBytes(upstream.lastBody, "client_metadata.thread_id").String())
- require.Equal(t, wantThread+":0", gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-window-id").String())
- bodyTurnMetadata := gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-turn-metadata").String()
- headerTurnMetadata := upstream.lastReq.Header.Get("x-codex-turn-metadata")
- require.Equal(t, wantSession, gjson.Get(bodyTurnMetadata, "session_id").String())
- require.Equal(t, wantSession, gjson.Get(headerTurnMetadata, "session_id").String())
- require.Equal(t, gjson.Get(bodyTurnMetadata, "turn_id").String(), gjson.Get(headerTurnMetadata, "turn_id").String())
- }
- func TestOpenAIGatewayService_CodexFingerprintHTTPRawPassthroughHeaderBodyParityAndDefaultCacheKey(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- c.Request.Header.Set("originator", "codex_cli_rs")
- c.Request.Header.Set("session-id", "header-session")
- c.Request.Header.Set("x-codex-turn-metadata", `{"installation_id":"header-install","session_id":"header-session","thread_id":"header-thread","turn_id":"header-turn","window_id":"header-window","sandbox":"seatbelt"}`)
- body := []byte(`{"model":"gpt-5.6-sol","stream":false,"prompt_cache_key":"body-session","client_metadata":{"session_id":"body-session","x-codex-turn-metadata":"{\"installation_id\":\"body-install\",\"session_id\":\"body-session\",\"thread_id\":\"body-thread\",\"turn_id\":\"body-turn\",\"window_id\":\"body-window\",\"sandbox\":\"seatbelt\"}"},"input":[{"type":"message","role":"user","content":"hi"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- }
- account := newTestOAuthAccount(4402, map[string]any{
- codexFingerprintModeExtraKey: "session",
- "openai_oauth_passthrough": true,
- })
- account.Name = "oauth-raw"
- account.Status = StatusActive
- account.Schedulable = true
- account.Concurrency = 1
- account.Credentials = map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"}
- _, err := svc.Forward(context.Background(), c, account, body)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- seed, ok := codexFingerprintSeed(account.Extra)
- require.True(t, ok)
- wantInstall := resolveConvergedInstallationID(account, seed)
- wantSession := resolveConvergedSessionID(seed)
- wantThread := resolveConvergedThreadID(seed, "header-session")
- require.Equal(t, wantInstall, upstream.lastReq.Header.Get("x-codex-installation-id"))
- require.Equal(t, wantSession, upstream.lastReq.Header.Get("session-id"))
- require.Equal(t, wantSession, upstream.lastReq.Header.Get("session_id"))
- require.Equal(t, wantThread, upstream.lastReq.Header.Get("thread-id"))
- require.Equal(t, wantThread, upstream.lastReq.Header.Get("x-client-request-id"))
- require.Equal(t, wantThread+":0", upstream.lastReq.Header.Get("x-codex-window-id"))
- require.Equal(t, wantSession, gjson.GetBytes(upstream.lastBody, "prompt_cache_key").String())
- require.Equal(t, wantInstall, gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-installation-id").String())
- require.Equal(t, wantSession, gjson.GetBytes(upstream.lastBody, "client_metadata.session_id").String())
- require.Equal(t, wantThread, gjson.GetBytes(upstream.lastBody, "client_metadata.thread_id").String())
- require.Equal(t, wantThread+":0", gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-window-id").String())
- bodyTurnMetadata := gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-turn-metadata").String()
- headerTurnMetadata := upstream.lastReq.Header.Get("x-codex-turn-metadata")
- require.Equal(t, wantSession, gjson.Get(bodyTurnMetadata, "session_id").String())
- require.Equal(t, wantSession, gjson.Get(headerTurnMetadata, "session_id").String())
- require.Equal(t, gjson.Get(bodyTurnMetadata, "turn_id").String(), gjson.Get(headerTurnMetadata, "turn_id").String())
- }
- func TestOpenAIGatewayService_CodexFingerprintCompactDoesNotRewriteBodyCacheKeyOrMetadata(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses/compact", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- c.Request.Header.Set("originator", "codex_cli_rs")
- c.Request.Header.Set("session-id", "header-session")
- body := []byte(`{"model":"gpt-5.4","stream":false,"prompt_cache_key":"body-session","client_metadata":{"session_id":"body-session"},"input":[{"type":"message","role":"user","content":"compress"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(compactProbeSSESuccessBody)),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- toolCorrector: NewCodexToolCorrector(),
- }
- account := newTestOAuthAccount(4403, map[string]any{codexFingerprintModeExtraKey: "session"})
- account.Name = "oauth-compact"
- account.Status = StatusActive
- account.Schedulable = true
- account.Concurrency = 1
- account.Credentials = map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"}
- staleIDs := resolveCodexFingerprintIDs(account, "stale-session", codexFingerprintSession)
- require.NotNil(t, staleIDs)
- stageCodexFingerprintIDs(c, staleIDs)
- _, err := svc.Forward(context.Background(), c, account, body)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- seed, ok := codexFingerprintSeed(account.Extra)
- require.True(t, ok)
- require.NotEqual(t, resolveConvergedSessionID(seed), gjson.GetBytes(upstream.lastBody, "prompt_cache_key").String())
- require.Equal(t, "body-session", gjson.GetBytes(upstream.lastBody, "prompt_cache_key").String())
- require.Equal(t, "body-session", gjson.GetBytes(upstream.lastBody, "client_metadata.session_id").String())
- require.False(t, gjson.GetBytes(upstream.lastBody, "client_metadata.x-codex-installation-id").Exists())
- require.Empty(t, upstream.lastReq.Header.Get("x-codex-window-id"))
- }
- func TestOpenAIGatewayService_CodexFingerprintMessagesBridgeDoesNotInjectBodyPromptCacheKey(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.144.1")
- c.Request.Header.Set("originator", "codex_cli_rs")
- c.Request.Header.Set("session-id", "header-session")
- body := []byte(`{"model":"gpt-5.5","stream":true,"prompt_cache_key":"anthropic-metadata-session-1","client_metadata":{"session_id":"anthropic-metadata-session-1"},"input":[{"type":"message","role":"developer","content":[{"type":"input_text","text":"` + openAICompatClaudeCodeTodoGuardMarker + `"}]},{"type":"message","role":"user","content":"hello"}]}`)
- upstream := &httpUpstreamRecorder{resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{},
- httpUpstream: upstream,
- toolCorrector: NewCodexToolCorrector(),
- }
- account := newTestOAuthAccount(4404, map[string]any{codexFingerprintModeExtraKey: "session"})
- account.Name = "oauth-messages-bridge"
- account.Status = StatusActive
- account.Schedulable = true
- account.Concurrency = 1
- account.Credentials = map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"}
- _, err := svc.Forward(context.Background(), c, account, body)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- seed, ok := codexFingerprintSeed(account.Extra)
- require.True(t, ok)
- wantSession := resolveConvergedSessionID(seed)
- require.False(t, gjson.GetBytes(upstream.lastBody, "prompt_cache_key").Exists())
- require.Equal(t, wantSession, gjson.GetBytes(upstream.lastBody, "client_metadata.session_id").String())
- require.Equal(t, wantSession, upstream.lastReq.Header.Get("session_id"))
- }
- func TestOpenAIGatewayService_CodexCLIOnly_RejectsNonCodexClient(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "curl/8.0")
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true, "codex_cli_only": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.Error(t, err)
- require.Equal(t, http.StatusForbidden, rec.Code)
- require.Contains(t, rec.Body.String(), "Codex official clients")
- }
- func TestOpenAIGatewayService_CodexCLIOnly_AllowOfficialClientFamilies(t *testing.T) {
- gin.SetMode(gin.TestMode)
- tests := []struct {
- name string
- ua string
- originator string
- }{
- {name: "codex_cli_rs", ua: "codex_cli_rs/0.99.0", originator: ""},
- {name: "codex_vscode", ua: "codex_vscode/1.0.0", originator: ""},
- {name: "codex_app", ua: "codex_app/2.1.0", originator: ""},
- // req②:codex_cli_only 下 UA 须能解析出引擎版本;originator 命中路径用可解析的非官方前缀 UA。
- {name: "originator_codex_chatgpt_desktop", ua: "myterm/0.141.0", originator: "codex_chatgpt_desktop"},
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", tt.ua)
- if tt.originator != "" {
- c.Request.Header.Set("originator", tt.originator)
- }
- // 引擎指纹头:真实官方客户端必带。本测试用 nil settingService 构造 gateway,
- // detectCodexClientRestriction 会兜底默认种子指纹信号(只勾 x-codex-),与生产默认策略一致,
- // 故官方家族也须携带 x-codex-* 才能过门(对齐 TestDetect_EngineFingerprintSignals)。
- c.Request.Header.Set("x-codex-window-id", "1")
- inputBody := []byte(`{"model":"gpt-5.2","stream":false,"store":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true, "codex_cli_only": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, inputBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- })
- }
- }
- func TestOpenAIGatewayService_OAuthPassthrough_StreamingSetsFirstTokenMs(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"service_tier":"fast","input":[{"type":"text","text":"hi"}]}`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_text.delta","delta":"h"}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- start := time.Now()
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- // sanity: duration after start
- require.GreaterOrEqual(t, time.Since(start), time.Duration(0))
- require.NotNil(t, result.FirstTokenMs)
- require.GreaterOrEqual(t, *result.FirstTokenMs, 0)
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, "priority", *result.ServiceTier)
- }
- func TestOpenAIGatewayService_OAuthPassthrough_StreamClientDisconnectStillCollectsUsage(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- // 首次写入成功,后续写入失败,模拟客户端中途断开。
- c.Writer = &failingGinWriter{ResponseWriter: c.Writer, failAfter: 1}
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- upstreamSSE := strings.Join([]string{
- `data: {"type":"response.output_text.delta","delta":"h"}`,
- "",
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":11,"output_tokens":7,"input_tokens_details":{"cached_tokens":3}}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n")
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(upstreamSSE)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 123,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.True(t, result.Stream)
- require.NotNil(t, result.FirstTokenMs)
- require.Equal(t, 11, result.Usage.InputTokens)
- require.Equal(t, 7, result.Usage.OutputTokens)
- require.Equal(t, 3, result.Usage.CacheReadInputTokens)
- }
- func TestOpenAIGatewayService_APIKeyPassthrough_PreservesBodyAndUsesResponsesEndpoint(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "curl/8.0")
- c.Request.Header.Set("X-Test", "keep")
- c.Request.Header.Set("x-codex-beta-features", "remote_compaction_v2")
- c.Request.Header.Set("X-Codex-Window-ID", "window-passthrough")
- c.Request.Header.Set("X-Codex-Installation-ID", "installation-passthrough")
- originalBody := []byte(`{"model":"gpt-5.2","stream":false,"service_tier":"flex","max_output_tokens":128,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"application/json"}, "x-request-id": []string{"rid"}},
- Body: io.NopCloser(strings.NewReader(`{"output":[],"usage":{"input_tokens":1,"output_tokens":1,"input_tokens_details":{"cached_tokens":0}}}`)),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 456,
- Name: "apikey-acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-api-key",
- "base_url": "https://api.openai.com",
- "model_mapping": map[string]any{"gpt-5.2": "gpt-5.2-account"},
- },
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- result, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, result)
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, "flex", *result.ServiceTier)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, originalBody, upstream.lastBody)
- require.False(t, gjson.GetBytes(upstream.lastBody, "instructions").Exists())
- require.Equal(t, "https://api.openai.com/v1/responses", upstream.lastReq.URL.String())
- require.Equal(t, "Bearer sk-api-key", upstream.lastReq.Header.Get("Authorization"))
- require.Equal(t, "curl/8.0", upstream.lastReq.Header.Get("User-Agent"))
- require.Equal(t, "remote_compaction_v2", upstream.lastReq.Header.Get("x-codex-beta-features"))
- require.Equal(t, "window-passthrough", upstream.lastReq.Header.Get("X-Codex-Window-ID"))
- require.Equal(t, "installation-passthrough", upstream.lastReq.Header.Get("X-Codex-Installation-ID"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Test"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_WarnOnTimeoutHeadersForStream(t *testing.T) {
- gin.SetMode(gin.TestMode)
- logSink, restore := captureStructuredLog(t)
- defer restore()
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- c.Request.Header.Set("x-stainless-timeout", "10000")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "X-Request-Id": []string{"rid-timeout"}},
- Body: io.NopCloser(strings.NewReader("data: [DONE]\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 321,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.True(t, logSink.ContainsMessage("检测到超时相关请求头,将按配置过滤以降低断流风险"))
- require.True(t, logSink.ContainsFieldValue("timeout_headers", "x-stainless-timeout=10000"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_InfoWhenStreamEndsWithoutDone(t *testing.T) {
- gin.SetMode(gin.TestMode)
- logSink, restore := captureStructuredLog(t)
- defer restore()
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- // 注意:刻意不发送 [DONE],模拟上游中途断流。
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "X-Request-Id": []string{"rid-truncate"}},
- Body: io.NopCloser(strings.NewReader("data: {\"type\":\"response.output_text.delta\",\"delta\":\"h\"}\n\n")),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 654,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.EqualError(t, err, "stream usage incomplete: missing terminal event")
- require.True(t, logSink.ContainsMessage("上游流在未收到 [DONE] 时结束,疑似断流"))
- require.True(t, logSink.ContainsMessageAtLevel("上游流在未收到 [DONE] 时结束,疑似断流", "info"))
- require.True(t, logSink.ContainsFieldValue("upstream_request_id", "rid-truncate"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_DefaultFiltersTimeoutHeaders(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- c.Request.Header.Set("x-stainless-timeout", "120000")
- c.Request.Header.Set("X-Test", "keep")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "X-Request-Id": []string{"rid-filter-default"}},
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":1,"output_tokens":1,"input_tokens_details":{"cached_tokens":0}}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n"))),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{ForceCodexCLI: false}},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 111,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- require.Empty(t, upstream.lastReq.Header.Get("x-stainless-timeout"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Test"))
- }
- func TestOpenAIGatewayService_OAuthPassthrough_AllowTimeoutHeadersWhenConfigured(t *testing.T) {
- gin.SetMode(gin.TestMode)
- rec := httptest.NewRecorder()
- c, _ := gin.CreateTestContext(rec)
- c.Request = httptest.NewRequest(http.MethodPost, "/v1/responses", bytes.NewReader(nil))
- c.Request.Header.Set("User-Agent", "codex_cli_rs/0.1.0")
- c.Request.Header.Set("x-stainless-timeout", "120000")
- c.Request.Header.Set("X-Test", "keep")
- originalBody := []byte(`{"model":"gpt-5.2","stream":true,"input":[{"type":"text","text":"hi"}]}`)
- resp := &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "X-Request-Id": []string{"rid-filter-allow"}},
- Body: io.NopCloser(strings.NewReader(strings.Join([]string{
- `data: {"type":"response.completed","response":{"usage":{"input_tokens":1,"output_tokens":1,"input_tokens_details":{"cached_tokens":0}}}}`,
- "",
- "data: [DONE]",
- "",
- }, "\n"))),
- }
- upstream := &httpUpstreamRecorder{resp: resp}
- svc := &OpenAIGatewayService{
- cfg: &config.Config{Gateway: config.GatewayConfig{
- ForceCodexCLI: false,
- OpenAIPassthroughAllowTimeoutHeaders: true,
- }},
- httpUpstream: upstream,
- }
- account := &Account{
- ID: 222,
- Name: "acc",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Concurrency: 1,
- Credentials: map[string]any{"access_token": "oauth-token", "chatgpt_account_id": "chatgpt-acc"},
- Extra: map[string]any{"openai_passthrough": true},
- Status: StatusActive,
- Schedulable: true,
- RateMultiplier: f64p(1),
- }
- _, err := svc.Forward(context.Background(), c, account, originalBody)
- require.NoError(t, err)
- require.NotNil(t, upstream.lastReq)
- require.Equal(t, "120000", upstream.lastReq.Header.Get("x-stainless-timeout"))
- require.Empty(t, upstream.lastReq.Header.Get("X-Test"))
- }
|