feat: Add credential-level option to disable session recording and logging
Nessuno ha ancora preso questa issue.
- #11 di @snkrheadz — chiusa senza merge
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 72/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
- Ambito
- backend
Direzione di ricerca
Inizia in credentials/BrowserbaseApi.credentials.ts e individua la logica di esecuzione del nodo che costruisce le impostazioni della sessione Browserbase a partire dai valori delle credenziali e di Browser Options. Aggiungi i due campi opzionali delle credenziali con false come valori predefiniti e verifica che l’abilitazione di uno dei due forzi la disattivazione dell’impostazione corrispondente, mantenendo il comportamento attuale quando sono disabilitati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
When using the Browserbase node for back-office automation (e.g., expense processing, HR workflows, internal tools), browser sessions may display sensitive employee and business data. The current default behavior (recordSession: true, logSession: true) causes this data to be captured and stored on Browserbase servers for up to 30 days.
While individual workflow creators can toggle these options off in Browser Options, there is no way for an organization's n8n administrator to enforce this policy across all workflows using the same Browserbase credential.
Proposal
Add two optional boolean fields to BrowserbaseApi.credentials.ts that allow credential owners to force-disable session recording and logging for all workflows using that credential:
// credentials/BrowserbaseApi.credentials.ts
{
displayName: 'Disable Session Recording',
name: 'disableSessionRecording',
type: 'boolean',
default: false,
description: 'Force-disable session recording for all workflows using this credential. When enabled, the per-node "Record Session" option is ignored.',
},
{
displayName: 'Disable Session Logging',
name: 'disableSessionLogging',
type: 'boolean',
default: false,
description: 'Force-disable session logging for all workflows using this credential. When enabled, the per-node "Log Session" option is ignored.',
},
In the node execution logic:
const credentials = await this.getCredentials('browserbaseApi');
const browserSettings = {
recordSession: credentials.disableSessionRecording
? false
: (browserOptions.recordSession ?? true),
logSession: credentials.disableSessionLogging
? false
: (browserOptions.logSession ?? true),
// ...
};
Why credential-level?
- Admin control: In n8n, credentials are typically managed by admins/owners — regular workflow creators use shared credentials without editing them.
- Organization-wide enforcement: A single credential shared across multiple workflows applies the policy uniformly.
- No breaking changes: Defaults to
false, preserving existing behavior for all current users. - Works on n8n Cloud and self-hosted: No dependency on environment variables or instance-level settings.
Alternatives considered
| Approach | Pros | Cons |
|---|---|---|
Change defaults to false |
Simple | Users can re-enable; no enforcement |
| Environment variable | True admin control | Not available on n8n Cloud |
| Credential-level option | Admin control + Cloud compatible | Slightly unconventional use of credentials |
Context
We operate n8n Cloud for back-office automation (CS, Finance, HR departments) and encountered this issue when evaluating Browserbase for internal workflows. We're happy to submit a PR implementing this change if the approach is acceptable.
- Lingua principale
- TypeScript
- Stelle
- 5
- Fork
- 2
- Merge medio
- 2g 19h
- PR unite (30g)
- 2
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di browserbase/n8n-node
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
browserbase/n8n-node#22 ·
Tutte le issue di browserbase/n8n-node
Issue simili
-
submodule-pointer-regression
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
smith-horn/skillsmith#3061 ·
I maintainer di solito rispondono entro 1 giorno
-
area: ops type: test
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
accensa/x402-facilitator-stellar#559 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
cosimochellini/one-piece-zero-spoiler#551 ·
I maintainer di solito rispondono entro 1 giorno
-
getWatched() omits __proto__ directories when cwd is setForse già presa @maxazure l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
-
area:web enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno