fix: Locator parsing fixes

- Fixes #169 where the output of where values are stored is wrong.
- Fixes #170 where the ignored value is truncated when it should not be.

id.go

@@ -47,7 +47,7 @@ var (
 
 	// LocatorPattern is the precompiled regular expression that all locators must match.
 	LocatorPattern = regexp.MustCompile(
-		`^(?P<scheme>(?i)mac|uuid|dns|serial|event|self):(?P<authority>[^/]+)?(?P<service>/[^/]+)?(?P<ignored>/[^/]+)?`,
+		`^(?P<scheme>(?i)mac|uuid|dns|serial|event|self):(?P<authority>[^/]+)?(?P<service>/[^/]+)?(?P<ignored>.+)?`,
 	)
 )
 
@@ -152,10 +152,18 @@ func ParseLocator(locator string) (Locator, error) {
 
 	// If the locator is a device identifier, then we need to parse it.
 	switch l.Scheme {
-	case SchemeDNS, SchemeEvent:
+	case SchemeDNS:
 		if l.Authority == "" {
 			return Locator{}, ErrorInvalidLocator
 		}
+	case SchemeEvent:
+		if l.Authority == "" {
+			return Locator{}, ErrorInvalidLocator
+		}
+		if l.Service != "" {
+			l.Ignored = "/" + l.Service + l.Ignored
+			l.Service = ""
+		}
 	case SchemeMAC, SchemeUUID, SchemeSerial, SchemeSelf:
 		id, err := makeDeviceID(l.Scheme, l.Authority)
 		if err != nil {
@@ -187,10 +195,10 @@ func (l Locator) String() string {
 	if l.Service != "" {
 		buf.WriteString("/")
 		buf.WriteString(l.Service)
+	}
 
-		if l.Ignored != "" {
-			buf.WriteString(l.Ignored)
-		}
+	if l.Ignored != "" {
+		buf.WriteString(l.Ignored)
 	}
 
 	return buf.String()

id_test.go

@@ -142,22 +142,23 @@ func TestParseLocator(t *testing.T) {
 			},
 		}, {
 			description: "locator with service",
-			locator:     "DNS:foo.bar.com/service",
-			str:         "dns:foo.bar.com/service",
+			locator:     "DNS:foo.bar.com/service/ignored/really/really/ignored",
+			str:         "dns:foo.bar.com/service/ignored/really/really/ignored",
 			want: Locator{
 				Scheme:    SchemeDNS,
 				Authority: "foo.bar.com",
 				Service:   "service",
+				Ignored:   "/ignored/really/really/ignored",
 			},
 		}, {
 			description: "locator with service everything",
-			locator:     "event:something/service/ignored",
-			str:         "event:something/service/ignored",
+			locator:     "event:event_name/ignored/really/really/ignored",
+			str:         "event:event_name/ignored/really/really/ignored",
 			want: Locator{
 				Scheme:    SchemeEvent,
-				Authority: "something",
-				Service:   "service",
-				Ignored:   "/ignored",
+				Authority: "event_name",
+				Service:   "",
+				Ignored:   "/ignored/really/really/ignored",
 			},
 		}, {
 			description: "self locator with service",
@@ -170,12 +171,12 @@ func TestParseLocator(t *testing.T) {
 			},
 		}, {
 			description: "self locator with service everything",
-			locator:     "self:/service/ignored",
-			str:         "self:/service/ignored",
+			locator:     "self:/service/ignored/really/really/ignored",
+			str:         "self:/service/ignored/really/really/ignored",
 			want: Locator{
 				Scheme:  SchemeSelf,
 				Service: "service",
-				Ignored: "/ignored",
+				Ignored: "/ignored/really/really/ignored",
 				ID:      "self:",
 			},
 		},