Nail/fission-console
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: валидация имён env vars — только [A-Za-z_][A-Za-z0-9_]*

Browse Source
This commit is contained in:
“Naeel”
2026-05-09 07:22:35 +04:00
parent ee97eebbff
commit 089994256c
3 changed files with 15 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
console/deploy/console.yaml
+1 -1
View File
@@ -55,7 +55,7 @@ spec:
serviceAccountName: fission-console serviceAccountName: fission-console
containers: containers:
- name: console - name: console
image: naeel/fission-console:v1.3.75 image: naeel/fission-console:v1.3.76
imagePullPolicy: Always imagePullPolicy: Always
ports: ports:
- containerPort: 8090 - containerPort: 8090
console/ui/index.html
+2 -2
View File
@@ -102,7 +102,7 @@
<div class="nubes">NUBES</div> <div class="nubes">NUBES</div>
<div class="product">FISSION CONSOLE</div> <div class="product">FISSION CONSOLE</div>
</div> </div>
<div style="font-size:0.65rem; color:var(--text-secondary); margin-left:10px; align-self:center; opacity:0.7;">v1.3.75</div> <div style="font-size:0.65rem; color:var(--text-secondary); margin-left:10px; align-self:center; opacity:0.7;">v1.3.76</div>
</div> </div>
<div class="row" style="margin:0;"> <div class="row" style="margin:0;">
<button class="btn ghost" onclick="reloadAll()">Refresh</button> <button class="btn ghost" onclick="reloadAll()">Refresh</button>
@@ -510,7 +510,7 @@
</div> </div>
<div class="actions" style="justify-content:space-between; align-items:center;"> <div class="actions" style="justify-content:space-between; align-items:center;">
<span style="font-size:0.75rem; color:var(--text-secondary);">v1.3.75</span> <span style="font-size:0.75rem; color:var(--text-secondary);">v1.3.76</span>
<button class="btn ghost" onclick="closeHelp()">Закрыть</button> <button class="btn ghost" onclick="closeHelp()">Закрыть</button>
</div> </div>
</div> </div>
console/ui/js/functions.js
+12 -4
View File
@@ -250,6 +250,11 @@ async function submitEdit() {
var isTfFn = /^tf-/.test(name); var isTfFn = /^tf-/.test(name);
if (!isTfFn) { if (!isTfFn) {
var envVars = collectEnvVars('e'); var envVars = collectEnvVars('e');
// Проверка формата ключей
var invalidKeys = envVars.map(function(ev) { return ev.name; }).filter(function(k) { return !ENV_KEY_RE.test(k); });
if (invalidKeys.length > 0) {
throw new Error('Недопустимые имена переменных: ' + invalidKeys.join(', ') + ' (только латиница, цифры, _, не с цифры)');
}
// Проверка на дубли ключей // Проверка на дубли ключей
var keys = envVars.map(function(ev) { return ev.name; }); var keys = envVars.map(function(ev) { return ev.name; });
var dupes = keys.filter(function(k, i) { return keys.indexOf(k) !== i; }); var dupes = keys.filter(function(k, i) { return keys.indexOf(k) !== i; });
@@ -318,7 +323,7 @@ function makeEnvVarRow(prefix, key, val, readOnly, idx) {
row.dataset.envIdx = idx; row.dataset.envIdx = idx;
var kInput = document.createElement('input'); var kInput = document.createElement('input');
kInput.placeholder = 'KEY'; kInput.placeholder = 'KEY (A-Z, a-z, 0-9, _)';
kInput.value = key; kInput.value = key;
kInput.disabled = readOnly; kInput.disabled = readOnly;
kInput.style.cssText = 'flex:1; font-size:12px; font-family:monospace;'; kInput.style.cssText = 'flex:1; font-size:12px; font-family:monospace;';
@@ -354,7 +359,9 @@ function makeEnvVarRow(prefix, key, val, readOnly, idx) {
return row; return row;
} }
// highlightDupeKeys подсвечивает красным все поля KEY с одинаковыми именами var ENV_KEY_RE = /^[A-Za-z_][A-Za-z0-9_]*$/;
// highlightDupeKeys подсвечивает красным все поля KEY с одинаковыми именами или невалидным именем
function highlightDupeKeys(prefix) { function highlightDupeKeys(prefix) {
var list = document.getElementById(prefix + '-envvars-list'); var list = document.getElementById(prefix + '-envvars-list');
if (!list) return; if (!list) return;
@@ -363,8 +370,9 @@ function highlightDupeKeys(prefix) {
inputs.forEach(function(inp) { inputs.forEach(function(inp) {
var k = inp.value.trim(); var k = inp.value.trim();
var isDupe = k !== '' && keys.filter(function(x) { return x === k; }).length > 1; var isDupe = k !== '' && keys.filter(function(x) { return x === k; }).length > 1;
inp.style.outline = isDupe ? '2px solid #f44' : ''; var isInvalid = k !== '' && !ENV_KEY_RE.test(k);
inp.title = isDupe ? 'Дублирующийся ключ!' : ''; inp.style.outline = (isDupe || isInvalid) ? '2px solid #f44' : '';
inp.title = isDupe ? 'Дублирующийся ключ!' : isInvalid ? 'Только латиница, цифры и _. Не начинается с цифры.' : '';
}); });
} }