Skip to content

Commit 31e40e9

Browse files
xueyurfmeta-codesync[bot]
authored andcommitted
Make response explicit in Thrift Policy Enforcer
Summary: As discussed in the [prev diff](https://www.internalfb.com/diff/D87361059?dst_version_fbid=905143925172710&transaction_fbid=2102626750543907), we want `$response` explicit for durable integration This diff: - pass `$response` as an argument of `genProcessResponse` - make privacylib take `$response` directly - legacy probes integration is intact and pending for privacylib migration Differential Revision: D88085026 fbshipit-source-id: bab0848a0a33c46a4daf213f0b1c78e5e63324f0
1 parent ad0db57 commit 31e40e9

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

third-party/thrift/src/thrift/lib/hack/src/ThriftServiceMethodNameVirtualPolicyEnforcer.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ private static function customLogging(
184184
string $policy_enforcer_api,
185185
PolicyEnforcerCallerIdentity $caller,
186186
PolicyEnforcerContext $context,
187+
mixed $response,
187188
): Awaitable<void> {
188189
// Pending Probes PrivacyLib post-read Integration
189190
await self::genExecuteStandaloneProbes(

0 commit comments

Comments
 (0)