Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Limit modifications to the event payload #7255

Closed
yakkomajuri opened this issue Nov 22, 2021 · 3 comments
Closed

Limit modifications to the event payload #7255

yakkomajuri opened this issue Nov 22, 2021 · 3 comments

Comments

@yakkomajuri
Copy link
Contributor

Our ingestion pipeline currently overwrites, deletes, and updates values in the event payload before ingestion, such as $elements.

This makes things confusing, hard to debug, and potentially detrimental to exports, as the stored payload doesn't match the payload we accept as input.

An example of this is how we expect $elements to contain properties such as attr__class, which we convert to attr_class. However, if you send attr_class in the payload, we don't accept it :D

@yakkomajuri
Copy link
Contributor Author

#4448 is something that would also help tangentially

@posthog-bot
Copy link
Contributor

This issue hasn't seen activity in two years! If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in two weeks.

@posthog-bot
Copy link
Contributor

This issue was closed due to lack of activity. Feel free to reopen if it's still relevant.

@posthog-bot posthog-bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants