fix(deps): Update maxminddb (#5432)

Author: jjbayer

Cargo.lock

@@ -133,7 +133,7 @@ json-forensics = "0.1.1"
 libc = "0.2.177"
 liblzma = "0.4.5"
 lru = "0.16.1"
-maxminddb = "0.26.0"
+maxminddb = "0.27.0"
 md5 = "0.8.0"
 memchr = "2.7.6"
 mimalloc = { version = "0.1.48", features = ["v3"] }
@@ -180,7 +180,9 @@ sentry = { version = "0.41.0", default-features = false, features = [
 ] }
 sentry-core = "0.41.0"
 sentry-kafka-schemas = { version = "2.1.14", default-features = false }
-sentry-release-parser = { version = "1.4.0", default-features = false, features = ["semver-1"] }
+sentry-release-parser = { version = "1.4.0", default-features = false, features = [
+  "semver-1",
+] }
 sentry-types = "0.41.0"
 sentry_protos = "0.4.2"
 serde = { version = "=1.0.228", features = ["derive", "rc"] }

Cargo.toml

@@ -28,7 +28,7 @@ impl GeoIpLookup {
         P: AsRef<Path>,
     {
         #[cfg(feature = "mmap")]
-        let reader = maxminddb::Reader::open_mmap(path)?;
+        let reader = unsafe { maxminddb::Reader::open_mmap(path)? };
         #[cfg(not(feature = "mmap"))]
         let reader = maxminddb::Reader::open_readfile(path)?;
         Ok(GeoIpLookup(Some(Arc::new(reader))))
@@ -45,37 +45,22 @@ impl GeoIpLookup {
             return Ok(None);
         };
 
-        let city: maxminddb::geoip2::City = match reader.lookup(ip_address) {
-            Ok(Some(x)) => x,
-            Ok(None) => return Ok(None),
-            Err(e) => return Err(e),
+        let Some(city) = reader
+            .lookup(ip_address)?
+            .decode::<maxminddb::geoip2::City>()?
+        else {
+            return Ok(None);
         };
 
         Ok(Some(Geo {
-            country_code: Annotated::from(
-                city.country
-                    .as_ref()
-                    .and_then(|country| Some(country.iso_code.as_ref()?.to_string())),
-            ),
-            city: Annotated::from(
-                city.city
-                    .as_ref()
-                    .and_then(|city| Some(city.names.as_ref()?.get("en")?.to_string())),
-            ),
-            subdivision: Annotated::from(city.subdivisions.as_ref().and_then(|subdivisions| {
-                subdivisions.first().and_then(|subdivision| {
-                    subdivision.names.as_ref().and_then(|subdivision_names| {
-                        subdivision_names
-                            .get("en")
-                            .map(|subdivision_name| subdivision_name.to_string())
-                    })
-                })
-            })),
-            region: Annotated::from(
-                city.country
-                    .as_ref()
-                    .and_then(|country| Some(country.names.as_ref()?.get("en")?.to_string())),
+            country_code: Annotated::from(city.country.iso_code.map(String::from)),
+            city: Annotated::from(city.city.names.english.map(String::from)),
+            subdivision: Annotated::from(
+                city.subdivisions
+                    .first()
+                    .and_then(|subdivision| subdivision.names.english.map(String::from)),
             ),
+            region: Annotated::from(city.country.names.english.map(String::from)),
             ..Default::default()
         }))
     }