| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617161816191620162116221623162416251626162716281629163016311632163316341635163616371638163916401641164216431644164516461647164816491650165116521653165416551656165716581659166016611662166316641665166616671668166916701671167216731674167516761677167816791680168116821683168416851686168716881689169016911692169316941695169616971698169917001701170217031704170517061707170817091710171117121713171417151716171717181719172017211722172317241725172617271728172917301731173217331734173517361737173817391740174117421743174417451746174717481749175017511752175317541755175617571758175917601761176217631764176517661767176817691770177117721773177417751776177717781779178017811782178317841785178617871788178917901791179217931794179517961797179817991800180118021803180418051806180718081809181018111812181318141815181618171818181918201821182218231824182518261827182818291830183118321833183418351836183718381839184018411842184318441845184618471848184918501851185218531854185518561857185818591860186118621863186418651866186718681869187018711872187318741875187618771878187918801881188218831884188518861887188818891890189118921893189418951896189718981899190019011902190319041905190619071908190919101911191219131914191519161917191819191920192119221923192419251926192719281929193019311932193319341935193619371938193919401941194219431944194519461947194819491950195119521953195419551956195719581959196019611962196319641965196619671968196919701971197219731974197519761977197819791980198119821983198419851986198719881989199019911992199319941995199619971998199920002001200220032004200520062007200820092010201120122013201420152016201720182019202020212022202320242025202620272028202920302031203220332034203520362037203820392040204120422043204420452046204720482049205020512052205320542055205620572058205920602061206220632064206520662067206820692070207120722073207420752076207720782079208020812082208320842085208620872088208920902091209220932094209520962097209820992100210121022103210421052106210721082109211021112112211321142115211621172118211921202121212221232124212521262127212821292130213121322133213421352136213721382139214021412142214321442145214621472148214921502151215221532154215521562157215821592160216121622163216421652166216721682169217021712172217321742175217621772178217921802181218221832184218521862187218821892190219121922193219421952196219721982199220022012202220322042205220622072208220922102211221222132214221522162217221822192220222122222223222422252226222722282229223022312232223322342235223622372238223922402241224222432244224522462247224822492250225122522253225422552256225722582259226022612262226322642265226622672268226922702271227222732274227522762277227822792280228122822283228422852286228722882289229022912292229322942295229622972298229923002301230223032304230523062307230823092310231123122313231423152316231723182319232023212322232323242325232623272328232923302331233223332334233523362337233823392340234123422343234423452346234723482349235023512352235323542355235623572358235923602361236223632364236523662367236823692370237123722373237423752376237723782379238023812382238323842385238623872388238923902391239223932394239523962397239823992400240124022403240424052406240724082409241024112412241324142415241624172418241924202421242224232424242524262427242824292430243124322433243424352436243724382439244024412442244324442445244624472448244924502451245224532454245524562457245824592460246124622463246424652466246724682469247024712472247324742475247624772478247924802481248224832484248524862487248824892490249124922493249424952496249724982499250025012502250325042505250625072508250925102511251225132514251525162517251825192520252125222523252425252526252725282529253025312532253325342535253625372538253925402541254225432544254525462547254825492550255125522553255425552556255725582559256025612562256325642565256625672568256925702571257225732574257525762577257825792580258125822583258425852586258725882589259025912592259325942595259625972598259926002601260226032604260526062607260826092610261126122613261426152616261726182619262026212622262326242625262626272628262926302631263226332634263526362637263826392640264126422643264426452646264726482649265026512652265326542655265626572658265926602661266226632664266526662667266826692670267126722673267426752676267726782679268026812682268326842685268626872688268926902691269226932694269526962697269826992700270127022703270427052706270727082709271027112712271327142715271627172718271927202721272227232724272527262727272827292730273127322733273427352736273727382739274027412742274327442745274627472748274927502751275227532754275527562757275827592760276127622763276427652766276727682769277027712772277327742775277627772778277927802781278227832784278527862787278827892790279127922793279427952796279727982799280028012802280328042805280628072808280928102811281228132814281528162817281828192820282128222823282428252826282728282829283028312832283328342835283628372838283928402841284228432844284528462847284828492850285128522853285428552856285728582859286028612862286328642865286628672868286928702871287228732874287528762877287828792880288128822883288428852886288728882889289028912892289328942895289628972898289929002901290229032904290529062907290829092910291129122913291429152916291729182919292029212922292329242925292629272928292929302931293229332934293529362937293829392940294129422943294429452946294729482949295029512952295329542955295629572958295929602961296229632964296529662967296829692970297129722973297429752976297729782979298029812982298329842985298629872988298929902991299229932994299529962997299829993000300130023003300430053006300730083009301030113012301330143015301630173018301930203021302230233024302530263027302830293030303130323033303430353036303730383039304030413042304330443045304630473048304930503051305230533054305530563057305830593060306130623063306430653066306730683069307030713072307330743075307630773078307930803081308230833084308530863087308830893090309130923093309430953096309730983099310031013102310331043105310631073108310931103111311231133114311531163117311831193120312131223123312431253126312731283129313031313132313331343135313631373138313931403141314231433144314531463147314831493150315131523153315431553156315731583159316031613162316331643165316631673168316931703171317231733174317531763177317831793180318131823183318431853186318731883189319031913192319331943195319631973198319932003201320232033204320532063207320832093210321132123213321432153216321732183219322032213222322332243225322632273228322932303231323232333234323532363237323832393240324132423243324432453246324732483249325032513252325332543255325632573258325932603261326232633264326532663267326832693270327132723273327432753276327732783279328032813282328332843285328632873288328932903291329232933294329532963297329832993300330133023303330433053306330733083309331033113312331333143315331633173318331933203321332233233324332533263327332833293330333133323333333433353336333733383339334033413342334333443345334633473348334933503351335233533354335533563357335833593360336133623363336433653366336733683369337033713372337333743375337633773378337933803381338233833384338533863387338833893390339133923393339433953396339733983399340034013402340334043405340634073408340934103411341234133414341534163417341834193420342134223423342434253426342734283429343034313432343334343435343634373438343934403441344234433444344534463447344834493450345134523453345434553456345734583459346034613462346334643465346634673468346934703471347234733474347534763477347834793480348134823483348434853486348734883489349034913492349334943495349634973498349935003501350235033504350535063507350835093510351135123513351435153516351735183519352035213522352335243525352635273528352935303531353235333534353535363537353835393540354135423543354435453546354735483549355035513552355335543555355635573558355935603561356235633564356535663567356835693570357135723573357435753576357735783579358035813582358335843585358635873588358935903591359235933594359535963597359835993600360136023603360436053606360736083609361036113612361336143615361636173618361936203621362236233624362536263627362836293630363136323633363436353636363736383639364036413642364336443645364636473648364936503651365236533654365536563657365836593660366136623663366436653666366736683669367036713672367336743675367636773678367936803681368236833684368536863687368836893690369136923693369436953696369736983699370037013702370337043705370637073708370937103711371237133714371537163717371837193720372137223723372437253726372737283729373037313732373337343735373637373738373937403741374237433744374537463747374837493750375137523753375437553756375737583759376037613762376337643765376637673768376937703771377237733774377537763777377837793780378137823783378437853786378737883789379037913792379337943795379637973798379938003801380238033804380538063807380838093810381138123813381438153816381738183819382038213822382338243825382638273828382938303831383238333834383538363837383838393840384138423843384438453846384738483849385038513852385338543855385638573858385938603861386238633864386538663867386838693870387138723873387438753876387738783879388038813882388338843885388638873888388938903891389238933894389538963897389838993900390139023903390439053906390739083909391039113912391339143915391639173918391939203921392239233924392539263927392839293930393139323933393439353936393739383939394039413942394339443945394639473948394939503951395239533954395539563957395839593960396139623963396439653966396739683969397039713972397339743975397639773978397939803981398239833984398539863987398839893990399139923993399439953996399739983999400040014002400340044005400640074008400940104011401240134014401540164017401840194020402140224023402440254026402740284029403040314032403340344035403640374038403940404041404240434044404540464047404840494050405140524053405440554056405740584059406040614062406340644065406640674068406940704071407240734074407540764077407840794080408140824083408440854086408740884089409040914092409340944095409640974098409941004101410241034104410541064107410841094110411141124113411441154116411741184119412041214122412341244125412641274128412941304131413241334134413541364137413841394140414141424143414441454146414741484149415041514152415341544155415641574158415941604161416241634164416541664167416841694170417141724173417441754176417741784179418041814182418341844185418641874188418941904191419241934194419541964197419841994200420142024203420442054206420742084209421042114212421342144215421642174218421942204221422242234224422542264227422842294230423142324233423442354236423742384239424042414242424342444245424642474248424942504251425242534254425542564257425842594260426142624263426442654266426742684269427042714272427342744275427642774278427942804281428242834284428542864287428842894290429142924293429442954296429742984299430043014302430343044305430643074308430943104311431243134314431543164317431843194320432143224323432443254326432743284329433043314332433343344335433643374338433943404341434243434344434543464347434843494350435143524353435443554356435743584359436043614362436343644365436643674368436943704371437243734374437543764377437843794380438143824383438443854386438743884389439043914392439343944395439643974398439944004401440244034404440544064407440844094410441144124413441444154416441744184419442044214422442344244425442644274428442944304431443244334434443544364437443844394440444144424443444444454446444744484449445044514452445344544455445644574458445944604461446244634464446544664467446844694470447144724473447444754476447744784479448044814482448344844485448644874488448944904491449244934494449544964497449844994500450145024503450445054506450745084509451045114512451345144515451645174518451945204521452245234524452545264527452845294530453145324533453445354536453745384539454045414542454345444545454645474548454945504551455245534554455545564557455845594560456145624563456445654566456745684569457045714572457345744575457645774578457945804581458245834584458545864587458845894590459145924593459445954596459745984599460046014602460346044605460646074608460946104611461246134614461546164617461846194620462146224623462446254626462746284629463046314632463346344635463646374638463946404641464246434644464546464647464846494650465146524653465446554656465746584659466046614662466346644665466646674668466946704671467246734674467546764677467846794680468146824683468446854686468746884689469046914692469346944695469646974698469947004701470247034704470547064707470847094710471147124713471447154716471747184719472047214722472347244725472647274728472947304731473247334734473547364737 |
- package service
- import (
- "context"
- "errors"
- "io"
- "net/http"
- "net/http/httptest"
- "strings"
- "sync"
- "testing"
- "time"
- "github.com/Wei-Shaw/sub2api/internal/config"
- coderws "github.com/coder/websocket"
- "github.com/gin-gonic/gin"
- "github.com/stretchr/testify/require"
- "github.com/tidwall/gjson"
- )
- type openAIWSLeaseLossAfterReadConn struct {
- *openAIWSCaptureConn
- cancel context.CancelCauseFunc
- once sync.Once
- }
- func (c *openAIWSLeaseLossAfterReadConn) ReadMessage(ctx context.Context) ([]byte, error) {
- message, err := c.openAIWSCaptureConn.ReadMessage(ctx)
- if err == nil {
- c.once.Do(func() {
- c.cancel(ErrOpenAIWSIngressLeaseLost)
- })
- }
- return message, err
- }
- type openAIWSSingleConnDialer struct {
- conn openAIWSClientConn
- }
- func (d *openAIWSSingleConnDialer) Dial(
- ctx context.Context,
- wsURL string,
- headers http.Header,
- proxyURL string,
- ) (openAIWSClientConn, int, http.Header, error) {
- return d.conn, 0, nil, nil
- }
- func TestOpenAIWSDownstreamWriteContext_CancellationOwnership(t *testing.T) {
- t.Run("pre-canceled ordinary context is canceled before return", func(t *testing.T) {
- controlCtx, cancelControl := context.WithCancelCause(context.Background())
- cancelControl(context.Canceled)
- writeCtx, cancelWrite := newOpenAIWSDownstreamWriteContext(controlCtx, nil, time.Second)
- defer cancelWrite()
- require.ErrorIs(t, writeCtx.Err(), context.Canceled)
- })
- t.Run("lease loss keeps current write alive", func(t *testing.T) {
- lifecycleCtx, cancelLifecycle := context.WithCancelCause(context.Background())
- controlCtx, cancelControl := context.WithCancelCause(lifecycleCtx)
- hooks := &OpenAIWSIngressHooks{ClientLifecycleContext: lifecycleCtx}
- writeCtx, cancelWrite := newOpenAIWSDownstreamWriteContext(controlCtx, hooks, time.Second)
- defer cancelWrite()
- cancelControl(ErrOpenAIWSIngressLeaseLost)
- select {
- case <-writeCtx.Done():
- t.Fatalf("lease loss unexpectedly canceled downstream write: %v", writeCtx.Err())
- case <-time.After(20 * time.Millisecond):
- }
- clientDisconnected := errors.New("client disconnected")
- cancelLifecycle(clientDisconnected)
- <-writeCtx.Done()
- require.ErrorIs(t, context.Cause(writeCtx), clientDisconnected)
- })
- t.Run("ordinary cancellation is direct and preserves cause", func(t *testing.T) {
- lifecycleCtx, cancelLifecycle := context.WithCancelCause(context.Background())
- controlCtx, cancelControl := context.WithCancelCause(lifecycleCtx)
- defer cancelControl(context.Canceled)
- hooks := &OpenAIWSIngressHooks{ClientLifecycleContext: lifecycleCtx}
- writeCtx, cancelWrite := newOpenAIWSDownstreamWriteContext(controlCtx, hooks, time.Second)
- defer cancelWrite()
- serverShutdown := errors.New("server shutdown")
- cancelLifecycle(serverShutdown)
- require.ErrorIs(t, writeCtx.Err(), context.Canceled)
- require.ErrorIs(t, context.Cause(writeCtx), serverShutdown)
- })
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_KeepLeaseAcrossTurns(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.output_item.done","item":{"id":"ig_ingress_1","type":"image_generation_call","status":"generating","result":"iVBORw0KGgoAAAANSUhEUg/+=="}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_ingress_turn_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_ingress_turn_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 114,
- Name: "openai-ingress-session-lease",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- turnTerminalCh := make(chan string, 2)
- hooks := &OpenAIWSIngressHooks{
- AfterTurn: func(_ int, result *OpenAIForwardResult, turnErr error) {
- if turnErr == nil && result != nil {
- turnTerminalCh <- result.UpstreamTerminalEvent
- }
- },
- }
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, hooks)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false}`)
- firstTurnImageEvent := readMessage()
- require.Equal(t, "response.output_item.done", gjson.GetBytes(firstTurnImageEvent, "type").String())
- require.Equal(t, "completed", gjson.GetBytes(firstTurnImageEvent, "item.status").String())
- require.Equal(t, "iVBORw0KGgoAAAANSUhEUg/+==", gjson.GetBytes(firstTurnImageEvent, "item.result").String())
- firstTurnEvent := readMessage()
- require.Equal(t, "response.completed", gjson.GetBytes(firstTurnEvent, "type").String())
- require.Equal(t, "resp_ingress_turn_1", gjson.GetBytes(firstTurnEvent, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_ingress_turn_1"}`)
- secondTurnEvent := readMessage()
- require.Equal(t, "response.completed", gjson.GetBytes(secondTurnEvent, "type").String())
- require.Equal(t, "resp_ingress_turn_2", gjson.GetBytes(secondTurnEvent, "response.id").String())
- require.Equal(t, "response.completed", <-turnTerminalCh, "首轮 turn 应保留成功终态")
- require.Equal(t, "response.completed", <-turnTerminalCh, "第二轮 turn 应保留成功终态")
- _ = clientConn.Close(coderws.StatusNormalClosure, "done")
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- metrics := svc.SnapshotOpenAIWSPoolMetrics()
- require.Equal(t, int64(1), metrics.AcquireTotal, "同一 ingress 会话多 turn 应只获取一次上游 lease")
- require.Equal(t, 1, captureDialer.DialCount(), "同一 ingress 会话应保持同一上游连接")
- require.Len(t, captureConn.writes, 2, "应向同一上游连接发送两轮 response.create")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_LeaseLossSendsRetryClose(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- lifecycleCtx, cancelLifecycle := context.WithCancelCause(context.Background())
- defer cancelLifecycle(context.Canceled)
- controlCtx, cancelControl := context.WithCancelCause(lifecycleCtx)
- upstreamConn := &openAIWSLeaseLossAfterReadConn{
- openAIWSCaptureConn: &openAIWSCaptureConn{events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_lease_loss","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- }},
- cancel: cancelControl,
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(&openAIWSSingleConnDialer{conn: upstreamConn})
- defer pool.Close()
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 118,
- Name: "openai-ingress-lease-loss",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test"},
- Extra: map[string]any{"responses_websockets_v2_enabled": true},
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{CompressionMode: coderws.CompressionContextTakeover})
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(controlCtx)
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancelRead := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancelRead()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(
- controlCtx,
- ginCtx,
- conn,
- account,
- "sk-test",
- firstMessage,
- &OpenAIWSIngressHooks{ClientLifecycleContext: lifecycleCtx},
- )
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- msgType, event, err := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, err)
- require.Equal(t, coderws.MessageText, msgType)
- require.Equal(t, "response.completed", gjson.GetBytes(event, "type").String())
- closeReadCtx, cancelCloseRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, _, err = clientConn.Read(closeReadCtx)
- cancelCloseRead()
- var closeErr coderws.CloseError
- require.ErrorAs(t, err, &closeErr)
- require.Equal(t, coderws.StatusTryAgainLater, closeErr.Code)
- require.Equal(t, "websocket ingress capacity lease lost; please reconnect", closeErr.Reason)
- select {
- case serverErr := <-serverErrCh:
- var clientCloseErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &clientCloseErr)
- require.Equal(t, coderws.StatusTryAgainLater, clientCloseErr.StatusCode())
- require.ErrorIs(t, serverErr, ErrOpenAIWSIngressLeaseLost)
- case <-time.After(3 * time.Second):
- t.Fatal("ingress lease-loss reader did not exit")
- }
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_IdleTimeoutReleasesStoreDisabledSession(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.IngressInterTurnIdleTimeoutSeconds = 1
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_idle_timeout","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- }}
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- defer pool.Close()
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 116,
- Name: "openai-ingress-idle-timeout",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test"},
- Extra: map[string]any{"responses_websockets_v2_enabled": true},
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{CompressionMode: coderws.CompressionContextTakeover})
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() { _ = conn.CloseNow() }()
- readCtx, cancelRead := context.WithTimeout(r.Context(), 3*time.Second)
- _, firstMessage, err := conn.Read(readCtx)
- cancelRead()
- if err != nil {
- serverErrCh <- err
- return
- }
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- ginCtx.Request = r.Clone(r.Context())
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() { _ = clientConn.CloseNow() }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, event, err := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, err)
- require.Equal(t, "response.completed", gjson.GetBytes(event, "type").String())
- closeReadCtx, cancelCloseRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, _, err = clientConn.Read(closeReadCtx)
- cancelCloseRead()
- var clientClose coderws.CloseError
- require.ErrorAs(t, err, &clientClose)
- require.Equal(t, coderws.StatusNormalClosure, clientClose.Code)
- require.Equal(t, "websocket idle timeout", clientClose.Reason)
- select {
- case proxyErr := <-serverErrCh:
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, proxyErr, &closeErr)
- require.Equal(t, coderws.StatusNormalClosure, closeErr.StatusCode())
- require.Equal(t, "websocket idle timeout", closeErr.Reason())
- case <-time.After(4 * time.Second):
- t.Fatal("timed out waiting for idle ingress session to close")
- }
- ap, ok := pool.getAccountPool(account.ID)
- require.True(t, ok)
- ap.mu.Lock()
- require.Empty(t, ap.pinnedConns, "idle close must unpin a store=false session")
- for _, conn := range ap.conns {
- require.False(t, conn.isLeased(), "idle close must release the upstream lease")
- }
- ap.mu.Unlock()
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_FollowupCreateCanOmitModel(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_omit_model_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_omit_model_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 115,
- Name: "openai-ingress-omit-model",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "model_mapping": map[string]any{
- "client-model": "gpt-5.1",
- },
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"client-model","stream":false}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, firstEvent, readErr := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr)
- require.Equal(t, "resp_omit_model_1", gjson.GetBytes(firstEvent, "response.id").String())
- writeCtx, cancelWrite = context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","stream":false,"previous_response_id":"resp_omit_model_1"}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead = context.WithTimeout(context.Background(), 3*time.Second)
- _, secondEvent, readErr := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr)
- require.Equal(t, "resp_omit_model_2", gjson.GetBytes(secondEvent, "response.id").String())
- _ = clientConn.Close(coderws.StatusNormalClosure, "done")
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Len(t, captureConn.writes, 2)
- require.Equal(t, "gpt-5.1", gjson.Get(requestToJSONString(captureConn.writes[0]), "model").String())
- require.Equal(t, "gpt-5.1", gjson.Get(requestToJSONString(captureConn.writes[1]), "model").String())
- require.Equal(t, "resp_omit_model_1", gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_CodexImageBridgeRespectsResponsesLite(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_codex_image_bridge","model":"gpt-5.5","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_codex_image_lite","model":"gpt-5.5","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_codex_image_function","model":"gpt-5.5","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- groupID := int64(3)
- apiKey := &APIKey{
- ID: 1,
- UserID: 1,
- GroupID: &groupID,
- Group: &Group{
- ID: groupID,
- AllowImageGeneration: true,
- },
- }
- account := &Account{
- ID: 31,
- Name: "openai-codex-image-ws",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "test-token",
- },
- Extra: map[string]any{
- "openai_oauth_responses_websockets_v2_enabled": true,
- "codex_image_generation_bridge": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "codex_cli_rs/0.98.0")
- ginCtx.Request = req
- ginCtx.Set("api_key", apiKey)
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "test-token", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.5","stream":false,"input":"draw a cat","parallel_tool_calls":true,"sequence":900719925474099312345}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- msgType, message, err := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, err)
- require.Equal(t, coderws.MessageText, msgType)
- require.Equal(t, "resp_codex_image_bridge", gjson.GetBytes(message, "response.id").String())
- writeCtx, cancelWrite = context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{
- "type":"response.create",
- "model":"gpt-5.5",
- "stream":false,
- "previous_response_id":"resp_codex_image_bridge",
- "reasoning":{"effort":"high"},
- "parallel_tool_calls":true,
- "client_metadata":{"ws_request_header_x_openai_internal_codex_responses_lite":"true"},
- "tools":[{"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent"}]}],
- "input":[
- {"type":"additional_tools","role":"developer","tools":[{"type":"custom","name":"exec","description":"Execute code-mode tools, including image_gen.imagegen."}]},
- {"type":"message","role":"user","content":[{"type":"input_text","text":"draw a cat"}]}
- ],
- "tool_choice":{"type":"namespace","name":"collaboration"}
- }`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead = context.WithTimeout(context.Background(), 3*time.Second)
- msgType, message, err = clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, err)
- require.Equal(t, coderws.MessageText, msgType)
- require.Equal(t, "resp_codex_image_lite", gjson.GetBytes(message, "response.id").String())
- writeCtx, cancelWrite = context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{
- "type":"response.create",
- "model":"gpt-5.5",
- "stream":false,
- "previous_response_id":"resp_codex_image_lite",
- "input":"draw a cat",
- "tools":[{"type":"function","name":"image_gen.imagegen","parameters":{"type":"object"}}]
- }`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead = context.WithTimeout(context.Background(), 3*time.Second)
- msgType, message, err = clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, err)
- require.Equal(t, coderws.MessageText, msgType)
- require.Equal(t, "resp_codex_image_function", gjson.GetBytes(message, "response.id").String())
- writeCtx, cancelWrite = context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{
- "type":"response.create",
- "model":"gpt-5.5",
- "parallel_tool_calls":"false",
- "client_metadata":{"ws_request_header_x_openai_internal_codex_responses_lite":"true"},
- "tools":[{"type":"function","name":"shell"}]
- }`))
- cancelWrite()
- require.NoError(t, err)
- select {
- case serverErr := <-serverErrCh:
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &closeErr)
- require.Equal(t, coderws.StatusPolicyViolation, closeErr.StatusCode())
- require.Contains(t, closeErr.Reason(), "parallel_tool_calls to be a boolean")
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Len(t, captureConn.writes, 3)
- nonLitePayload := requestToJSONString(captureConn.writes[0])
- require.True(t, gjson.Get(nonLitePayload, `tools.#(type=="image_generation")`).Exists())
- require.Equal(t, "png", gjson.Get(nonLitePayload, `tools.#(type=="image_generation").output_format`).String())
- require.Equal(t, "auto", gjson.Get(nonLitePayload, "tool_choice").String())
- require.Contains(t, gjson.Get(nonLitePayload, "instructions").String(), "image_generation")
- require.False(t, gjson.Get(nonLitePayload, "reasoning.context").Exists())
- require.True(t, gjson.Get(nonLitePayload, "parallel_tool_calls").Bool())
- require.Equal(t, "900719925474099312345", gjson.Get(nonLitePayload, "sequence").Raw)
- litePayload := requestToJSONString(captureConn.writes[1])
- require.False(t, gjson.Get(litePayload, `tools.#(type=="image_generation")`).Exists())
- require.NotContains(t, gjson.Get(litePayload, "instructions").String(), "image_generation")
- require.Equal(t, "exec", gjson.Get(litePayload, `input.#(type=="additional_tools").tools.0.name`).String())
- require.Contains(t, gjson.Get(litePayload, `input.#(type=="additional_tools").tools.0.description`).String(), "image_gen.imagegen")
- require.False(t, gjson.Get(litePayload, `tools.#(type=="namespace")`).Exists())
- require.Equal(t, "collaboration", gjson.Get(litePayload, `input.#(type=="additional_tools").tools.1.name`).String())
- require.Equal(t, "namespace", gjson.Get(litePayload, "tool_choice.type").String())
- require.Equal(t, "collaboration", gjson.Get(litePayload, "tool_choice.name").String())
- require.Equal(t, "high", gjson.Get(litePayload, "reasoning.effort").String())
- require.Equal(t, "all_turns", gjson.Get(litePayload, "reasoning.context").String())
- require.True(t, gjson.Get(litePayload, "parallel_tool_calls").Exists())
- require.False(t, gjson.Get(litePayload, "parallel_tool_calls").Bool())
- functionPayload := requestToJSONString(captureConn.writes[2])
- require.True(t, gjson.Get(functionPayload, `tools.#(name=="image_gen.imagegen")`).Exists())
- require.False(t, gjson.Get(functionPayload, `tools.#(type=="image_generation")`).Exists())
- require.False(t, gjson.Get(functionPayload, "tool_choice").Exists())
- require.NotContains(t, gjson.Get(functionPayload, "instructions").String(), codexImageGenerationBridgeMarker)
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_DedicatedModeDoesNotReuseConnAcrossSessions(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeShared
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- upstreamConn1 := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_dedicated_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- upstreamConn2 := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_dedicated_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{upstreamConn1, upstreamConn2},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 441,
- Name: "openai-ingress-dedicated",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "openai_apikey_responses_websockets_v2_mode": OpenAIWSIngressModeDedicated,
- },
- }
- serverErrCh := make(chan error, 2)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- runSingleTurnSession := func(expectedResponseID string) {
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- msgType, event, readErr := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- require.Equal(t, expectedResponseID, gjson.GetBytes(event, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- }
- runSingleTurnSession("resp_dedicated_1")
- runSingleTurnSession("resp_dedicated_2")
- require.Equal(t, 2, dialer.DialCount(), "dedicated 模式下跨客户端会话不应复用上游连接")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PassthroughModeRelaysByCaddyAdapter(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeCtxPool
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- upstreamConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_passthrough_turn_1","model":"gpt-5.1","output":[{"id":"ig_passthrough_1","type":"image_generation_call","status":"generating","result":"final-image"}],"usage":{"input_tokens":2,"output_tokens":3}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: upstreamConn}
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPassthroughDialer: captureDialer,
- }
- account := &Account{
- ID: 452,
- Name: "openai-ingress-passthrough",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "openai_apikey_responses_websockets_v2_mode": OpenAIWSIngressModePassthrough,
- },
- }
- serverErrCh := make(chan error, 1)
- resultCh := make(chan *OpenAIForwardResult, 1)
- hooks := &OpenAIWSIngressHooks{
- AfterTurn: func(_ int, result *OpenAIForwardResult, turnErr error) {
- if turnErr == nil && result != nil {
- resultCh <- result
- }
- },
- }
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, hooks)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false,"service_tier":"fast","reasoning":{"effort":"HIGH"}}`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, event, readErr := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr)
- require.Equal(t, "response.completed", gjson.GetBytes(event, "type").String())
- require.Equal(t, "resp_passthrough_turn_1", gjson.GetBytes(event, "response.id").String())
- require.Equal(t, "completed", gjson.GetBytes(event, "response.output.0.status").String())
- _ = clientConn.Close(coderws.StatusNormalClosure, "done")
- select {
- case serverErr := <-serverErrCh:
- // After normal client close, the server goroutine may receive the close frame
- // as an error — this is expected behavior, not a test failure.
- if serverErr != nil {
- require.Contains(t, serverErr.Error(), "StatusNormalClosure",
- "server error should only be a normal close frame, got: %v", serverErr)
- }
- case <-time.After(5 * time.Second):
- t.Fatal("等待 passthrough websocket 结束超时")
- }
- select {
- case result := <-resultCh:
- require.Equal(t, "resp_passthrough_turn_1", result.RequestID)
- require.True(t, result.OpenAIWSMode)
- require.Equal(t, 2, result.Usage.InputTokens)
- require.Equal(t, 3, result.Usage.OutputTokens)
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, "priority", *result.ServiceTier)
- require.NotNil(t, result.ReasoningEffort)
- require.Equal(t, "high", *result.ReasoningEffort)
- case <-time.After(2 * time.Second):
- t.Fatal("未收到 passthrough turn 结果回调")
- }
- require.Equal(t, 1, captureDialer.DialCount(), "passthrough 模式应直接建立上游 websocket")
- require.Len(t, upstreamConn.writes, 1, "passthrough 模式应透传首条 response.create")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PassthroughBridgeBoundaries(t *testing.T) {
- tests := []struct {
- name string
- payload string
- threshold int64
- wantRelayReject bool
- }{
- {
- name: "small response create",
- payload: `{"type":"response.create","model":"gpt-5.1"}`,
- threshold: 1024,
- },
- {
- name: "continuation response create",
- payload: `{"type":"response.create","previous_response_id":"resp_previous","model":"gpt-5.1"}`,
- threshold: 1,
- },
- {
- name: "other event type",
- payload: `{"type":"session.update","padding":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"}`,
- threshold: 1,
- },
- {
- name: "malformed data",
- payload: `{"type":"response.create","padding":"xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"`,
- threshold: 1,
- wantRelayReject: true,
- },
- {
- name: "duplicate type",
- payload: `{"type":"response.create","type":"response.create","model":"gpt-5.1"}`,
- threshold: 1,
- },
- {
- name: "duplicate previous response id",
- payload: `{"type":"response.create","previous_response_id":null,"previous_response_id":null,"model":"gpt-5.1"}`,
- threshold: 1,
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeCtxPool
- cfg.Gateway.OpenAIWS.HTTPBridgeEnabled = true
- cfg.Gateway.OpenAIWS.HTTPBridgeThresholdBytes = tt.threshold
- upstreamConn := &openAIWSCaptureConn{events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_duplicate_keys","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- }}
- dialer := &openAIWSCaptureDialer{conn: upstreamConn}
- httpUpstream := &httpUpstreamRecorder{}
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: httpUpstream,
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPassthroughDialer: dialer,
- }
- account := &Account{
- ID: 453,
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{"api_key": "sk-test"},
- Extra: map[string]any{
- "openai_apikey_responses_websockets_v2_mode": OpenAIWSIngressModePassthrough,
- },
- }
- errCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, nil)
- if err != nil {
- errCh <- err
- return
- }
- defer func() { _ = conn.CloseNow() }()
- _, firstMessage, err := conn.Read(r.Context())
- if err != nil {
- errCh <- err
- return
- }
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- ginCtx.Request = r
- errCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- clientConn, _, err := coderws.Dial(context.Background(), "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- require.NoError(t, err)
- defer func() { _ = clientConn.CloseNow() }()
- require.NoError(t, clientConn.Write(context.Background(), coderws.MessageText, []byte(tt.payload)))
- _, event, err := clientConn.Read(context.Background())
- if tt.wantRelayReject {
- require.Error(t, err)
- } else {
- require.NoError(t, err)
- require.Equal(t, "resp_duplicate_keys", gjson.GetBytes(event, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- }
- select {
- case proxyErr := <-errCh:
- if tt.wantRelayReject {
- require.Error(t, proxyErr)
- } else if proxyErr != nil {
- require.Contains(t, proxyErr.Error(), "StatusNormalClosure")
- }
- case <-time.After(5 * time.Second):
- t.Fatal("等待 boundary passthrough websocket 结束超时")
- }
- require.Empty(t, httpUpstream.requests, "boundary frame must not use the HTTP bridge")
- if tt.wantRelayReject {
- require.Zero(t, dialer.DialCount())
- require.Empty(t, upstreamConn.writes)
- } else {
- require.Equal(t, 1, dialer.DialCount())
- require.Len(t, upstreamConn.writes, 1)
- }
- })
- }
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PassthroughHeadersUsePromptCacheAndTurnState(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeCtxPool
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- upstreamConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_passthrough_headers","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: upstreamConn}
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPassthroughDialer: captureDialer,
- }
- account := &Account{
- ID: 453,
- Name: "openai-ingress-passthrough-headers",
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "access_token": "oauth-token",
- },
- Extra: map[string]any{
- "openai_oauth_responses_websockets_v2_mode": OpenAIWSIngressModePassthrough,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "codex_cli_rs/0.98.0")
- req.Header.Set(openAIWSTurnStateHeader, "turn-state-1")
- req.Header.Set(openAIWSTurnMetadataHeader, "turn-meta-1")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "oauth-token", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{
- "type":"response.create",
- "model":"gpt-5.1",
- "stream":false,
- "prompt_cache_key":"pcache_passthrough",
- "reasoning":{"effort":"medium","context":"current_turn"},
- "parallel_tool_calls":true,
- "client_metadata":{"ws_request_header_x_openai_internal_codex_responses_lite":"true"},
- "tools":[{"type":"namespace","name":"collaboration","tools":[{"type":"function","name":"spawn_agent"}]}],
- "input":[{"type":"message","role":"user","content":"hello"}],
- "tool_choice":{"type":"namespace","name":"collaboration"}
- }`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, event, readErr := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr)
- require.Equal(t, "resp_passthrough_headers", gjson.GetBytes(event, "response.id").String())
- _ = clientConn.Close(coderws.StatusNormalClosure, "done")
- select {
- case serverErr := <-serverErrCh:
- if serverErr != nil {
- require.Contains(t, serverErr.Error(), "StatusNormalClosure")
- }
- case <-time.After(5 * time.Second):
- t.Fatal("等待 passthrough websocket 结束超时")
- }
- require.Equal(t, isolateOpenAIUpstreamSessionID(0, account, "pcache_passthrough"), captureDialer.lastHeaders.Get("session_id"))
- require.Equal(t, "turn-state-1", captureDialer.lastHeaders.Get(openAIWSTurnStateHeader))
- require.Equal(t, "turn-meta-1", captureDialer.lastHeaders.Get(openAIWSTurnMetadataHeader))
- require.Len(t, upstreamConn.writes, 1)
- forwarded := requestToJSONString(upstreamConn.writes[0])
- require.False(t, gjson.Get(forwarded, `tools.#(type=="namespace")`).Exists())
- require.Equal(t, "collaboration", gjson.Get(forwarded, `input.#(type=="additional_tools").tools.0.name`).String())
- require.Equal(t, "namespace", gjson.Get(forwarded, "tool_choice.type").String())
- require.Equal(t, "collaboration", gjson.Get(forwarded, "tool_choice.name").String())
- require.Equal(t, "medium", gjson.Get(forwarded, "reasoning.effort").String())
- require.Equal(t, "all_turns", gjson.Get(forwarded, "reasoning.context").String())
- require.True(t, gjson.Get(forwarded, "parallel_tool_calls").Exists())
- require.False(t, gjson.Get(forwarded, "parallel_tool_calls").Bool())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_HTTPBridgeModeRelaysHTTPStream(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeCtxPool
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- upstream := &httpUpstreamRecorder{
- resp: &http.Response{
- StatusCode: http.StatusOK,
- Header: http.Header{"Content-Type": []string{"text/event-stream"}, "x-request-id": []string{"rid_bridge_1"}},
- Body: io.NopCloser(strings.NewReader(
- "data: {\"type\":\"response.output_text.delta\",\"delta\":\"hello\"}\n\n" +
- "data: {\"type\":\"response.done\",\"response\":{\"id\":\"resp_http_bridge_1\",\"output\":[{\"id\":\"ig_bridge_1\",\"type\":\"image_generation_call\",\"status\":\"in_progress\",\"result\":\"final-image\"}],\"usage\":{\"input_tokens\":2,\"output_tokens\":1,\"input_tokens_details\":{\"cached_tokens\":1}}}}\n\n" +
- "data: [DONE]\n\n",
- )),
- },
- }
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: upstream,
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- }
- account := &Account{
- ID: 552,
- Name: "openai-ingress-http-bridge",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "openai_apikey_responses_websockets_v2_mode": OpenAIWSIngressModeHTTPBridge,
- },
- }
- serverErrCh := make(chan error, 1)
- resultCh := make(chan *OpenAIForwardResult, 1)
- hooks := &OpenAIWSIngressHooks{
- AfterTurn: func(_ int, result *OpenAIForwardResult, turnErr error) {
- if turnErr == nil && result != nil {
- resultCh <- result
- }
- },
- }
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, hooks)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{
- "type":"response.create","model":"gpt-5.1","stream":false,
- "parallel_tool_calls":true,
- "client_metadata":{"ws_request_header_x_openai_internal_codex_responses_lite":"true"}
- }`))
- cancelWrite()
- require.NoError(t, err)
- readCtx, cancelRead := context.WithTimeout(context.Background(), 3*time.Second)
- _, event1, readErr1 := clientConn.Read(readCtx)
- cancelRead()
- require.NoError(t, readErr1)
- require.Equal(t, "response.output_text.delta", gjson.GetBytes(event1, "type").String())
- require.Equal(t, "hello", gjson.GetBytes(event1, "delta").String())
- readCtx2, cancelRead2 := context.WithTimeout(context.Background(), 3*time.Second)
- _, event2, readErr2 := clientConn.Read(readCtx2)
- cancelRead2()
- require.NoError(t, readErr2)
- require.Equal(t, "response.done", gjson.GetBytes(event2, "type").String())
- require.Equal(t, "resp_http_bridge_1", gjson.GetBytes(event2, "response.id").String())
- require.Equal(t, "completed", gjson.GetBytes(event2, "response.output.0.status").String())
- _ = clientConn.Close(coderws.StatusNormalClosure, "done")
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 http_bridge websocket 结束超时")
- }
- select {
- case result := <-resultCh:
- require.Equal(t, "resp_http_bridge_1", result.RequestID)
- require.True(t, result.OpenAIWSMode)
- require.Equal(t, 2, result.Usage.InputTokens)
- require.Equal(t, 1, result.Usage.OutputTokens)
- require.Equal(t, 1, result.Usage.CacheReadInputTokens)
- require.NotNil(t, result.FirstTokenMs)
- case <-time.After(2 * time.Second):
- t.Fatal("未收到 http_bridge turn 结果回调")
- }
- require.NotNil(t, upstream.lastReq, "http_bridge 模式应调用 HTTP 上游")
- require.Equal(t, "true", upstream.lastReq.Header.Get(responsesLiteHeader))
- require.True(t, gjson.GetBytes(upstream.lastBody, "parallel_tool_calls").Exists())
- require.False(t, gjson.GetBytes(upstream.lastBody, "parallel_tool_calls").Bool())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_ModeOffReturnsPolicyViolation(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.ModeRouterV2Enabled = true
- cfg.Gateway.OpenAIWS.IngressModeDefault = OpenAIWSIngressModeShared
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: newOpenAIWSConnPool(cfg),
- }
- account := &Account{
- ID: 442,
- Name: "openai-ingress-off",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "openai_apikey_responses_websockets_v2_mode": OpenAIWSIngressModeOff,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false}`))
- cancelWrite()
- require.NoError(t, err)
- select {
- case serverErr := <-serverErrCh:
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &closeErr)
- require.Equal(t, coderws.StatusPolicyViolation, closeErr.StatusCode())
- require.Equal(t, "websocket mode is disabled for this account", closeErr.Reason())
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPrevResponseStrictDropToFullCreate(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_preflight_rewrite_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_preflight_rewrite_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 140,
- Name: "openai-ingress-prev-preflight-rewrite",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_preflight_rewrite_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_stale_external","input":[{"type":"input_text","text":"world"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_preflight_rewrite_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount(), "严格增量不成立时应在同一连接内降级为 full create")
- require.Len(t, captureConn.writes, 2)
- secondWrite := requestToJSONString(captureConn.writes[1])
- require.False(t, gjson.Get(secondWrite, "previous_response_id").Exists(), "严格增量不成立时应移除 previous_response_id,改为 full create")
- require.Equal(t, 2, len(gjson.Get(secondWrite, "input").Array()), "严格降级为 full create 时应重放完整 input 上下文")
- require.Equal(t, "hello", gjson.Get(secondWrite, "input.0.text").String())
- require.Equal(t, "world", gjson.Get(secondWrite, "input.1.text").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPrevResponseStrictDropBeforePreflightPingFailReconnects(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 2
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 2
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_drop_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_drop_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 142,
- Name: "openai-ingress-prev-strict-drop-before-ping",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_drop_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_stale_external","input":[{"type":"input_text","text":"world"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_ping_drop_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 严格降级后预检换连超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "严格降级为 full create 后,预检 ping 失败应允许换连")
- require.Equal(t, 1, firstConn.WriteCount(), "首连接在预检失败后不应继续发送第二轮")
- require.GreaterOrEqual(t, firstConn.PingCount(), 1, "第二轮前应执行 preflight ping")
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1)
- secondWrite := requestToJSONString(secondWrites[0])
- require.False(t, gjson.Get(secondWrite, "previous_response_id").Exists(), "严格降级后重试应移除 previous_response_id")
- require.Equal(t, 2, len(gjson.Get(secondWrite, "input").Array()))
- require.Equal(t, "hello", gjson.Get(secondWrite, "input.0.text").String())
- require.Equal(t, "world", gjson.Get(secondWrite, "input.1.text").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreEnabledSkipsStrictPrevResponseEval(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_store_enabled_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_store_enabled_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 143,
- Name: "openai-ingress-store-enabled-skip-strict",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":true}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_store_enabled_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":true,"previous_response_id":"resp_stale_external"}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_store_enabled_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 store=true 场景 websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.Equal(t, "resp_stale_external", gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").String(), "store=true 场景不应触发 store-disabled strict 规则")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPrevResponsePreflightSkipForFunctionCallOutput(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_preflight_skip_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_preflight_skip_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 141,
- Name: "openai-ingress-prev-preflight-skip-fco",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_preflight_skip_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_stale_external","input":[{"type":"function_call_output","call_id":"call_1","output":"ok"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_preflight_skip_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.Equal(t, "resp_stale_external", gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").String(), "function_call_output 场景不应预改写 previous_response_id")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledFunctionCallOutputAutoAttachPreviousResponseID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 143,
- Name: "openai-ingress-fco-auto-prev",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_auto_prev_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"function_call_output","call_id":"call_auto_1","output":"ok"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_auto_prev_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.Equal(t, "resp_auto_prev_1", gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").String(), "function_call_output 缺失 previous_response_id 时应回填上一轮响应 ID")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledToolSearchOutputAutoAttachesPreviousResponseID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_tool_search_prev_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_tool_search_prev_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 145,
- Name: "openai-ingress-tool-search-output-auto-prev",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_tool_search_prev_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"tool_search_output","call_id":"call_search_1","output":"ok"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_tool_search_prev_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- secondWrite := requestToJSONString(captureConn.writes[1])
- require.Equal(t, "resp_tool_search_prev_1", gjson.Get(secondWrite, "previous_response_id").String(), "tool_search_output 缺失 previous_response_id 时应回填上一轮响应 ID")
- require.Equal(t, "tool_search_output", gjson.Get(secondWrite, "input.0.type").String())
- require.Equal(t, "call_search_1", gjson.Get(secondWrite, "input.0.call_id").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledFunctionCallOutputSkipsAutoAttachWhenLastResponseIDMissing(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_skip_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 144,
- Name: "openai-ingress-fco-auto-prev-skip",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "response.completed", gjson.GetBytes(firstTurn, "type").String())
- require.Empty(t, gjson.GetBytes(firstTurn, "response.id").String(), "首轮响应不返回 response.id,模拟无法推导续链锚点")
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"function_call_output","call_id":"call_auto_skip_1","output":"ok"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_auto_prev_skip_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.False(t, gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").Exists(), "上一轮缺失 response.id 时不应自动补齐 previous_response_id")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledFunctionCallOutputSkipsAutoAttachWhenToolCallContextPresent(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_ctx_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_ctx_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{captureConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 114,
- Name: "openai-ingress-tool-context",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_auto_prev_ctx_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"function_call","call_id":"call_ctx_1","name":"shell","arguments":"{}"},{"type":"function_call_output","call_id":"call_ctx_1","output":"ok"},{"type":"message","role":"user","content":[{"type":"input_text","text":"retry"}]}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_auto_prev_ctx_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.False(t, gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").Exists(), "请求已包含 function_call 上下文时不应自动补齐 previous_response_id")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledFunctionCallOutputAutoAttachWhenOnlyItemReferencesPresent(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- captureConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_ref_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_auto_prev_ref_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{captureConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 115,
- Name: "openai-ingress-item-reference",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_auto_prev_ref_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"item_reference","id":"call_ref_1"},{"type":"function_call_output","call_id":"call_ref_1","output":"ok"},{"type":"message","role":"user","content":[{"type":"input_text","text":"retry"}]}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_auto_prev_ref_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, captureDialer.DialCount())
- require.Len(t, captureConn.writes, 2)
- require.Equal(t, "resp_auto_prev_ref_1", gjson.Get(requestToJSONString(captureConn.writes[1]), "previous_response_id").String(), "仅有 item_reference 不足以自包含 function_call_output,应回填上一轮响应 ID")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PreflightPingFailReconnectsBeforeTurn(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 116,
- Name: "openai-ingress-preflight-ping",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_turn_ping_1"}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_ping_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "第二轮 turn 前 ping 失败应触发换连")
- require.Equal(t, 1, firstConn.WriteCount(), "preflight ping 失败后不应继续向旧连接发送第二轮 turn")
- require.GreaterOrEqual(t, firstConn.PingCount(), 1, "第二轮前应对旧连接执行 preflight ping")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledStrictAffinityPreflightPingFailAutoRecoveryReconnects(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 2
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 2
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_strict_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_strict_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 121,
- Name: "openai-ingress-preflight-ping-strict-affinity",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_strict_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_turn_ping_strict_1","input":[{"type":"input_text","text":"world"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_ping_strict_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 严格亲和自动恢复后结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "严格亲和 preflight ping 失败后应自动降级并换连重放")
- require.Equal(t, 1, firstConn.WriteCount(), "preflight ping 失败后不应继续在旧连接写第二轮")
- require.GreaterOrEqual(t, firstConn.PingCount(), 1, "第二轮前应执行 preflight ping")
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1)
- secondWrite := requestToJSONString(secondWrites[0])
- require.False(t, gjson.Get(secondWrite, "previous_response_id").Exists(), "自动恢复重放应移除 previous_response_id")
- require.Equal(t, 2, len(gjson.Get(secondWrite, "input").Array()), "自动恢复重放应使用完整 input 上下文")
- require.Equal(t, "hello", gjson.Get(secondWrite, "input.0.text").String())
- require.Equal(t, "world", gjson.Get(secondWrite, "input.1.text").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPreflightPingFailReplaysFunctionCallOutputWithContext(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 2
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 2
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_replay_ctx_1","model":"gpt-5.1","output":[{"type":"function_call","id":"fc_replay_1","call_id":"call_replay_1","name":"shell","arguments":"{}"}],"usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_replay_ctx_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 128,
- Name: "openai-ingress-preflight-replay-function-output-with-context",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"message","role":"user","content":[{"type":"input_text","text":"call tool"}]}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_replay_ctx_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_turn_ping_replay_ctx_1","input":[{"type":"function_call_output","call_id":"call_replay_1","output":"ok"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_ping_replay_ctx_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket function_call_output 自包含重放后结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "带完整 tool 上下文的 function_call_output 应在 ping 失败后换新连接重放")
- require.Equal(t, 1, firstConn.WriteCount())
- require.GreaterOrEqual(t, firstConn.PingCount(), 1)
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1)
- secondWrite := requestToJSONString(secondWrites[0])
- require.False(t, gjson.Get(secondWrite, "previous_response_id").Exists())
- require.Equal(t, 3, len(gjson.Get(secondWrite, "input").Array()))
- require.Equal(t, "message", gjson.Get(secondWrite, "input.0.type").String())
- require.Equal(t, "function_call", gjson.Get(secondWrite, "input.1.type").String())
- require.Equal(t, "call_replay_1", gjson.Get(secondWrite, "input.1.call_id").String())
- require.Equal(t, "function_call_output", gjson.Get(secondWrite, "input.2.type").String())
- require.Equal(t, "call_replay_1", gjson.Get(secondWrite, "input.2.call_id").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPreflightPingFailClosesWhenFunctionCallOutputNeedsPreviousResponseID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 2
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 2
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_replay_fc_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"error","error":{"type":"invalid_request_error","code":"previous_response_not_found","message":"Previous response not found."}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 129,
- Name: "openai-ingress-preflight-replay-function-output",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"function_call","call_id":"call_other","name":"shell","arguments":"{}"},{"type":"function_call_output","call_id":"call_replay_1","output":"ok"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_replay_fc_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_turn_ping_replay_fc_1","input":[{"type":"function_call_output","call_id":"call_replay_1","output":"ok"}]}`)
- select {
- case serverErr := <-serverErrCh:
- require.Error(t, serverErr)
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &closeErr)
- require.Equal(t, coderws.StatusPolicyViolation, closeErr.StatusCode())
- require.Contains(t, closeErr.Reason(), "upstream continuation connection is unavailable")
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, dialer.DialCount(), "需要 previous_response_id 的 function_call_output 在原连接不可用时不应换新连接重试")
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Empty(t, secondWrites, "不能把旧连接的 previous_response_id 发送到新上游,否则会触发 previous_response_not_found")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledPreflightPingFailClosesWhenReplayHasFunctionCallOutput(t *testing.T) {
- gin.SetMode(gin.TestMode)
- prevPreflightPingIdle := openAIWSIngressPreflightPingIdle
- openAIWSIngressPreflightPingIdle = 0
- defer func() {
- openAIWSIngressPreflightPingIdle = prevPreflightPingIdle
- }()
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 2
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 2
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSPreflightFailConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_ping_replay_only_fc_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"error","error":{"type":"invalid_request_error","message":"No tool call found for function call output with call_id call_replay_1.","param":"input"}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 130,
- Name: "openai-ingress-preflight-replay-only-function-output",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"input":[{"type":"function_call","call_id":"call_other","name":"shell","arguments":"{}"},{"type":"function_call_output","call_id":"call_replay_1","output":"ok"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_ping_replay_only_fc_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"previous_response_id":"resp_turn_ping_replay_only_fc_1","input":[{"type":"input_text","text":"after tool output"}]}`)
- select {
- case serverErr := <-serverErrCh:
- require.Error(t, serverErr)
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &closeErr)
- require.Equal(t, coderws.StatusPolicyViolation, closeErr.StatusCode())
- require.Contains(t, closeErr.Reason(), "upstream continuation connection is unavailable")
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 1, dialer.DialCount(), "replay input 带 function_call_output 时不应换新连接重试")
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Empty(t, secondWrites, "不能把会触发 No tool call found 的重放请求发到新上游")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_WriteFailBeforeDownstreamRetriesOnce(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSWriteFailAfterFirstTurnConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_write_retry_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_write_retry_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 117,
- Name: "openai-ingress-write-retry",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- var hooksMu sync.Mutex
- beforeTurnCalls := make(map[int]int)
- afterTurnCalls := make(map[int]int)
- hooks := &OpenAIWSIngressHooks{
- BeforeTurn: func(turn int) error {
- hooksMu.Lock()
- beforeTurnCalls[turn]++
- hooksMu.Unlock()
- return nil
- },
- AfterTurn: func(turn int, _ *OpenAIForwardResult, _ error) {
- hooksMu.Lock()
- afterTurnCalls[turn]++
- hooksMu.Unlock()
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, hooks)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_write_retry_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_turn_write_retry_1"}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_write_retry_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "第二轮 turn 上游写失败且未写下游时应自动重试并换连")
- hooksMu.Lock()
- beforeTurn1 := beforeTurnCalls[1]
- beforeTurn2 := beforeTurnCalls[2]
- afterTurn1 := afterTurnCalls[1]
- afterTurn2 := afterTurnCalls[2]
- hooksMu.Unlock()
- require.Equal(t, 1, beforeTurn1, "首轮 turn BeforeTurn 应执行一次")
- require.Equal(t, 1, beforeTurn2, "同一 turn 重试不应重复触发 BeforeTurn")
- require.Equal(t, 1, afterTurn1, "首轮 turn AfterTurn 应执行一次")
- require.Equal(t, 1, afterTurn2, "第二轮 turn AfterTurn 应执行一次")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PreviousResponseNotFoundRecoversByDroppingPrevID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.IngressPreviousResponseRecoveryEnabled = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_recover_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"error","error":{"type":"invalid_request_error","code":"previous_response_not_found","message":""}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_recover_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 118,
- Name: "openai-ingress-prev-recovery",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_seed_anchor"}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_prev_recover_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_turn_prev_recover_1"}`)
- secondTurn := readMessage()
- require.Equal(t, "response.completed", gjson.GetBytes(secondTurn, "type").String())
- require.Equal(t, "resp_turn_prev_recover_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "previous_response_not_found 恢复应触发换连重试")
- firstConn.mu.Lock()
- firstWrites := append([]map[string]any(nil), firstConn.writes...)
- firstConn.mu.Unlock()
- require.Len(t, firstWrites, 2, "首个连接应处理首轮与失败的第二轮请求")
- require.True(t, gjson.Get(requestToJSONString(firstWrites[1]), "previous_response_id").Exists(), "失败轮次首发请求应包含 previous_response_id")
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1, "恢复重试应在第二个连接发送一次请求")
- require.False(t, gjson.Get(requestToJSONString(secondWrites[0]), "previous_response_id").Exists(), "恢复重试应移除 previous_response_id")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_StoreDisabledStrictAffinityPreviousResponseNotFoundLayer2Recovery(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.IngressPreviousResponseRecoveryEnabled = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_strict_recover_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"error","error":{"type":"invalid_request_error","code":"previous_response_not_found","message":"missing strict anchor"}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_strict_recover_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 122,
- Name: "openai-ingress-prev-strict-layer2",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"prompt_cache_key":"pk_strict_layer2","input":[{"type":"input_text","text":"hello"}]}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_prev_strict_recover_1", gjson.GetBytes(firstTurn, "response.id").String())
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"store":false,"prompt_cache_key":"pk_strict_layer2","previous_response_id":"resp_turn_prev_strict_recover_1","input":[{"type":"input_text","text":"world"}]}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_prev_strict_recover_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 严格亲和 Layer2 恢复结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "严格亲和链路命中 previous_response_not_found 应触发 Layer2 恢复重试")
- firstConn.mu.Lock()
- firstWrites := append([]map[string]any(nil), firstConn.writes...)
- firstConn.mu.Unlock()
- require.Len(t, firstWrites, 2, "首连接应收到首轮请求和失败的续链请求")
- require.True(t, gjson.Get(requestToJSONString(firstWrites[1]), "previous_response_id").Exists())
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1, "Layer2 恢复应仅重放一次")
- secondWrite := requestToJSONString(secondWrites[0])
- require.False(t, gjson.Get(secondWrite, "previous_response_id").Exists(), "Layer2 恢复重放应移除 previous_response_id")
- require.True(t, gjson.Get(secondWrite, "store").Exists(), "Layer2 恢复不应改变 store 标志")
- require.False(t, gjson.Get(secondWrite, "store").Bool())
- require.Equal(t, 2, len(gjson.Get(secondWrite, "input").Array()), "Layer2 恢复应重放完整 input 上下文")
- require.Equal(t, "hello", gjson.Get(secondWrite, "input.0.text").String())
- require.Equal(t, "world", gjson.Get(secondWrite, "input.1.text").String())
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_PreviousResponseNotFoundRecoveryRemovesDuplicatePrevID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.IngressPreviousResponseRecoveryEnabled = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- firstConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_once_1","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- []byte(`{"type":"error","error":{"type":"invalid_request_error","code":"previous_response_not_found","message":"first missing"}}`),
- },
- }
- secondConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"response.completed","response":{"id":"resp_turn_prev_once_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{firstConn, secondConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 120,
- Name: "openai-ingress-prev-recovery-once",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false}`)
- firstTurn := readMessage()
- require.Equal(t, "resp_turn_prev_once_1", gjson.GetBytes(firstTurn, "response.id").String())
- // duplicate previous_response_id: 恢复重试时应删除所有重复键,避免再次 previous_response_not_found。
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"resp_turn_prev_once_1","input":[],"previous_response_id":"resp_turn_prev_duplicate"}`)
- secondTurn := readMessage()
- require.Equal(t, "resp_turn_prev_once_2", gjson.GetBytes(secondTurn, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- require.Equal(t, 2, dialer.DialCount(), "previous_response_not_found 恢复应只重试一次")
- firstConn.mu.Lock()
- firstWrites := append([]map[string]any(nil), firstConn.writes...)
- firstConn.mu.Unlock()
- require.Len(t, firstWrites, 2)
- require.True(t, gjson.Get(requestToJSONString(firstWrites[1]), "previous_response_id").Exists())
- secondConn.mu.Lock()
- secondWrites := append([]map[string]any(nil), secondConn.writes...)
- secondConn.mu.Unlock()
- require.Len(t, secondWrites, 1)
- require.False(t, gjson.Get(requestToJSONString(secondWrites[0]), "previous_response_id").Exists(), "重复键场景恢复重试后不应保留 previous_response_id")
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_RejectsMessageIDAsPreviousResponseID(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- }
- account := &Account{
- ID: 119,
- Name: "openai-ingress-prev-validation",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"gpt-5.1","stream":false,"previous_response_id":"msg_123456"}`))
- cancelWrite()
- require.NoError(t, err)
- select {
- case serverErr := <-serverErrCh:
- require.Error(t, serverErr)
- var closeErr *OpenAIWSClientCloseError
- require.ErrorAs(t, serverErr, &closeErr)
- require.Equal(t, coderws.StatusPolicyViolation, closeErr.StatusCode())
- require.Contains(t, closeErr.Reason(), "previous_response_id must be a response.id")
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- }
- type openAIWSQueueDialer struct {
- mu sync.Mutex
- conns []openAIWSClientConn
- dialCount int
- }
- func (d *openAIWSQueueDialer) Dial(
- ctx context.Context,
- wsURL string,
- headers http.Header,
- proxyURL string,
- ) (openAIWSClientConn, int, http.Header, error) {
- _ = ctx
- _ = wsURL
- _ = headers
- _ = proxyURL
- d.mu.Lock()
- defer d.mu.Unlock()
- d.dialCount++
- if len(d.conns) == 0 {
- return nil, 503, nil, errors.New("no test conn")
- }
- conn := d.conns[0]
- if len(d.conns) > 1 {
- d.conns = d.conns[1:]
- }
- return conn, 0, nil, nil
- }
- func (d *openAIWSQueueDialer) DialCount() int {
- d.mu.Lock()
- defer d.mu.Unlock()
- return d.dialCount
- }
- type openAIWSPreflightFailConn struct {
- mu sync.Mutex
- events [][]byte
- pingFails bool
- writeCount int
- pingCount int
- }
- func (c *openAIWSPreflightFailConn) WriteJSON(context.Context, any) error {
- c.mu.Lock()
- c.writeCount++
- c.mu.Unlock()
- return nil
- }
- func (c *openAIWSPreflightFailConn) ReadMessage(context.Context) ([]byte, error) {
- c.mu.Lock()
- defer c.mu.Unlock()
- if len(c.events) == 0 {
- return nil, io.EOF
- }
- event := c.events[0]
- c.events = c.events[1:]
- if len(c.events) == 0 {
- c.pingFails = true
- }
- return event, nil
- }
- func (c *openAIWSPreflightFailConn) Ping(context.Context) error {
- c.mu.Lock()
- defer c.mu.Unlock()
- c.pingCount++
- if c.pingFails {
- return errors.New("preflight ping failed")
- }
- return nil
- }
- func (c *openAIWSPreflightFailConn) Close() error {
- return nil
- }
- func (c *openAIWSPreflightFailConn) WriteCount() int {
- c.mu.Lock()
- defer c.mu.Unlock()
- return c.writeCount
- }
- func (c *openAIWSPreflightFailConn) PingCount() int {
- c.mu.Lock()
- defer c.mu.Unlock()
- return c.pingCount
- }
- type openAIWSWriteFailAfterFirstTurnConn struct {
- mu sync.Mutex
- events [][]byte
- failOnWrite bool
- }
- func (c *openAIWSWriteFailAfterFirstTurnConn) WriteJSON(context.Context, any) error {
- c.mu.Lock()
- defer c.mu.Unlock()
- if c.failOnWrite {
- return errors.New("write failed on stale conn")
- }
- return nil
- }
- func (c *openAIWSWriteFailAfterFirstTurnConn) ReadMessage(context.Context) ([]byte, error) {
- c.mu.Lock()
- defer c.mu.Unlock()
- if len(c.events) == 0 {
- return nil, io.EOF
- }
- event := c.events[0]
- c.events = c.events[1:]
- if len(c.events) == 0 {
- c.failOnWrite = true
- }
- return event, nil
- }
- func (c *openAIWSWriteFailAfterFirstTurnConn) Ping(context.Context) error {
- return nil
- }
- func (c *openAIWSWriteFailAfterFirstTurnConn) Close() error {
- return nil
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_ClientDisconnectStillDrainsUpstream(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- // 多个上游事件:前几个为非 terminal 事件,最后一个为 terminal。
- // 第一个事件延迟 250ms 让客户端 RST 有时间传播,使 writeClientMessage 可靠失败。
- captureConn := &openAIWSCaptureConn{
- readDelays: []time.Duration{250 * time.Millisecond, 0, 0},
- events: [][]byte{
- []byte(`{"type":"response.created","response":{"id":"resp_ingress_disconnect","model":"gpt-5.1"}}`),
- []byte(`{"type":"response.output_item.added","response":{"id":"resp_ingress_disconnect"}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_ingress_disconnect","model":"gpt-5.1","usage":{"input_tokens":2,"output_tokens":1}}}`),
- },
- }
- captureDialer := &openAIWSCaptureDialer{conn: captureConn}
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(captureDialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 115,
- Name: "openai-ingress-client-disconnect",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- "model_mapping": map[string]any{
- "custom-original-model": "gpt-5.1",
- },
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- resultCh := make(chan *OpenAIForwardResult, 1)
- hooks := &OpenAIWSIngressHooks{
- AfterTurn: func(_ int, result *OpenAIForwardResult, turnErr error) {
- if turnErr == nil && result != nil {
- resultCh <- result
- }
- },
- }
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, hooks)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- writeCtx, cancelWrite := context.WithTimeout(context.Background(), 3*time.Second)
- err = clientConn.Write(writeCtx, coderws.MessageText, []byte(`{"type":"response.create","model":"custom-original-model","stream":false,"service_tier":"flex"}`))
- cancelWrite()
- require.NoError(t, err)
- // 立即关闭客户端,模拟客户端在 relay 期间断连。
- require.NoError(t, clientConn.CloseNow(), "模拟 ingress 客户端提前断连")
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr, "客户端断连后应继续 drain 上游直到 terminal 或正常结束")
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- select {
- case result := <-resultCh:
- require.Equal(t, "resp_ingress_disconnect", result.RequestID)
- require.Equal(t, 2, result.Usage.InputTokens)
- require.Equal(t, 1, result.Usage.OutputTokens)
- require.NotNil(t, result.ServiceTier)
- require.Equal(t, "flex", *result.ServiceTier)
- case <-time.After(2 * time.Second):
- t.Fatal("未收到断连后的 turn 结果回调")
- }
- }
- func TestOpenAIGatewayService_ProxyResponsesWebSocketFromClient_InvalidEncryptedContentLineageStripsNextTurn(t *testing.T) {
- gin.SetMode(gin.TestMode)
- cfg := &config.Config{}
- cfg.Security.URLAllowlist.Enabled = false
- cfg.Security.URLAllowlist.AllowInsecureHTTP = true
- cfg.Gateway.OpenAIWS.Enabled = true
- cfg.Gateway.OpenAIWS.OAuthEnabled = true
- cfg.Gateway.OpenAIWS.APIKeyEnabled = true
- cfg.Gateway.OpenAIWS.ResponsesWebsocketsV2 = true
- cfg.Gateway.OpenAIWS.MaxConnsPerAccount = 1
- cfg.Gateway.OpenAIWS.MinIdlePerAccount = 0
- cfg.Gateway.OpenAIWS.MaxIdlePerAccount = 1
- cfg.Gateway.OpenAIWS.QueueLimitPerConn = 8
- cfg.Gateway.OpenAIWS.DialTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.ReadTimeoutSeconds = 3
- cfg.Gateway.OpenAIWS.WriteTimeoutSeconds = 3
- upstreamConn := &openAIWSCaptureConn{
- events: [][]byte{
- []byte(`{"type":"error","error":{"type":"invalid_request_error","code":"invalid_encrypted_content","message":"The encrypted content could not be verified"}}`),
- []byte(`{"type":"response.failed","response":{"id":"resp_enc_lineage_1","model":"gpt-5.1","error":{"code":"invalid_encrypted_content","message":"The encrypted content could not be verified"}}}`),
- []byte(`{"type":"response.completed","response":{"id":"resp_enc_lineage_2","model":"gpt-5.1","usage":{"input_tokens":1,"output_tokens":1}}}`),
- },
- }
- dialer := &openAIWSQueueDialer{
- conns: []openAIWSClientConn{upstreamConn},
- }
- pool := newOpenAIWSConnPool(cfg)
- pool.setClientDialerForTest(dialer)
- svc := &OpenAIGatewayService{
- cfg: cfg,
- httpUpstream: &httpUpstreamRecorder{},
- cache: &stubGatewayCache{},
- openaiWSResolver: NewOpenAIWSProtocolResolver(cfg),
- toolCorrector: NewCodexToolCorrector(),
- openaiWSPool: pool,
- }
- account := &Account{
- ID: 119,
- Name: "openai-ingress-enc-lineage",
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- Status: StatusActive,
- Schedulable: true,
- Concurrency: 1,
- Credentials: map[string]any{
- "api_key": "sk-test",
- },
- Extra: map[string]any{
- "responses_websockets_v2_enabled": true,
- },
- }
- serverErrCh := make(chan error, 1)
- wsServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- conn, err := coderws.Accept(w, r, &coderws.AcceptOptions{
- CompressionMode: coderws.CompressionContextTakeover,
- })
- if err != nil {
- serverErrCh <- err
- return
- }
- defer func() {
- _ = conn.CloseNow()
- }()
- rec := httptest.NewRecorder()
- ginCtx, _ := gin.CreateTestContext(rec)
- req := r.Clone(r.Context())
- req.Header = req.Header.Clone()
- req.Header.Set("User-Agent", "unit-test-agent/1.0")
- ginCtx.Request = req
- readCtx, cancel := context.WithTimeout(r.Context(), 3*time.Second)
- msgType, firstMessage, readErr := conn.Read(readCtx)
- cancel()
- if readErr != nil {
- serverErrCh <- readErr
- return
- }
- if msgType != coderws.MessageText && msgType != coderws.MessageBinary {
- serverErrCh <- errors.New("unsupported websocket client message type")
- return
- }
- serverErrCh <- svc.ProxyResponsesWebSocketFromClient(r.Context(), ginCtx, conn, account, "sk-test", firstMessage, nil)
- }))
- defer wsServer.Close()
- dialCtx, cancelDial := context.WithTimeout(context.Background(), 3*time.Second)
- clientConn, _, err := coderws.Dial(dialCtx, "ws"+strings.TrimPrefix(wsServer.URL, "http"), nil)
- cancelDial()
- require.NoError(t, err)
- defer func() {
- _ = clientConn.CloseNow()
- }()
- writeMessage := func(payload string) {
- writeCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- require.NoError(t, clientConn.Write(writeCtx, coderws.MessageText, []byte(payload)))
- }
- readMessage := func() []byte {
- readCtx, cancel := context.WithTimeout(context.Background(), 3*time.Second)
- defer cancel()
- msgType, message, readErr := clientConn.Read(readCtx)
- require.NoError(t, readErr)
- require.Equal(t, coderws.MessageText, msgType)
- return message
- }
- // turn1:携带失效密文,上游以 invalid_encrypted_content 拒绝(error + response.failed 透传给客户端)。
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"input":[{"type":"reasoning","id":"rs_1","encrypted_content":"stale-cipher","summary":[]},{"type":"input_text","text":"hi"}]}`)
- firstEvent := readMessage()
- require.Equal(t, "error", gjson.GetBytes(firstEvent, "type").String())
- require.Equal(t, "invalid_encrypted_content", gjson.GetBytes(firstEvent, "error.code").String())
- secondEvent := readMessage()
- require.Equal(t, "response.failed", gjson.GetBytes(secondEvent, "type").String())
- // turn2:客户端历史仍带同一失效密文,进场应被 lineage 预剥离后再发上游。
- writeMessage(`{"type":"response.create","model":"gpt-5.1","stream":false,"input":[{"type":"reasoning","id":"rs_1","encrypted_content":"stale-cipher","summary":[]},{"type":"input_text","text":"hi"},{"type":"input_text","text":"again"}]}`)
- thirdEvent := readMessage()
- require.Equal(t, "response.completed", gjson.GetBytes(thirdEvent, "type").String())
- require.Equal(t, "resp_enc_lineage_2", gjson.GetBytes(thirdEvent, "response.id").String())
- require.NoError(t, clientConn.Close(coderws.StatusNormalClosure, "done"))
- select {
- case serverErr := <-serverErrCh:
- require.NoError(t, serverErr)
- case <-time.After(5 * time.Second):
- t.Fatal("等待 ingress websocket 结束超时")
- }
- upstreamConn.mu.Lock()
- writes := append([]map[string]any(nil), upstreamConn.writes...)
- upstreamConn.mu.Unlock()
- require.Len(t, writes, 2, "两轮各应发送一次上游请求")
- firstUpstream := requestToJSONString(writes[0])
- require.Equal(t, "stale-cipher", gjson.Get(firstUpstream, "input.0.encrypted_content").String(), "首轮请求原样携带密文")
- secondUpstream := requestToJSONString(writes[1])
- secondInput := gjson.Get(secondUpstream, "input").Array()
- require.Len(t, secondInput, 3, "剥离仅移除 encrypted_content 字段,reasoning 骨架保留")
- for _, item := range secondInput {
- require.False(t, item.Get("encrypted_content").Exists(), "第二轮请求不得再携带已失效密文: %s", item.Raw)
- }
- require.Equal(t, "rs_1", gjson.Get(secondUpstream, "input.0.id").String())
- require.Equal(t, "again", gjson.Get(secondUpstream, "input.2.text").String())
- }
|