Automattic / Automattic/jetpack

Review permission checks for both plugin and wpcom APIs

Open
#24,445 0 comments 0 reactions 0 assignees View on GitHub
[Feature] Search [Package] Search [Plugin] Search
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

Previously we had several issues regarding permission checks, user token, blog token etc. For example: https://github.com/Automattic/jetpack/pull/24099

We probably want to check all REST API our plugin provided have the right access control or using the right token to authenticate:

https://github.com/Automattic/jetpack/pull/24099

Also we need to check the WPCOM endpoints:

- `sites/{site}/jetpack_search/plan`
- `sites/{site}/jetpack_search/stats`
- etc

More information: p9dueE-55u#comment-7999

The outcome of the task would be whether they are okay, if not we'll need to file issues.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the linked pull request and the listed WPCOM endpoints: sites/{site}/jetpack_search/plan and sites/{site}/jetpack_search/stats. Check whether the plugin and WPCOM APIs use the appropriate permissions and user or blog tokens. Done means documenting which checks are correct and filing follow-up issues for any gaps.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
api, authentication, authorization, security
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.