Opened 6 months ago

Last modified 6 months ago

#6991 new change

Stop normalizing domain arguments in filter classes

Reported by: mjethani Assignee: mjethani
Priority: P2 Milestone:
Module: Core Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description

Background

In the isActiveOnDomain method of each ActiveFilter object, the parameter docDomain is lowercased and any trailing dots are removed. This is done for each filter. Most of the time, the caller either already has a lowercased domain with no trailing dot, or can do this transformation once on its own (calling isActiveOnDomain on each filter in a subscription, for example).

If this transformation is removed from the implementation of the function, it significantly improves the performance.

What to change

Remove the lowercasing and removal of any trailing dots from the implementation of isActiveOnDomain. Make sure callers are doing this on their own. Update the tests.

Change History (3)

comment:1 Changed 6 months ago by mjethani

  • Summary changed from Stop lowercasing and removing trailing dots in isActiveOnDomain to Stop lowercasing and removing trailing dots domain in filter classes

comment:2 Changed 6 months ago by mjethani

  • Summary changed from Stop lowercasing and removing trailing dots domain in filter classes to Stop normalizing domain arguments in filter classes

comment:3 Changed 6 months ago by mjethani

  • Ready unset
Note: See TracTickets for help on using tickets.