extern/bruno
1
0
Fork 1
mirror of https://github.com/usebruno/bruno.git synced 2025-01-05 13:39:13 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(#964): Allow "." in variable names + make error message more consistent

Browse Source
This commit is contained in:
Nelu Platonov 2023-11-15 16:27:14 +01:00
parent f617504cd6
commit 2ee6c5effc
No known key found for this signature in database
GPG Key ID: DC1D6718097E5B33
4 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/bruno-app/src/components/Environments/EnvironmentSettings/EnvironmentList/EnvironmentDetails/EnvironmentVariables/index.js
View File

@ -26,7 +26,7 @@ const EnvironmentVariables = ({ environment, collection }) => {
.required('Name cannot be empty') .required('Name cannot be empty')
.matches( .matches(
envVariableNameRegex, envVariableNameRegex,
'Name contains invalid characters. Must only contain alphanumeric characters, "-" and "_"' 'Name contains invalid characters. Must only contain alphanumeric characters, "-", "_", "." and cannot start with a digit.'
) )
.trim(), .trim(),
secret: Yup.boolean(), secret: Yup.boolean(),

7
packages/bruno-app/src/components/RequestPane/Vars/VarsTable/index.js
View File

@ -33,14 +33,9 @@ const VarsTable = ({ item, collection, vars, varType }) => {
case 'name': { case 'name': {
const value = e.target.value; const value = e.target.value;
if (/^(?!\d).*$/.test(value) === false) {
toast.error('Variable names must not start with a number!');
return;
}
if (envVariableNameRegex.test(value) === false) { if (envVariableNameRegex.test(value) === false) {
toast.error( toast.error(
'Variable contains invalid character! Variables must only contain alpha-numeric characters, "-" and "_".' 'Variable contains invalid characters! Variables must only contain alpha-numeric characters, "-", "_", "." and cannot start with a digit.'
); );
return; return;
} }

2
packages/bruno-app/src/utils/common/regex.js
View File

@ -1 +1 @@
export const envVariableNameRegex = /^(?!\d)[\w-]*$/; export const envVariableNameRegex = /^(?!\d)[\w-.]*$/;

6
packages/bruno-js/src/bru.js
View File

@ -1,7 +1,7 @@
const Handlebars = require('handlebars'); const Handlebars = require('handlebars');
const { cloneDeep } = require('lodash'); const { cloneDeep } = require('lodash');
const envVariableNameRegex = /^(?!\d)[\w-]*$/; const envVariableNameRegex = /^(?!\d)[\w-.]*$/;
class Bru { class Bru {
constructor(envVariables, collectionVariables, processEnvVars, collectionPath) { constructor(envVariables, collectionVariables, processEnvVars, collectionPath) {
@ -64,7 +64,7 @@ class Bru {
if (envVariableNameRegex.test(key) === false) { if (envVariableNameRegex.test(key) === false) {
throw new Error( throw new Error(
`Variable name: "${key}" contains invalid characters!` + `Variable name: "${key}" contains invalid characters!` +
' Names must only contain alpha-numeric characters, "-", "_" and cannot start with a digit.' ' Names must only contain alpha-numeric characters, "-", "_", "." and cannot start with a digit.'
); );
} }
@ -75,7 +75,7 @@ class Bru {
if (envVariableNameRegex.test(key) === false) { if (envVariableNameRegex.test(key) === false) {
throw new Error( throw new Error(
`Variable name: "${key}" contains invalid characters!` + `Variable name: "${key}" contains invalid characters!` +
' Names must only contain alpha-numeric characters and cannot start with a digit.' ' Names must only contain alpha-numeric characters, "-", "_", "." and cannot start with a digit.'
); );
} }