wso2 / wso2/api-platform

Add PR check for policies in gateway-controllers

Open
#1,119 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Type/Task
Dominant language
Go
Stars
71
Forks
111
Avg merge
1d 14h
Merged PRs (30d)
110

Description

Description

Static check in PR validation

  • If a root property under parameters.properties is marked x-wso2-policy-advanced-param: true and is required, it must have a JSON Schema default.
  • If default is missing, static check fails.
  • Requiredness detection should include:
    • direct required: [...]
    • conditional requiredness through anyOf / allOf / if-then.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the gateway-controllers PR validation entry point and its existing schema checks. Trace how requiredness is detected, then verify the check covers direct required lists and anyOf/allOf/if-then cases; it is done when a required advanced policy parameter without a JSON Schema default fails validation and one with a default passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, json
Domain
api, ci-cd
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.