search

fnproject / cli

CLI tool for fnproject.
https://fnproject.io
Apache License 2.0
132 stars 67 forks source link

fn update context allows arbitrary keys #439

Open jlbutler opened 6 years ago

jlbutler commented 6 years ago

Arbitrary keys may be added to a context via 'fn update context'. This may or may not be as intended, but it adds cruft where a use case is hard to discern. Possibly the call should check for the list of used / valid keys.

rikgibson commented 5 years ago

The namespace of available/valid keys really depends on the full range of context providers (admittedly only two at the moment) and then where do you go if you start validating keys to fn update context - do we start validating the context files? I get the point about cruft but invalid values just get ignored but I think we have bigger fish to fry.