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

No one assigned

    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