You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks for the feedback! You're absolutely right - applying this filter to all responses could corrupt actual data containing a `properties` field.
6
+
7
+
I've updated the implementation to only sanitize OPTIONS requests that contain schema data. Regular GET/POST/etc responses are completely unaffected now.
8
+
9
+
---
10
+
11
+
## Reply to @westonruter
12
+
13
+
Thanks for the detailed review! I've implemented all three suggestions:
14
+
15
+
1. ✅ Simplified to single positive conditional check (no more multiple returns)
16
+
2. ✅ Removed redundant type validation in recursive function
17
+
3. ✅ Changed `new stdClass()` to `(object) array()` for consistency
18
+
19
+
The code is cleaner and more maintainable while preserving the same functionality.
0 commit comments