fix(general): Do not remove empty cookies or headers (#138)

getsentry · Jan 11, 2019 · 1ba31fc · 1ba31fc
1 parent c846f83
commit 1ba31fc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/general/src/protocol/request.rs b/general/src/protocol/request.rs
@@ -216,17 +216,17 @@ pub struct Request {
 
     /// URL encoded contents of the Cookie header.
     #[metastructure(pii = "true", bag_size = "medium")]
-    #[metastructure(skip_serialization = "empty_deep")]
+    #[metastructure(skip_serialization = "empty")]
     pub cookies: Annotated<Cookies>,
 
     /// HTTP request headers.
     #[metastructure(pii = "true", bag_size = "large")]
-    #[metastructure(skip_serialization = "empty_deep")]
+    #[metastructure(skip_serialization = "empty")]
     pub headers: Annotated<Headers>,
 
     /// Server environment data, such as CGI/WSGI.
     #[metastructure(pii = "true", bag_size = "large")]
-    #[metastructure(skip_serialization = "empty_deep")]
+    #[metastructure(skip_serialization = "empty")]
     pub env: Annotated<Object<Value>>,
 
     /// The inferred content type of the request payload.

diff --git a/server/src/actors/project.rs b/server/src/actors/project.rs
@@ -733,10 +733,10 @@ impl Handler<FetchProjectState> for ProjectCache {
 
         match self.config.relay_mode() {
             RelayMode::Proxy => {
-                return Response::ok(ProjectStateResponse::local(ProjectState::allowed()))
+                return Response::ok(ProjectStateResponse::local(ProjectState::allowed()));
             }
             RelayMode::Static => {
-                return Response::ok(ProjectStateResponse::local(ProjectState::missing()))
+                return Response::ok(ProjectStateResponse::local(ProjectState::missing()));
             }
             RelayMode::Managed => {
                 // Proceed with loading the config from upstream