comparing feature/ipv6 and feature/tangled-ci on seiso.moe/magna

+30 -1

resource_record.go

···

       26
       26
        
       		return nil, fmt.Errorf("A record: cannot encode non-IPv4 address %s", a.Address.String())

     

       27
       27
        
       	}

     

       28
       28
        
       

     

       29
       29
       -
       	return append(bytes, a.Address.To4()...), nil

     

       29
       29
       +
       	return append(bytes, ipv4...), nil

     

       30
       30
        
       }

     

       31
       31
        
       

     

       32
       32
        
       func (a A) String() string {

     
···

       751
       751
        
       	return fmt.Sprintf("%d %s", rt.Preference, rt.IntermediateHost)

     

       752
       752
        
       }

     

       753
       753
        
       

     

       754
       754
       +
       func (a *AAAA) Decode(buf []byte, offset int, rdlength int) (int, error) {

     

       755
       755
       +
       	bytes, offset, err := getSlice(buf, offset, rdlength)

     

       756
       756
       +
       	if err != nil {

     

       757
       757
       +
       		return offset, fmt.Errorf("AAAA record: failed to read address data: %w", err)

     

       758
       758
       +
       	}

     

       759
       759
       +
       

     

       760
       760
       +
       	a.Address = net.IP(bytes)

     

       761
       761
       +
       	if a.Address.To16() == nil {

     

       762
       762
       +
       		return offset, fmt.Errorf("AAAA record: decoded data is not a valid IPv6 address: %v", bytes)

     

       763
       763
       +
       	}

     

       764
       764
       +
       	return offset, nil

     

       765
       765
       +
       }

     

       766
       766
       +
       

     

       767
       767
       +
       func (a *AAAA) Encode(bytes []byte, offsets *map[string]uint16) ([]byte, error) {

     

       768
       768
       +
       	ipv6 := a.Address.To16()

     

       769
       769
       +
       	// XXX: ipv6 encodes ipv4 so need to check its not ipv4

     

       770
       770
       +
       	if a.Address.To4() != nil || ipv6 == nil {

     

       771
       771
       +
       		return nil, fmt.Errorf("AAAA record: cannot encode non-IPv6 address %s", a.Address.String())

     

       772
       772
       +
       	}

     

       773
       773
       +
       

     

       774
       774
       +
       	return append(bytes, ipv6...), nil

     

       775
       775
       +
       }

     

       776
       776
       +
       

     

       777
       777
       +
       func (a AAAA) String() string {

     

       778
       778
       +
       	return a.Address.String()

     

       779
       779
       +
       }

     

       780
       780
       +
       

     

       754
       781
        
       func (opt *OPT) Decode(buf []byte, offset int, rdlength int) (int, error) {

     

       755
       782
        
       	// s := offset

     

       756
       783
        
       	opt.Options = make([]EDNSOption, 0)

     
···

       915
       942
        
       		r.RData = &ISDN{}

     

       916
       943
        
       	case 21:

     

       917
       944
        
       		r.RData = &RT{}

     

       945
       945
       +
       	case 28:

     

       946
       946
       +
       		r.RData = &AAAA{}

     

       918
       947
        
       	case 41:

     

       919
       948
        
       		r.RData = &OPT{}

     

       920
       949
        
       	default:

+30

resource_record_test.go

···

       516
       516
        
       		})

     

       517
       517
        
       	}

     

       518
       518
        
       }

     

       519
       519
       +
       

     

       520
       520
       +
       func TestAAAARecord(t *testing.T) {

     

       521
       521
       +
       	addr := net.ParseIP("2001:db8::1")

     

       522
       522
       +
       	rdataBytes := []byte(addr)

     

       523
       523
       +
       	aaaa := &AAAA{}

     

       524
       524
       +
       

     

       525
       525
       +
       	offset, err := aaaa.Decode([]byte{}, 0, 16)

     

       526
       526
       +
       	assert.Error(t, err, "Decode should fail with empty buffer")

     

       527
       527
       +
       	assert.True(t, errors.Is(err, &BufferOverflowError{}))

     

       528
       528
       +
       

     

       529
       529
       +
       	offset, err = aaaa.Decode(rdataBytes, 0, 16)

     

       530
       530
       +
       	assert.NoError(t, err)

     

       531
       531
       +
       	assert.Equal(t, 16, offset)

     

       532
       532
       +
       	assert.Equal(t, addr, aaaa.Address)

     

       533
       533
       +
       

     

       534
       534
       +
       	_, err = aaaa.Decode([]byte{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15}, 0, 15)

     

       535
       535
       +
       	assert.Error(t, err)

     

       536
       536
       +
       	assert.Contains(t, err.Error(), "AAAA record:")

     

       537
       537
       +
       

     

       538
       538
       +
       	aaaaEncode := &AAAA{Address: addr}

     

       539
       539
       +
       	encoded := encodeRData(t, aaaaEncode)

     

       540
       540
       +
       	assert.Equal(t, rdataBytes, encoded)

     

       541
       541
       +
       

     

       542
       542
       +
       	ipv4 := net.ParseIP("192.168.1.1")

     

       543
       543
       +
       	aaaaEncodeInvalid := &AAAA{Address: ipv4}

     

       544
       544
       +
       	_, err = aaaaEncodeInvalid.Encode([]byte{}, &map[string]uint16{})

     

       545
       545
       +
       

     

       546
       546
       +
       	assert.Error(t, err)

     

       547
       547
       +
       	assert.Contains(t, err.Error(), "cannot encode non-IPv6 address")

     

       548
       548
       +
       }

+7

types.go

···

       92
       92
        
       	X25Type           = 19

     

       93
       93
        
       	ISDNType          = 20

     

       94
       94
        
       	RTType            = 21

     

       95
       95
       +
       	AAAAType          = 28

     

       95
       96
        
       

     

       96
       97
        
       	OPTType = 41

     

       97
       98
        
       

     
···

       145
       146
        
       		return "ISDN"

     

       146
       147
        
       	case RTType:

     

       147
       148
        
       		return "RT"

     

       149
       149
       +
       	case AAAAType:

     

       150
       150
       +
       		return "AAAA"

     

       148
       151
        
       	case OPTType:

     

       149
       152
        
       		return "OPT"

     

       150
       153
        
       	case AXFRType:

     
···

       375
       378
        
       	IntermediateHost string

     

       376
       379
        
       }

     

       377
       380
        
       

     

       381
       381
       +
       type AAAA struct {

     

       382
       382
       +
       	Address net.IP

     

       383
       383
       +
       }

     

       384
       384
       +
       

     

       378
       385
        
       type EDNSOption struct {

     

       379
       386
        
       	Code uint16

     

       380
       387
        
       	Data []byte

+7 -1

.tangled/workflows/lint.yml

···

       1
       1
        
       when:

     

       2
       2
       -
         - event: ["push", "pull_request"]

     

       2
       2
       +
         - event: ["push", "pull_request", "manual"]

     

       3
       3
        
           branch: ["main"]

     

       4
       4
        
       

     

       5
       5
       +
       engine: "nixery"

     

       6
       6
       +
       

     

       7
       7
       +
       clone:

     

       8
       8
       +
         depth: 1

     

       9
       9
       +
         submodules: false

     

       10
       10
       +
       

     

       5
       11
        
       dependencies:

     

       6
       12
        
         nixpkgs:

     

       7
       13
        
           - go

+7 -2

.tangled/workflows/test.yml

···

       1
       1
        
       when:

     

       2
       2
       -
         - event: ["push", "pull_request"]

     

       2
       2
       +
         - event: ["push", "pull_request", "manual"]

     

       3
       3
        
           branch: ["main"]

     

       4
       4
       -
         - event: ["manual"]

     

       4
       4
       +
       

     

       5
       5
       +
       engine: "nixery"

     

       6
       6
       +
       

     

       7
       7
       +
       clone:

     

       8
       8
       +
         depth: 1

     

       9
       9
       +
         submodules: false

     

       5
       10
        
       

     

       6
       11
        
       dependencies:

     

       7
       12
        
         nixpkgs:

+18

.tangled/workflows/staticcheck.yml

···

       1
       1
       +
       when:

     

       2
       2
       +
         - event: ["push", "pull_request", "manual"]

     

       3
       3
       +
           branch: ["main"]

     

       4
       4
       +
       

     

       5
       5
       +
       engine: "nixery"

     

       6
       6
       +
       

     

       7
       7
       +
       clone:

     

       8
       8
       +
         depth: 1

     

       9
       9
       +
         submodules: false

     

       10
       10
       +
       

     

       11
       11
       +
       dependencies:

     

       12
       12
       +
         nixpkgs:

     

       13
       13
       +
           - go

     

       14
       14
       +
       

     

       15
       15
       +
       steps:

     

       16
       16
       +
         - name: "staticcheck"

     

       17
       17
       +
           command: |

     

       18
       18
       +
              go run honnef.co/go/tools/cmd/staticcheck@latest

Compare changes