CERBERUS/LibreChat
1
0
Fork 0
mirror of https://github.com/danny-avila/LibreChat.git synced 2026-08-27 04:07:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 13

🪢 feat: add Langfuse session links (#14776)

Browse source 
* feat: add Langfuse session links

* fix: tighten Langfuse session link resolution

* fix: clear stale Langfuse session links

* test: verify tenant Langfuse session links

* fix: align Langfuse link with client conventions
This commit is contained in:
Ravi Kumar L 2026-08-13 04:25:23 +02:00 • committed by GitHub
parent 5ff282f900
commit 9980b6221f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
14 changed files with 377 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

2
api/server/routes/admin/langfuse.js
View file

@ -38,12 +38,14 @@ const handlers = createAdminLangfuseHandlers({
findConfigByPrincipal: db.findConfigByPrincipal,
patchConfigFields: db.patchConfigFields,
toggleConfigActive: db.toggleConfigActive,
getMessages: db.getMessages,
invalidateConfigCaches,
});
router.use(requireJwtAuth, requireAdminAccess, requireLangfuseManage);
router.get('/connection', handlers.getConnection);
router.get('/connection/session/:conversationId', handlers.getSessionLink);
router.put('/connection', handlers.updateConnection);
router.post('/connection/test', handlers.testConnection);

22
api/server/routes/admin/langfuse.test.js
View file

@ -19,6 +19,7 @@ const mockRequireCapability = jest.fn((capability) => (req, res, next) => {
});
const mockHandlers = {
getConnection: jest.fn((_req, res) => res.status(200).json({ handler: 'get' })),
getSessionLink: jest.fn((_req, res) => res.status(200).json({ handler: 'session' })),
updateConnection: jest.fn((_req, res) => res.status(200).json({ handler: 'update' })),
testConnection: jest.fn((_req, res) => res.status(200).json({ handler: 'test' })),
};
@ -48,6 +49,7 @@ jest.mock('~/models', () => ({
findConfigByPrincipal: jest.fn(),
patchConfigFields: jest.fn(),
toggleConfigActive: jest.fn(),
getMessages: jest.fn(),
}));
describe('admin Langfuse routes', () => {
@ -85,24 +87,32 @@ describe('admin Langfuse routes', () => {
});
it.each([
['GET', '/api/admin/langfuse/connection/session/conversation-1', 'getSessionLink'],
['PUT', '/api/admin/langfuse/connection', 'updateConnection'],
['POST', '/api/admin/langfuse/connection/test', 'testConnection'],
])('requires Langfuse manage access for %s %s', async (method, path, handlerName) => {
const app = createApp();
const response = await request(app)[method.toLowerCase()](path).send({}).expect(200);
const expectedHandlers = {
getSessionLink: 'session',
updateConnection: 'update',
testConnection: 'test',
};
expect(response.body).toEqual({
handler: handlerName === 'updateConnection' ? 'update' : 'test',
});
expect(response.body).toEqual({ handler: expectedHandlers[handlerName] });
expect(middlewareCalls).toEqual(['jwt', 'access:admin']);
expect(mockHandlers[handlerName]).toHaveBeenCalledTimes(1);
});
it('blocks updates when the user lacks Langfuse manage access', async () => {
it.each([
['GET', '/api/admin/langfuse/connection/session/conversation-1', 'getSessionLink'],
['PUT', '/api/admin/langfuse/connection', 'updateConnection'],
['POST', '/api/admin/langfuse/connection/test', 'testConnection'],
])('blocks %s %s without Langfuse manage access', async (method, path, handlerName) => {
canManageLangfuse = false;
await request(createApp()).put('/api/admin/langfuse/connection').send({}).expect(403);
await request(createApp())[method.toLowerCase()](path).send({}).expect(403);
expect(mockHandlers.updateConnection).not.toHaveBeenCalled();
expect(mockHandlers[handlerName]).not.toHaveBeenCalled();
});
});