basecamp / basecamp/hotcell

Consider a server-side allowlist for operation arguments

Open
#19 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Ruby
Stars
23
Forks
0
Avg merge
11h 43m
Merged PRs (30d)
27

Description

The cell accepts the arguments the application assembled and refuses almost nothing. `Transforming#operations_for` refuses `combine_options` and hands everything else to `ImageProcessing`; the tool operations take raw argv.

Worth considering whether the cell should name what it accepts rather than pass through what it is given.

This revisits the argument-injection item the pre-release review raised and set aside, whose premise was that the payload comes from the trusted side.

No proposal yet, and no urgency.

Contributor guide

Open the contributing guide

Research direction

Start with Transforming#operations_for and trace how arguments reach ImageProcessing and the tool operations' raw argv. Read the pre-release review's argument-injection item for the prior premise and discussion. Done requires a decided server-side allowlist design, including what the cell accepts and how refused arguments are handled.

Written by the indexing model from the issue text.

Assessment

Tech stack
ruby
Domain
backend, security
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.