Skip to content

Overly conservative check and deduct for budget #336

@bmcase

Description

@bmcase

The current text in [=deduct privacy budget=] function is a bit overly conservative when checking and deducting budget

  1. If |deductionFp| is negative or greater than [=maximum epsilon=], [=map/set|set=] the value of |key| in the [=privacy budget store=] to 0
    and return false.

Also

  1. If |deduction| is greater than |currentValue|, [=map/set|set=] the value of |key| in the [=privacy budget store=] to 0
    and return false.

If someone asks for a really big epsilon, you don’t have to both zero out their budget and give them a null report. You can just give them a null report and hope they figure out how this API works to come back and spend their budget properly…

I think I'll end up addressing this in #309 as need to also make it such that privacy budget deduction occurs if and only if all safety limit deductions can be made, otherwise just drop the epoch's impressions from consideration in this measureConversion request.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions