Skip to content

Shivankar#213

Open
shivankar-singh wants to merge 5 commits intomainfrom
shivankar
Open

Shivankar#213
shivankar-singh wants to merge 5 commits intomainfrom
shivankar

Conversation

@shivankar-singh
Copy link
Contributor

Added API specs for default team members (GET, SET, DELETE)

shivankar-singh and others added 5 commits February 19, 2026 19:50
Introduces a new Guest user type to the five supported user roles, providing limited read access for external stakeholders or temporary access scenarios.

Co-authored-by: Cursor <cursoragent@cursor.com>
Introduces comprehensive documentation for Asset Policies feature, including:
- Introduction and use cases
- 3-step walkthrough with visual guides
- Policy execution tracking
- Action types and examples

Co-authored-by: Cursor <cursoragent@cursor.com>
Introduces comprehensive API documentation for managing default team members:
- GET endpoint to retrieve configured default members
- POST endpoint to set/replace default member list
- DELETE endpoint to remove specific or all default members
- Includes examples, validation rules, and error handling

Made-with: Cursor
@neo-by-projectdiscovery-dev
Copy link

neo-by-projectdiscovery-dev bot commented Mar 2, 2026

Neo - PR Security Review

No security issues found

Highlights

  • Adds OpenAPI specification for three new endpoints: GET, POST, and DELETE /v1/user/team/default-members
  • Includes comprehensive MDX documentation with usage examples, validation rules, and error handling
  • All endpoints require X-API-Key authentication and X-Team-Id header for team context
  • Documentation specifies owner/admin-only access control for all operations
Hardening Notes
  • Consider adding rate limiting documentation for the POST endpoint to prevent abuse of team member provisioning
  • Document maximum array size limits for the members array in SetDefaultTeamMembersRequest to prevent potential DoS via large payloads
  • Add explicit documentation about email validation patterns to help API consumers avoid 400 errors

Comment @neo help for available commands. · Open in Neo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant