Posthog: drop $initial_person_info from outgoing events

[fkwp]

The $initial_person_info super-property is auto-set by posthog-js. Drop it for increased privacy.

[BillCarsonFr]

We could have make our life simpler by making sanitize an public utility method for testing.

By the way the sanitize_properties appears to be deprecated https://github.com/PostHog/posthog-js/blob/5b4eda14c99bed660bc31fd863dfcecaa7cde1e4/packages/types/src/posthog-config.ts#L1393

/** @deprecated - use `before_send` instead */

Do we have a real life capture showing it is still working?

[fkwp]

We could have make our life simpler by making sanitize an public utility method for testing.

By the way the sanitize_properties appears to be deprecated https://github.com/PostHog/posthog-js/blob/5b4eda14c99bed660bc31fd863dfcecaa7cde1e4/packages/types/src/posthog-config.ts#L1393

/** @deprecated - use `before_send` instead */

Do we have a real life capture showing it is still working?

fair point. bumped posthog and refactoring is here: a08a561

still added an "end-to-end" posthog test to verify before_send does actually what we expect it to do.

[toger5]

Lets wait for the merge until we see a posthog event form this PR in the dashboard.