Sync: add rest_api_allowed_public_metadata data to the whitelist.

Automattic / jetpack

Security, performance, marketing, and design tools — Jetpack is made by WordPress experts to make WP sites safer and faster, and help you grow your traffic.

https://jetpack.com/

Other

1.59k stars 798 forks source link

Sync: add rest_api_allowed_public_metadata data to the whitelist. #7099

Open enejb opened 7 years ago

enejb commented 7 years ago

Currently we don't sync the rest_api_allowed_public_metadata (we sync the whitelist) but not the data. We should add the info into the whitelist. So that we can rebuild the api endpoint in the future.

cc: @lezama Thoughts?

lezama commented 7 years ago

What do you mean by "the data"? we do sync it as a callable: https://github.com/Automattic/jetpack/blob/24ed0934bc8f62f7f00ea5dba0a2e52d02fec1f1/sync/class.jetpack-sync-defaults.php#L180

enejb commented 7 years ago

The post meta for example doesn't get synced even if it is maked as allowed public metadata.

stale[bot] commented 6 years ago

This issue has been marked as stale. This happened because:

It has been inactive in the past 6 months.
It hasn’t been labeled `[Pri] Blocker`, `[Pri] High`.

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.