fix: check if geo is configured before making calls #19

LaurenceJJones · 2024-12-27T11:50:01Z

fix: #18

LaurenceJJones · 2024-12-27T12:16:05Z

internal/api/root.go

 				val := net.ParseIP(args[0])

 				if val == nil {
 					return nil, fmt.Errorf("invalid IP")
 				}
+
 				record, err := a.GeoDatabase.GetCity(&val)

 				if err != nil && !errors.Is(err, geo.NotValidConfig) {


Most likely can remove the error check clause since it doesnt return this case anymore.

fix: check if geo is configured before making calls

1a54c1f

LaurenceJJones commented Dec 27, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: check if geo is configured before making calls #19

fix: check if geo is configured before making calls #19

LaurenceJJones commented Dec 27, 2024

LaurenceJJones Dec 27, 2024

fix: check if geo is configured before making calls #19

Are you sure you want to change the base?

fix: check if geo is configured before making calls #19

Conversation

LaurenceJJones commented Dec 27, 2024

LaurenceJJones Dec 27, 2024

Choose a reason for hiding this comment