Skip to content

fix: close session after credential test and deprecate project ID field#8

Merged
Kylejeong2 merged 1 commit intomainfrom
kylejeong/gro-1207-fix-n8n-node-credentialing-ping-and-deprecate-project-id
Apr 2, 2026
Merged

fix: close session after credential test and deprecate project ID field#8
Kylejeong2 merged 1 commit intomainfrom
kylejeong/gro-1207-fix-n8n-node-credentialing-ping-and-deprecate-project-id

Conversation

@Kylejeong2
Copy link
Copy Markdown
Member

Summary

Close orphaned sessions on credential test: The credential "Test" button was calling POST /v1/sessions/start but never ending the session, leaving orphaned sessions on every test. Switched from the declarative ICredentialTestRequest to a programmatic ICredentialTestFunction that starts a session, extracts the sessionId, and immediately calls POST /v1/sessions/{sessionId}/end.

Deprecate browserbaseProjectId: The project ID field is no longer required by the API. Made the credential field optional (backward compatible — existing users who have it set will still send the header, new users can skip it).

@Kylejeong2 Kylejeong2 merged commit b68b4ec into main Apr 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants