Merge lp:~gz/goamz/example_dns into lp:goamz
Proposed by
Martin Packman
Status: | Merged |
---|---|
Merged at revision: | 40 |
Proposed branch: | lp:~gz/goamz/example_dns |
Merge into: | lp:goamz |
Diff against target: |
29 lines (+5/-1) (has conflicts) 2 files modified
ec2/ec2t_test.go (+1/-1) ec2/ec2test/server.go (+4/-0) Text conflict in ec2/ec2test/server.go |
To merge this branch: | bzr merge lp:~gz/goamz/example_dns |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
goamz maintainers | Pending | ||
Review via email: mp+179717@code.launchpad.net |
Description of the change
Use testing.invalid rather than example.com
Stop using example.com addresses in testing code, which
may accidentally be used in a real lookup. Taking the
'.invalid' option from RFC 2606 means the resolve should
always fail locally.
To post a comment you must log in.
Reviewers: mp+179717_ code.launchpad. net,
Message:
Please take a look.
Description:
Use testing.invalid rather than example.com
Stop using example.com addresses in testing code, which
may accidentally be used in a real lookup. Taking the
'.invalid' option from RFC 2606 means the resolve should
always fail locally.
https:/ /code.launchpad .net/~gz/ goamz/example_ dns/+merge/ 179717
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/12765043/
Affected files: server. go
A [revision details]
M ec2/ec2t_test.go
M ec2/ec2test/
Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision: <email address hidden>
Index: ec2/ec2t_test.go inst.Instances[ 0].DNSName, Equals, 0].InstanceId+ ".example. com") inst.Instances[ 0].DNSName, Equals, 0].InstanceId+ ".testing. invalid" )
=== modified file 'ec2/ec2t_test.go'
--- ec2/ec2t_test.go 2013-01-31 17:06:10 +0000
+++ ec2/ec2t_test.go 2013-08-12 14:33:16 +0000
@@ -68,7 +68,7 @@
})
c.Assert(err, IsNil)
c.Assert(inst, NotNil)
- c.Assert(
inst.Instances[
+ c.Assert(
inst.Instances[
id := inst.Instances[ 0].InstanceId
Index: ec2/ec2test/ server. go server. go' server. go 2013-01-31 17:06:10 +0000 server. go 2013-08-12 14:33:16 +0000 "%s.example. com", inst.id), "%s.testing. invalid" , inst.id),
=== modified file 'ec2/ec2test/
--- ec2/ec2test/
+++ ec2/ec2test/
@@ -548,7 +548,7 @@
InstanceId: inst.id,
InstanceType: inst.instType,
ImageId: inst.imageId,
- DNSName: fmt.Sprintf(
+ DNSName: fmt.Sprintf(
// TODO the rest
}
}