| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546254725482549255025512552255325542555255625572558255925602561256225632564256525662567256825692570257125722573257425752576257725782579258025812582258325842585258625872588258925902591259225932594259525962597259825992600260126022603260426052606260726082609261026112612261326142615261626172618261926202621262226232624262526262627262826292630263126322633263426352636263726382639264026412642264326442645264626472648264926502651265226532654265526562657265826592660266126622663266426652666266726682669267026712672267326742675267626772678267926802681268226832684268526862687268826892690269126922693269426952696269726982699270027012702270327042705 |
- package dns
- import (
- "context"
- "net"
- "net/netip"
- "strings"
- "sync"
- "testing"
- "time"
- "github.com/sagernet/sing-box/adapter"
- C "github.com/sagernet/sing-box/constant"
- "github.com/sagernet/sing-box/experimental/deprecated"
- "github.com/sagernet/sing-box/log"
- "github.com/sagernet/sing-box/option"
- rulepkg "github.com/sagernet/sing-box/route/rule"
- "github.com/sagernet/sing-tun"
- E "github.com/sagernet/sing/common/exceptions"
- "github.com/sagernet/sing/common/json/badoption"
- N "github.com/sagernet/sing/common/network"
- "github.com/sagernet/sing/common/x/list"
- "github.com/sagernet/sing/service"
- mDNS "github.com/miekg/dns"
- "github.com/stretchr/testify/require"
- "go4.org/netipx"
- )
- type fakeDNSTransport struct {
- tag string
- transportType string
- }
- func (t *fakeDNSTransport) Start(adapter.StartStage) error { return nil }
- func (t *fakeDNSTransport) Close() error { return nil }
- func (t *fakeDNSTransport) Type() string { return t.transportType }
- func (t *fakeDNSTransport) Tag() string { return t.tag }
- func (t *fakeDNSTransport) Dependencies() []string { return nil }
- func (t *fakeDNSTransport) Reset() {}
- func (t *fakeDNSTransport) Exchange(context.Context, *mDNS.Msg) (*mDNS.Msg, error) {
- return nil, E.New("unused transport exchange")
- }
- type fakeDNSTransportManager struct {
- defaultTransport adapter.DNSTransport
- transports map[string]adapter.DNSTransport
- }
- func (m *fakeDNSTransportManager) Start(adapter.StartStage) error { return nil }
- func (m *fakeDNSTransportManager) Close() error { return nil }
- func (m *fakeDNSTransportManager) Transports() []adapter.DNSTransport {
- transports := make([]adapter.DNSTransport, 0, len(m.transports))
- for _, transport := range m.transports {
- transports = append(transports, transport)
- }
- return transports
- }
- func (m *fakeDNSTransportManager) Transport(tag string) (adapter.DNSTransport, bool) {
- transport, loaded := m.transports[tag]
- return transport, loaded
- }
- func (m *fakeDNSTransportManager) Default() adapter.DNSTransport { return m.defaultTransport }
- func (m *fakeDNSTransportManager) FakeIP() adapter.FakeIPTransport {
- return nil
- }
- func (m *fakeDNSTransportManager) Remove(string) error { return nil }
- func (m *fakeDNSTransportManager) Create(context.Context, log.ContextLogger, string, string, any) error {
- return E.New("unsupported")
- }
- type fakeDNSClient struct {
- beforeExchange func(ctx context.Context, transport adapter.DNSTransport, message *mDNS.Msg)
- exchange func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error)
- lookupWithCtx func(ctx context.Context, transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error)
- lookup func(transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error)
- }
- type fakeDeprecatedManager struct {
- features []deprecated.Note
- }
- type fakeRouter struct {
- access sync.RWMutex
- ruleSets map[string]adapter.RuleSet
- }
- func (r *fakeRouter) Start(adapter.StartStage) error { return nil }
- func (r *fakeRouter) Close() error { return nil }
- func (r *fakeRouter) PreMatch(metadata adapter.InboundContext, _ tun.DirectRouteContext, _ time.Duration, _ bool) (tun.DirectRouteDestination, error) {
- return nil, nil
- }
- func (r *fakeRouter) RouteConnection(context.Context, net.Conn, adapter.InboundContext) error {
- return nil
- }
- func (r *fakeRouter) RoutePacketConnection(context.Context, N.PacketConn, adapter.InboundContext) error {
- return nil
- }
- func (r *fakeRouter) RouteConnectionEx(context.Context, net.Conn, adapter.InboundContext, N.CloseHandlerFunc) {
- }
- func (r *fakeRouter) RoutePacketConnectionEx(context.Context, N.PacketConn, adapter.InboundContext, N.CloseHandlerFunc) {
- }
- func (r *fakeRouter) RuleSet(tag string) (adapter.RuleSet, bool) {
- r.access.RLock()
- defer r.access.RUnlock()
- ruleSet, loaded := r.ruleSets[tag]
- return ruleSet, loaded
- }
- func (r *fakeRouter) setRuleSet(tag string, ruleSet adapter.RuleSet) {
- r.access.Lock()
- defer r.access.Unlock()
- if r.ruleSets == nil {
- r.ruleSets = make(map[string]adapter.RuleSet)
- }
- r.ruleSets[tag] = ruleSet
- }
- func (r *fakeRouter) Rules() []adapter.Rule { return nil }
- func (r *fakeRouter) NeedFindProcess() bool { return false }
- func (r *fakeRouter) NeedFindNeighbor() bool { return false }
- func (r *fakeRouter) NeighborResolver() adapter.NeighborResolver { return nil }
- func (r *fakeRouter) AppendTracker(adapter.ConnectionTracker) {}
- func (r *fakeRouter) ResetNetwork() {}
- type fakeRuleSet struct {
- access sync.Mutex
- metadata adapter.RuleSetMetadata
- metadataRead func(adapter.RuleSetMetadata) adapter.RuleSetMetadata
- match func(*adapter.InboundContext) bool
- callbacks list.List[adapter.RuleSetUpdateCallback]
- refs int
- afterIncrementReference func()
- beforeDecrementReference func()
- }
- func (s *fakeRuleSet) Name() string { return "fake-rule-set" }
- func (s *fakeRuleSet) StartContext(context.Context, *adapter.HTTPStartContext) error { return nil }
- func (s *fakeRuleSet) PostStart() error { return nil }
- func (s *fakeRuleSet) Metadata() adapter.RuleSetMetadata {
- s.access.Lock()
- metadata := s.metadata
- metadataRead := s.metadataRead
- s.access.Unlock()
- if metadataRead != nil {
- return metadataRead(metadata)
- }
- return metadata
- }
- func (s *fakeRuleSet) ExtractIPSet() []*netipx.IPSet { return nil }
- func (s *fakeRuleSet) IncRef() {
- s.access.Lock()
- s.refs++
- afterIncrementReference := s.afterIncrementReference
- s.access.Unlock()
- if afterIncrementReference != nil {
- afterIncrementReference()
- }
- }
- func (s *fakeRuleSet) DecRef() {
- s.access.Lock()
- beforeDecrementReference := s.beforeDecrementReference
- s.access.Unlock()
- if beforeDecrementReference != nil {
- beforeDecrementReference()
- }
- s.access.Lock()
- defer s.access.Unlock()
- s.refs--
- if s.refs < 0 {
- panic("rule-set: negative refs")
- }
- }
- func (s *fakeRuleSet) Cleanup() {}
- func (s *fakeRuleSet) RegisterCallback(callback adapter.RuleSetUpdateCallback) *list.Element[adapter.RuleSetUpdateCallback] {
- s.access.Lock()
- defer s.access.Unlock()
- return s.callbacks.PushBack(callback)
- }
- func (s *fakeRuleSet) UnregisterCallback(element *list.Element[adapter.RuleSetUpdateCallback]) {
- s.access.Lock()
- defer s.access.Unlock()
- s.callbacks.Remove(element)
- }
- func (s *fakeRuleSet) Close() error { return nil }
- func (s *fakeRuleSet) Match(metadata *adapter.InboundContext) bool {
- s.access.Lock()
- match := s.match
- s.access.Unlock()
- if match != nil {
- return match(metadata)
- }
- return true
- }
- func (s *fakeRuleSet) String() string { return "fake-rule-set" }
- func (s *fakeRuleSet) updateMetadata(metadata adapter.RuleSetMetadata) {
- s.access.Lock()
- s.metadata = metadata
- callbacks := s.callbacks.Array()
- s.access.Unlock()
- for _, callback := range callbacks {
- callback(s)
- }
- }
- func (s *fakeRuleSet) snapshotCallbacks() []adapter.RuleSetUpdateCallback {
- s.access.Lock()
- defer s.access.Unlock()
- return s.callbacks.Array()
- }
- func (s *fakeRuleSet) refCount() int {
- s.access.Lock()
- defer s.access.Unlock()
- return s.refs
- }
- func (m *fakeDeprecatedManager) ReportDeprecated(feature deprecated.Note) {
- m.features = append(m.features, feature)
- }
- func (c *fakeDNSClient) Start() {}
- func (c *fakeDNSClient) Exchange(ctx context.Context, transport adapter.DNSTransport, message *mDNS.Msg, _ adapter.DNSQueryOptions, _ func(*mDNS.Msg) bool) (*mDNS.Msg, error) {
- if c.beforeExchange != nil {
- c.beforeExchange(ctx, transport, message)
- }
- if c.exchange == nil {
- if len(message.Question) != 1 {
- return nil, E.New("unused client exchange")
- }
- var (
- addresses []netip.Addr
- response *mDNS.Msg
- err error
- )
- if c.lookupWithCtx != nil {
- addresses, response, err = c.lookupWithCtx(ctx, transport, FqdnToDomain(message.Question[0].Name), adapter.DNSQueryOptions{})
- } else if c.lookup != nil {
- addresses, response, err = c.lookup(transport, FqdnToDomain(message.Question[0].Name), adapter.DNSQueryOptions{})
- } else {
- return nil, E.New("unused client exchange")
- }
- if err != nil {
- return nil, err
- }
- if response != nil {
- return response, nil
- }
- return FixedResponse(0, message.Question[0], addresses, 60), nil
- }
- return c.exchange(transport, message)
- }
- func (c *fakeDNSClient) Lookup(ctx context.Context, transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions, responseChecker func(*mDNS.Msg) bool) ([]netip.Addr, error) {
- if c.lookup == nil && c.lookupWithCtx == nil {
- return nil, E.New("unused client lookup")
- }
- var (
- addresses []netip.Addr
- response *mDNS.Msg
- err error
- )
- if c.lookupWithCtx != nil {
- addresses, response, err = c.lookupWithCtx(ctx, transport, domain, options)
- } else {
- addresses, response, err = c.lookup(transport, domain, options)
- }
- if err != nil {
- return nil, err
- }
- if response == nil {
- response = FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), addresses, 60)
- }
- if responseChecker != nil && !responseChecker(response) {
- return nil, ErrResponseRejected
- }
- if addresses != nil {
- return addresses, nil
- }
- return MessageToAddresses(response), nil
- }
- func (c *fakeDNSClient) ClearCache() {}
- func newTestRouter(t *testing.T, rules []option.DNSRule, transportManager *fakeDNSTransportManager, client *fakeDNSClient) *Router {
- router := newTestRouterWithContext(t, context.Background(), rules, transportManager, client)
- t.Cleanup(func() {
- router.Close()
- })
- return router
- }
- func newTestRouterWithContext(t *testing.T, ctx context.Context, rules []option.DNSRule, transportManager *fakeDNSTransportManager, client *fakeDNSClient) *Router {
- return newTestRouterWithContextAndLogger(t, ctx, rules, transportManager, client, log.NewNOPFactory().NewLogger("dns"))
- }
- func newTestRouterWithContextAndLogger(t *testing.T, ctx context.Context, rules []option.DNSRule, transportManager *fakeDNSTransportManager, client *fakeDNSClient, dnsLogger log.ContextLogger) *Router {
- t.Helper()
- router := &Router{
- ctx: ctx,
- logger: dnsLogger,
- transport: transportManager,
- client: client,
- rawRules: make([]option.DNSRule, 0, len(rules)),
- rules: make([]adapter.DNSRule, 0, len(rules)),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- if rules != nil {
- err := router.Initialize(rules)
- require.NoError(t, err)
- err = router.Start(adapter.StartStateStart)
- require.NoError(t, err)
- }
- return router
- }
- func waitForLogMessageContaining(t *testing.T, entries <-chan log.Entry, done <-chan struct{}, substring string) log.Entry {
- t.Helper()
- timeout := time.After(time.Second)
- for {
- select {
- case entry, ok := <-entries:
- if !ok {
- t.Fatal("log subscription closed")
- }
- if strings.Contains(entry.Message, substring) {
- return entry
- }
- case <-done:
- t.Fatal("log subscription closed")
- case <-timeout:
- t.Fatalf("timed out waiting for log message containing %q", substring)
- }
- }
- }
- func fixedQuestion(name string, qType uint16) mDNS.Question {
- return mDNS.Question{
- Name: mDNS.Fqdn(name),
- Qtype: qType,
- Qclass: mDNS.ClassINET,
- }
- }
- func mustRecord(t *testing.T, record string) option.DNSRecordOptions {
- t.Helper()
- var value option.DNSRecordOptions
- require.NoError(t, value.UnmarshalJSON([]byte(`"`+record+`"`)))
- return value
- }
- func TestInitializeRejectsDirectLegacyRuleWhenRuleSetForcesNew(t *testing.T) {
- t.Parallel()
- ctx := context.Background()
- ruleSet, err := rulepkg.NewRuleSet(ctx, log.NewNOPFactory().NewLogger("router"), option.RuleSet{
- Type: C.RuleSetTypeInline,
- Tag: "query-set",
- InlineOptions: option.PlainRuleSet{
- Rules: []option.HeadlessRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultHeadlessRule{
- QueryType: badoption.Listable[option.DNSQueryType]{option.DNSQueryType(mDNS.TypeA)},
- },
- }},
- },
- })
- require.NoError(t, err)
- ctx = service.ContextWith[adapter.Router](ctx, &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "query-set": ruleSet,
- },
- })
- router := &Router{
- ctx: ctx,
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 2),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err = router.Initialize([]option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"query-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPIsPrivate: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "private"},
- },
- },
- },
- })
- require.ErrorContains(t, err, "Response Match Fields")
- require.ErrorContains(t, err, "require match_response")
- }
- func TestLookupLegacyDNSModeDefersRuleSetDestinationIPMatch(t *testing.T) {
- t.Parallel()
- ctx := context.Background()
- ruleSet, err := rulepkg.NewRuleSet(ctx, log.NewNOPFactory().NewLogger("router"), option.RuleSet{
- Type: C.RuleSetTypeInline,
- Tag: "legacy-ipcidr-set",
- InlineOptions: option.PlainRuleSet{
- Rules: []option.HeadlessRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultHeadlessRule{
- IPCIDR: badoption.Listable[string]{"10.0.0.0/8"},
- },
- }},
- },
- })
- require.NoError(t, err)
- ctx = service.ContextWith[adapter.Router](ctx, &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "legacy-ipcidr-set": ruleSet,
- },
- })
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- privateTransport := &fakeDNSTransport{tag: "private", transportType: C.DNSTypeUDP}
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"legacy-ipcidr-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "private"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "private": privateTransport,
- },
- }, &fakeDNSClient{
- lookup: func(transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- require.Equal(t, "example.com", domain)
- require.Equal(t, "private", transport.Tag())
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("10.0.0.1")}, 60)
- return MessageToAddresses(response), response, nil
- },
- })
- require.True(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{
- LookupStrategy: C.DomainStrategyIPv4Only,
- })
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("10.0.0.1")}, addresses)
- }
- func TestRuleSetUpdateReleasesOldRuleSetRefs(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{}
- ctx := service.ContextWith[adapter.Router](context.Background(), &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- })
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{})
- require.Equal(t, 1, fakeSet.refCount())
- fakeSet.updateMetadata(adapter.RuleSetMetadata{})
- require.Equal(t, 1, fakeSet.refCount())
- fakeSet.updateMetadata(adapter.RuleSetMetadata{})
- require.Equal(t, 1, fakeSet.refCount())
- require.NoError(t, router.Close())
- require.Zero(t, fakeSet.refCount())
- }
- func TestValidateRuleSetMetadataUpdateRejectsRuleSetThatWouldDisableLegacyDNSMode(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPIsPrivate: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- lookup: func(adapter.DNSTransport, string, adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- return []netip.Addr{netip.MustParseAddr("10.0.0.1")}, nil, nil
- },
- })
- require.True(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{
- ContainsDNSQueryTypeRule: true,
- })
- require.ErrorContains(t, err, "require match_response")
- }
- func TestValidateRuleSetMetadataUpdateRejectsRuleSetOnlyLegacyModeSwitchToNew(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- lookup: func(adapter.DNSTransport, string, adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- return []netip.Addr{netip.MustParseAddr("10.0.0.1")}, nil, nil
- },
- })
- require.True(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- ContainsDNSQueryTypeRule: true,
- })
- require.ErrorContains(t, err, "Address Filter Fields")
- }
- func TestValidateRuleSetMetadataUpdateBeforeStartUsesStartupValidation(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{}
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := &Router{
- ctx: ctx,
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 2),
- rules: make([]adapter.DNSRule, 0, 2),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPIsPrivate: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- })
- require.NoError(t, err)
- require.False(t, router.started)
- err = router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{
- ContainsDNSQueryTypeRule: true,
- })
- require.ErrorContains(t, err, "require match_response")
- }
- func TestValidateRuleSetMetadataUpdateRejectsRuleSetThatWouldRequireLegacyDNSMode(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{}
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- lookup: func(adapter.DNSTransport, string, adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- return []netip.Addr{netip.MustParseAddr("1.1.1.1")}, nil, nil
- },
- })
- require.False(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- })
- require.ErrorContains(t, err, "Address Filter Fields")
- }
- func TestValidateRuleSetMetadataUpdateAllowsRuleSetThatKeepsNonLegacyDNSMode(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{}
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- ContainsNonIPCIDRRule: true,
- })
- require.NoError(t, err)
- }
- func TestValidateRuleSetMetadataUpdateAllowsRelaxingLegacyRequirement(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "dynamic-set": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"dynamic-set"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- lookup: func(adapter.DNSTransport, string, adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- return []netip.Addr{netip.MustParseAddr("10.0.0.1")}, nil, nil
- },
- })
- require.True(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("dynamic-set", adapter.RuleSetMetadata{})
- require.NoError(t, err)
- }
- func TestInitializeRejectsPureIPRuleSetWhenLegacyDNSModeDisabled(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "pure-ip": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := &Router{
- ctx: ctx,
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 2),
- rules: make([]adapter.DNSRule, 0, 2),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- QueryType: badoption.Listable[option.DNSQueryType]{option.DNSQueryType(mDNS.TypeA)},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"pure-ip"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- })
- require.ErrorContains(t, err, "Address Filter Fields")
- }
- func TestInitializeAllowsMixedRuleSetWhenLegacyDNSModeDisabled(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- ContainsNonIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "mixed": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- QueryType: badoption.Listable[option.DNSQueryType]{option.DNSQueryType(mDNS.TypeA)},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"mixed"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- }
- func TestValidateRuleSetMetadataUpdateRejectsRuleSetFlippingToPureIP(t *testing.T) {
- t.Parallel()
- fakeSet := &fakeRuleSet{
- metadata: adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- ContainsNonIPCIDRRule: true,
- },
- }
- routerService := &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "mixed": fakeSet,
- },
- }
- ctx := service.ContextWith[adapter.Router](context.Background(), routerService)
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- QueryType: badoption.Listable[option.DNSQueryType]{option.DNSQueryType(mDNS.TypeA)},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"mixed"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- err := router.ValidateRuleSetMetadataUpdate("mixed", adapter.RuleSetMetadata{
- ContainsIPCIDRRule: true,
- })
- require.ErrorContains(t, err, "Address Filter Fields")
- }
- func TestCloseWaitsForInFlightLookupUntilContextCancellation(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- selectedTransport := &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP}
- lookupStarted := make(chan struct{})
- var lookupStartedOnce sync.Once
- router := newTestRouter(t, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "selected": selectedTransport,
- },
- }, &fakeDNSClient{
- lookupWithCtx: func(ctx context.Context, transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- require.Equal(t, "selected", transport.Tag())
- require.Equal(t, "example.com", domain)
- lookupStartedOnce.Do(func() {
- close(lookupStarted)
- })
- <-ctx.Done()
- return nil, nil, ctx.Err()
- },
- })
- lookupCtx, cancelLookup := context.WithCancel(context.Background())
- defer cancelLookup()
- var (
- lookupErr error
- closeErr error
- )
- lookupDone := make(chan struct{})
- go func() {
- _, lookupErr = router.Lookup(lookupCtx, "example.com", adapter.DNSQueryOptions{})
- close(lookupDone)
- }()
- select {
- case <-lookupStarted:
- case <-time.After(time.Second):
- t.Fatal("lookup did not reach DNS client")
- }
- closeDone := make(chan struct{})
- go func() {
- closeErr = router.Close()
- close(closeDone)
- }()
- select {
- case <-closeDone:
- t.Fatal("close finished before lookup context cancellation")
- default:
- }
- cancelLookup()
- select {
- case <-lookupDone:
- case <-time.After(time.Second):
- t.Fatal("lookup did not finish after cancellation")
- }
- select {
- case <-closeDone:
- case <-time.After(time.Second):
- t.Fatal("close did not finish after lookup cancellation")
- }
- require.ErrorIs(t, lookupErr, context.Canceled)
- require.NoError(t, closeErr)
- }
- func TestLookupLegacyDNSModeDefersDirectDestinationIPMatch(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- privateTransport := &fakeDNSTransport{tag: "private", transportType: C.DNSTypeUDP}
- client := &fakeDNSClient{
- lookup: func(transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- require.Equal(t, "example.com", domain)
- require.Equal(t, C.DomainStrategyIPv4Only, options.LookupStrategy)
- switch transport.Tag() {
- case "private":
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("10.0.0.1")}, 60)
- return MessageToAddresses(response), response, nil
- case "default":
- t.Fatal("default transport should not be used when legacy rule matches after response")
- }
- return nil, nil, E.New("unexpected transport")
- },
- }
- router := newTestRouter(t, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPIsPrivate: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "private"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "private": privateTransport,
- },
- }, client)
- require.True(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{
- LookupStrategy: C.DomainStrategyIPv4Only,
- })
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("10.0.0.1")}, addresses)
- }
- func TestLookupLegacyDNSModeFallsBackAfterRejectedAddressLimitResponse(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- privateTransport := &fakeDNSTransport{tag: "private", transportType: C.DNSTypeUDP}
- var lookupAccess sync.Mutex
- var lookupTags []string
- recordLookup := func(tag string) {
- lookupAccess.Lock()
- lookupTags = append(lookupTags, tag)
- lookupAccess.Unlock()
- }
- currentLookupTags := func() []string {
- lookupAccess.Lock()
- defer lookupAccess.Unlock()
- return append([]string(nil), lookupTags...)
- }
- client := &fakeDNSClient{
- lookup: func(transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- require.Equal(t, "example.com", domain)
- require.Equal(t, C.DomainStrategyIPv4Only, options.LookupStrategy)
- recordLookup(transport.Tag())
- switch transport.Tag() {
- case "private":
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60)
- return MessageToAddresses(response), response, nil
- case "default":
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("9.9.9.9")}, 60)
- return MessageToAddresses(response), response, nil
- }
- return nil, nil, E.New("unexpected transport")
- },
- }
- router := newTestRouter(t, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPIsPrivate: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "private"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "private": privateTransport,
- },
- }, client)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{
- LookupStrategy: C.DomainStrategyIPv4Only,
- })
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("9.9.9.9")}, addresses)
- require.Equal(t, []string{"private", "default"}, currentLookupTags())
- }
- func TestLookupLegacyDNSModeRuleSetAcceptEmptyDoesNotTreatMismatchAsEmpty(t *testing.T) {
- t.Parallel()
- ctx := context.Background()
- ruleSet, err := rulepkg.NewRuleSet(ctx, log.NewNOPFactory().NewLogger("router"), option.RuleSet{
- Type: C.RuleSetTypeInline,
- Tag: "legacy-ipcidr-set",
- InlineOptions: option.PlainRuleSet{
- Rules: []option.HeadlessRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultHeadlessRule{
- IPCIDR: badoption.Listable[string]{"10.0.0.0/8"},
- },
- }},
- },
- })
- require.NoError(t, err)
- ctx = service.ContextWith[adapter.Router](ctx, &fakeRouter{
- ruleSets: map[string]adapter.RuleSet{
- "legacy-ipcidr-set": ruleSet,
- },
- })
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- privateTransport := &fakeDNSTransport{tag: "private", transportType: C.DNSTypeUDP}
- var lookupAccess sync.Mutex
- var lookupTags []string
- recordLookup := func(tag string) {
- lookupAccess.Lock()
- lookupTags = append(lookupTags, tag)
- lookupAccess.Unlock()
- }
- currentLookupTags := func() []string {
- lookupAccess.Lock()
- defer lookupAccess.Unlock()
- return append([]string(nil), lookupTags...)
- }
- router := newTestRouterWithContext(t, ctx, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- RuleSet: badoption.Listable[string]{"legacy-ipcidr-set"},
- RuleSetIPCIDRAcceptEmpty: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "private"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "private": privateTransport,
- },
- }, &fakeDNSClient{
- lookup: func(transport adapter.DNSTransport, domain string, options adapter.DNSQueryOptions) ([]netip.Addr, *mDNS.Msg, error) {
- require.Equal(t, "example.com", domain)
- require.Equal(t, C.DomainStrategyIPv4Only, options.LookupStrategy)
- recordLookup(transport.Tag())
- switch transport.Tag() {
- case "private":
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60)
- return MessageToAddresses(response), response, nil
- case "default":
- response := FixedResponse(0, fixedQuestion(domain, mDNS.TypeA), []netip.Addr{netip.MustParseAddr("9.9.9.9")}, 60)
- return MessageToAddresses(response), response, nil
- }
- return nil, nil, E.New("unexpected transport")
- },
- })
- require.True(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{
- LookupStrategy: C.DomainStrategyIPv4Only,
- })
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("9.9.9.9")}, addresses)
- require.Equal(t, []string{"private", "default"}, currentLookupTags())
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateMatchResponseRoute(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateMatchResponseRcodeRoute(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return &mDNS.Msg{
- MsgHdr: mDNS.MsgHdr{
- Response: true,
- Rcode: mDNS.RcodeNameError,
- },
- Question: []mDNS.Question{message.Question[0]},
- }, nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rcode := option.DNSRCode(mDNS.RcodeNameError)
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseRcode: &rcode,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateMatchResponseNsRoute(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- nsRecord := mustRecord(t, "example.com. IN NS ns1.example.com.")
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return &mDNS.Msg{
- MsgHdr: mDNS.MsgHdr{
- Response: true,
- Rcode: mDNS.RcodeSuccess,
- },
- Question: []mDNS.Question{message.Question[0]},
- Ns: []mDNS.RR{nsRecord.Build()},
- }, nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseNs: badoption.Listable[option.DNSRecordOptions]{nsRecord},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateMatchResponseExtraRoute(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- extraRecord := mustRecord(t, "ns1.example.com. IN A 192.0.2.53")
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return &mDNS.Msg{
- MsgHdr: mDNS.MsgHdr{
- Response: true,
- Rcode: mDNS.RcodeSuccess,
- },
- Question: []mDNS.Question{message.Question[0]},
- Extra: []mDNS.RR{extraRecord.Build()},
- }, nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseExtra: badoption.Listable[option.DNSRecordOptions]{extraRecord},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateDoesNotLeakAddressesToNextQuery(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- var inspectedSelected bool
- client := &fakeDNSClient{
- beforeExchange: func(ctx context.Context, transport adapter.DNSTransport, message *mDNS.Msg) {
- if transport.Tag() != "selected" {
- return
- }
- inspectedSelected = true
- metadata := adapter.ContextFrom(ctx)
- require.NotNil(t, metadata)
- require.Empty(t, metadata.DestinationAddresses)
- require.NotNil(t, metadata.DNSResponse)
- },
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.True(t, inspectedSelected)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateRouteResolutionFailureClearsResponse(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- case "default":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("4.4.4.4")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "missing"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("4.4.4.4")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledSecondEvaluateOverwritesFirstResponse(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "first-upstream": &fakeDNSTransport{tag: "first-upstream", transportType: C.DNSTypeUDP},
- "second-upstream": &fakeDNSTransport{tag: "second-upstream", transportType: C.DNSTypeUDP},
- "first-match": &fakeDNSTransport{tag: "first-match", transportType: C.DNSTypeUDP},
- "second-match": &fakeDNSTransport{tag: "second-match", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "first-upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case "second-upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("2.2.2.2")}, 60), nil
- case "first-match":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("7.7.7.7")}, 60), nil
- case "second-match":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- case "default":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("4.4.4.4")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "first-upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "second-upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "first-match"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 2.2.2.2")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "second-match"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("8.8.8.8")}, MessageToAddresses(response))
- }
- func TestExchangeLegacyDNSModeDisabledEvaluateExchangeFailureUsesMatchResponseBooleanSemantics(t *testing.T) {
- t.Parallel()
- testCases := []struct {
- name string
- invert bool
- expectedAddr netip.Addr
- }{
- {
- name: "plain match_response rule stays false",
- expectedAddr: netip.MustParseAddr("4.4.4.4"),
- },
- {
- name: "invert match_response rule becomes true",
- invert: true,
- expectedAddr: netip.MustParseAddr("8.8.8.8"),
- },
- }
- for _, testCase := range testCases {
- t.Run(testCase.name, func(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return nil, E.New("upstream exchange failed")
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- case "default":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("4.4.4.4")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- Invert: testCase.invert,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{testCase.expectedAddr}, MessageToAddresses(response))
- })
- }
- }
- func TestExchangeLegacyDNSModeDisabledRespondReturnsEvaluatedResponse(t *testing.T) {
- t.Parallel()
- var exchanges []string
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRespond,
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- exchanges = append(exchanges, transport.Tag())
- require.Equal(t, "upstream", transport.Tag())
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- },
- })
- require.False(t, router.legacyDNSMode)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []string{"upstream"}, exchanges)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("1.1.1.1")}, MessageToAddresses(response))
- }
- func TestLookupLegacyDNSModeDisabledRespondReturnsEvaluatedResponse(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRespond,
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- require.Equal(t, "upstream", transport.Tag())
- switch message.Question[0].Qtype {
- case mDNS.TypeA:
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case mDNS.TypeAAAA:
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("2001:db8::1")}, 60), nil
- default:
- return nil, E.New("unexpected qtype")
- }
- },
- })
- require.False(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{
- netip.MustParseAddr("1.1.1.1"),
- netip.MustParseAddr("2001:db8::1"),
- }, addresses)
- }
- func TestExchangeLegacyDNSModeDisabledRespondWithoutEvaluatedResponseReturnsError(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRespond,
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, _ *mDNS.Msg) (*mDNS.Msg, error) {
- require.Equal(t, "upstream", transport.Tag())
- return nil, E.New("upstream exchange failed")
- },
- })
- require.False(t, router.legacyDNSMode)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.Nil(t, response)
- require.ErrorContains(t, err, dnsRespondMissingResponseMessage)
- }
- func TestLookupLegacyDNSModeDisabledAllowsPartialSuccessForExchangeFailure(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, nil, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- require.Equal(t, "default", transport.Tag())
- switch message.Question[0].Qtype {
- case mDNS.TypeA:
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case mDNS.TypeAAAA:
- return nil, E.New("ipv6 failed")
- default:
- return nil, E.New("unexpected qtype")
- }
- },
- })
- router.legacyDNSMode = false
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("1.1.1.1")}, addresses)
- }
- func TestLookupLegacyDNSModeDisabledAllowsPartialSuccessForRcodeError(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, nil, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- require.Equal(t, "default", transport.Tag())
- switch message.Question[0].Qtype {
- case mDNS.TypeA:
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("1.1.1.1")}, 60), nil
- case mDNS.TypeAAAA:
- return &mDNS.Msg{
- MsgHdr: mDNS.MsgHdr{
- Response: true,
- Rcode: mDNS.RcodeNameError,
- },
- Question: []mDNS.Question{message.Question[0]},
- }, nil
- default:
- return nil, E.New("unexpected qtype")
- }
- },
- })
- router.legacyDNSMode = false
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("1.1.1.1")}, addresses)
- }
- func TestLookupLegacyDNSModeDisabledSkipsFakeIPRule(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "fake"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- "fake": &fakeDNSTransport{tag: "fake", transportType: C.DNSTypeFakeIP},
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- require.Equal(t, "default", transport.Tag())
- if message.Question[0].Qtype == mDNS.TypeA {
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("2.2.2.2")}, 60), nil
- }
- return FixedResponse(0, message.Question[0], nil, 60), nil
- },
- })
- router.legacyDNSMode = false
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("2.2.2.2")}, addresses)
- }
- func TestExchangeLegacyDNSModeDisabledAllowsRouteFakeIPRule(t *testing.T) {
- t.Parallel()
- fakeTransport := &fakeDNSTransport{tag: "fake", transportType: C.DNSTypeFakeIP}
- router := newTestRouter(t, []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "fake"},
- },
- },
- }}, &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- "fake": fakeTransport,
- },
- }, &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- require.Same(t, fakeTransport, transport)
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("198.18.0.1")}, 60), nil
- },
- })
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("198.18.0.1")}, MessageToAddresses(response))
- }
- func TestInitializeRejectsDNSRuleStrategyWhenLegacyDNSModeIsDisabledByEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{
- Server: "default",
- Strategy: option.DomainStrategy(C.DomainStrategyIPv4Only),
- },
- },
- },
- }})
- require.ErrorContains(t, err, "strategy")
- require.ErrorContains(t, err, "deprecated")
- }
- func TestInitializeRejectsEvaluateFakeIPServerInDefaultRule(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{transports: map[string]adapter.DNSTransport{"fake": &fakeDNSTransport{tag: "fake", transportType: C.DNSTypeFakeIP}}},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "fake"},
- },
- },
- }})
- require.ErrorContains(t, err, "evaluate action cannot use fakeip server")
- require.ErrorContains(t, err, "fake")
- }
- func TestInitializeRejectsEvaluateFakeIPServerInLogicalRule(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{transports: map[string]adapter.DNSTransport{"fake": &fakeDNSTransport{tag: "fake", transportType: C.DNSTypeFakeIP}}},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeLogical,
- LogicalOptions: option.LogicalDNSRule{
- RawLogicalDNSRule: option.RawLogicalDNSRule{
- Mode: C.LogicalTypeOr,
- Rules: []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- },
- }},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "fake"},
- },
- },
- }})
- require.ErrorContains(t, err, "evaluate action cannot use fakeip server")
- require.ErrorContains(t, err, "fake")
- }
- func TestInitializeRejectsDNSRuleStrategyWhenLegacyDNSModeIsDisabledByMatchResponse(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRouteOptions,
- RouteOptionsOptions: option.DNSRouteOptionsActionOptions{
- Strategy: option.DomainStrategy(C.DomainStrategyIPv4Only),
- },
- },
- },
- }})
- require.ErrorContains(t, err, "strategy")
- require.ErrorContains(t, err, "deprecated")
- }
- func TestInitializeRejectsDNSMatchResponseWithoutPrecedingEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- }})
- require.ErrorContains(t, err, "preceding evaluate action")
- }
- func TestInitializeRejectsDNSRespondWithoutPrecedingEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRespond,
- },
- },
- }})
- require.ErrorContains(t, err, "preceding evaluate action")
- }
- func TestInitializeRejectsLogicalDNSRespondWithoutPrecedingEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeLogical,
- LogicalOptions: option.LogicalDNSRule{
- RawLogicalDNSRule: option.RawLogicalDNSRule{
- Mode: C.LogicalTypeOr,
- Rules: []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- },
- }},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRespond,
- },
- },
- }})
- require.ErrorContains(t, err, "preceding evaluate action")
- }
- func TestInitializeRejectsEvaluateRuleWithResponseMatchWithoutPrecedingEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 1),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeLogical,
- LogicalOptions: option.LogicalDNSRule{
- RawLogicalDNSRule: option.RawLogicalDNSRule{
- Mode: C.LogicalTypeOr,
- Rules: []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- },
- },
- },
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- }})
- require.ErrorContains(t, err, "preceding evaluate action")
- }
- func TestInitializeAllowsEvaluateRuleWithResponseMatchAfterPrecedingEvaluate(t *testing.T) {
- t.Parallel()
- router := &Router{
- ctx: context.Background(),
- logger: log.NewNOPFactory().NewLogger("dns"),
- transport: &fakeDNSTransportManager{},
- client: &fakeDNSClient{},
- rawRules: make([]option.DNSRule, 0, 2),
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"bootstrap.example"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "bootstrap"},
- },
- },
- },
- {
- Type: C.RuleTypeLogical,
- LogicalOptions: option.LogicalDNSRule{
- RawLogicalDNSRule: option.RawLogicalDNSRule{
- Mode: C.LogicalTypeOr,
- Rules: []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- },
- },
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- ResponseAnswer: badoption.Listable[option.DNSRecordOptions]{mustRecord(t, "example.com. IN A 1.1.1.1")},
- },
- },
- },
- },
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- },
- })
- require.NoError(t, err)
- }
- func TestLookupLegacyDNSModeDisabledReturnsRejectedErrorForRejectAction(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeReject,
- RejectOptions: option.RejectActionOptions{
- Method: C.RuleActionRejectMethodDefault,
- },
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.Nil(t, addresses)
- require.Error(t, err)
- require.True(t, rulepkg.IsRejected(err))
- }
- func TestExchangeLegacyDNSModeDisabledReturnsRefusedResponseForRejectAction(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeReject,
- RejectOptions: option.RejectActionOptions{
- Method: C.RuleActionRejectMethodDefault,
- },
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, mDNS.RcodeRefused, response.Rcode)
- require.Equal(t, []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)}, response.Question)
- }
- func TestExchangeLegacyDNSModeDisabledReturnsDropErrorForRejectDropAction(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeReject,
- RejectOptions: option.RejectActionOptions{
- Method: C.RuleActionRejectMethodDrop,
- },
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.Nil(t, response)
- require.ErrorIs(t, err, tun.ErrDrop)
- }
- func TestLookupLegacyDNSModeDisabledFiltersPerQueryTypeAddressesBeforeMerging(t *testing.T) {
- t.Parallel()
- defaultTransport := &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP}
- router := newTestRouter(t, []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypePredefined,
- PredefinedOptions: option.DNSRouteActionPredefined{
- Answer: badoption.Listable[option.DNSRecordOptions]{
- mustRecord(t, "example.com. IN A 1.1.1.1"),
- mustRecord(t, "example.com. IN AAAA 2001:db8::1"),
- },
- },
- },
- },
- },
- }, &fakeDNSTransportManager{
- defaultTransport: defaultTransport,
- transports: map[string]adapter.DNSTransport{
- "default": defaultTransport,
- },
- }, &fakeDNSClient{})
- require.False(t, router.legacyDNSMode)
- addresses, err := router.Lookup(context.Background(), "example.com", adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{
- netip.MustParseAddr("1.1.1.1"),
- netip.MustParseAddr("2001:db8::1"),
- }, addresses)
- }
- func TestExchangeLegacyDNSModeDisabledLogicalMatchResponseIPCIDRFallsThrough(t *testing.T) {
- t.Parallel()
- transportManager := &fakeDNSTransportManager{
- defaultTransport: &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- transports: map[string]adapter.DNSTransport{
- "upstream": &fakeDNSTransport{tag: "upstream", transportType: C.DNSTypeUDP},
- "selected": &fakeDNSTransport{tag: "selected", transportType: C.DNSTypeUDP},
- "default": &fakeDNSTransport{tag: "default", transportType: C.DNSTypeUDP},
- },
- }
- client := &fakeDNSClient{
- exchange: func(transport adapter.DNSTransport, message *mDNS.Msg) (*mDNS.Msg, error) {
- switch transport.Tag() {
- case "upstream":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("9.9.9.9")}, 60), nil
- case "selected":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("8.8.8.8")}, 60), nil
- case "default":
- return FixedResponse(0, message.Question[0], []netip.Addr{netip.MustParseAddr("4.4.4.4")}, 60), nil
- default:
- return nil, E.New("unexpected transport")
- }
- },
- }
- rules := []option.DNSRule{
- {
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeEvaluate,
- RouteOptions: option.DNSRouteActionOptions{Server: "upstream"},
- },
- },
- },
- {
- Type: C.RuleTypeLogical,
- LogicalOptions: option.LogicalDNSRule{
- RawLogicalDNSRule: option.RawLogicalDNSRule{
- Mode: C.LogicalTypeOr,
- Rules: []option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- MatchResponse: true,
- IPCIDR: badoption.Listable[string]{"1.1.1.0/24"},
- },
- },
- }},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "selected"},
- },
- },
- },
- }
- router := newTestRouter(t, rules, transportManager, client)
- response, err := router.Exchange(context.Background(), &mDNS.Msg{
- Question: []mDNS.Question{fixedQuestion("example.com", mDNS.TypeA)},
- }, adapter.DNSQueryOptions{})
- require.NoError(t, err)
- require.Equal(t, []netip.Addr{netip.MustParseAddr("4.4.4.4")}, MessageToAddresses(response))
- }
- func TestLegacyDNSModeReportsLegacyAddressFilterDeprecation(t *testing.T) {
- t.Parallel()
- manager := &fakeDeprecatedManager{}
- ctx := service.ContextWith[deprecated.Manager](context.Background(), manager)
- router := &Router{
- ctx: ctx,
- logger: log.NewNOPFactory().NewLogger("dns"),
- client: &fakeDNSClient{},
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- IPCIDR: badoption.Listable[string]{"1.1.1.0/24"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{Server: "default"},
- },
- },
- }})
- require.NoError(t, err)
- err = router.Start(adapter.StartStateStart)
- require.NoError(t, err)
- require.Len(t, manager.features, 1)
- require.Equal(t, deprecated.OptionLegacyDNSAddressFilter.Name, manager.features[0].Name)
- }
- func TestLegacyDNSModeReportsDNSRuleStrategyDeprecation(t *testing.T) {
- t.Parallel()
- manager := &fakeDeprecatedManager{}
- ctx := service.ContextWith[deprecated.Manager](context.Background(), manager)
- router := &Router{
- ctx: ctx,
- logger: log.NewNOPFactory().NewLogger("dns"),
- client: &fakeDNSClient{},
- defaultDomainStrategy: C.DomainStrategyAsIS,
- }
- err := router.Initialize([]option.DNSRule{{
- Type: C.RuleTypeDefault,
- DefaultOptions: option.DefaultDNSRule{
- RawDefaultDNSRule: option.RawDefaultDNSRule{
- Domain: badoption.Listable[string]{"example.com"},
- },
- DNSRuleAction: option.DNSRuleAction{
- Action: C.RuleActionTypeRoute,
- RouteOptions: option.DNSRouteActionOptions{
- Server: "default",
- Strategy: option.DomainStrategy(C.DomainStrategyIPv4Only),
- },
- },
- },
- }})
- require.NoError(t, err)
- err = router.Start(adapter.StartStateStart)
- require.NoError(t, err)
- require.Len(t, manager.features, 1)
- require.Equal(t, deprecated.OptionLegacyDNSRuleStrategy.Name, manager.features[0].Name)
- }
|