12 Commits (release-4.3)

Author	SHA1	Message	Date
Matthieu MOREL	96b315f188	chore: fix emptyStringTest, nestingReduce and singleCaseSwitch checks issues from gocritic ... Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	2 months ago
George Jenkins	81d6f04c82	Merge pull request #30867 from mmorel-35/gocritic ... chore: fix deprecatedComment issues from gocritic	3 months ago
Matthieu MOREL	83a0138172	chore: fix deprecatedComment issues from gocritic ... Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
George Jenkins	36dcc27ca3	fix: Plugin version path traversal ... Signed-off-by: George Jenkins <gvjenkins@gmail.com>	6 months ago
Matthieu MOREL	d4f6193a7e	chore(internal): enable perfsprint linter (#31871) ... #### Description enable perfsprint linter in internal/plugin Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
Benoit Tigeot	acf331a005	fix: improve plugin name validation err messages early via unmarshalling ... - Add strict YAML unmarshalling for v1 plugins (d.KnownFields) - Add comprehensive test coverage for validPluginName regex - Maintain backwards compatibility for legacy plugins Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	9e1e3d21c5	fix: Make invalid name error message more similar and move tests ... Follow Scott comment\ https://github.com/helm/helm/pull/31491/files#r2524933784 Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	cf077ceb27	fix: focus only on plugin name but give more info about what we get ... And improve test Follow Scott comment: > here I would have a list of valid and invalid names based on pattern, with a check and intended outcome for each one so that we comprehensively cover the rules. https://github.com/helm/helm/pull/31491/files#r2524820312 Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	f4b139a82c	Make validation error similar and explicit for both metadatas ... Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	c04e18e452	fix: improve plugin name validation error messages ... Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
George Jenkins	c35755a197	Remove legacy Command/Hooks from v1 Subprocess (#23) ... Signed-off-by: George Jenkins <gvjenkins@gmail.com>	1 year ago
Scott Rigby	be74ab72a0	[HIP-0026] Plugin runtime interface (#31145) ... * Runtime abstraction to encapsulate subprocess code and enable future runtimes Also fix race condition in TestPrepareCommandExtraArgs by replacing the shared variable modification with a local copy Co-authored-by: George Jenkins <gvjenkins@gmail.com> Signed-off-by: Scott Rigby <scott@r6by.com> * Remove commented out code Co-authored-by: Joe Julian <me@joejulian.name> Signed-off-by: Scott Rigby <scott@r6by.com> * Check test failure string Co-authored-by: Jesse Simpson <jesse.simpson36@gmail.com> Signed-off-by: Scott Rigby <scott@r6by.com> --------- Signed-off-by: Scott Rigby <scott@r6by.com> Co-authored-by: George Jenkins <gvjenkins@gmail.com> Co-authored-by: Joe Julian <me@joejulian.name> Co-authored-by: Jesse Simpson <jesse.simpson36@gmail.com>	1 year ago