Mathis/lemma
1
0
Fork 0
mirror of https://github.com/lordmathis/lemma.git synced 2025-11-06 07:54:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #62 from lordmathis/fix/image-preview

Browse Source 
Fix get image url
This commit is contained in:
Matúš Námešný
2025-10-11 22:56:49 +02:00
committed by GitHub
parent f101376bef ec89f95880
commit e82f25a2ed
3 changed files with 20 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
app/src/utils/fileHelpers.test.ts
View File

@@ -83,7 +83,7 @@ describe('fileHelpers', () => {
const filePath = 'folder/file.md'; const filePath = 'folder/file.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/my-workspace/files/folder%2Ffile.md'; 'http://localhost:8080/api/v1/workspaces/my-workspace/files/content?file_path=folder%2Ffile.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -94,7 +94,7 @@ describe('fileHelpers', () => {
const filePath = 'file.md'; const filePath = 'file.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/my%20workspace%20with%20spaces/files/file.md'; 'http://localhost:8080/api/v1/workspaces/my%20workspace%20with%20spaces/files/content?file_path=file.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -105,7 +105,7 @@ describe('fileHelpers', () => {
const filePath = 'folder with spaces/file with spaces.md'; const filePath = 'folder with spaces/file with spaces.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/workspace/files/folder%20with%20spaces%2Ffile%20with%20spaces.md'; 'http://localhost:8080/api/v1/workspaces/workspace/files/content?file_path=folder%20with%20spaces%2Ffile%20with%20spaces.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -116,7 +116,7 @@ describe('fileHelpers', () => {
const filePath = 'file?name=test.md'; const filePath = 'file?name=test.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/test%26workspace/files/file%3Fname%3Dtest.md'; 'http://localhost:8080/api/v1/workspaces/test%26workspace/files/content?file_path=file%3Fname%3Dtest.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -127,7 +127,7 @@ describe('fileHelpers', () => {
const filePath = 'ファイル.md'; const filePath = 'ファイル.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/%E3%83%97%E3%83%AD%E3%82%B8%E3%82%A7%E3%82%AF%E3%83%88/files/%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB.md'; 'http://localhost:8080/api/v1/workspaces/%E3%83%97%E3%83%AD%E3%82%B8%E3%82%A7%E3%82%AF%E3%83%88/files/content?file_path=%E3%83%95%E3%82%A1%E3%82%A4%E3%83%AB.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -138,7 +138,7 @@ describe('fileHelpers', () => {
const filePath = 'projects/2024/q1/report.md'; const filePath = 'projects/2024/q1/report.md';
const expectedUrl = const expectedUrl =
'http://localhost:8080/api/v1/workspaces/docs/files/projects%2F2024%2Fq1%2Freport.md'; 'http://localhost:8080/api/v1/workspaces/docs/files/content?file_path=projects%2F2024%2Fq1%2Freport.md';
const actualUrl = getFileUrl(workspaceName, filePath); const actualUrl = getFileUrl(workspaceName, filePath);
expect(actualUrl).toBe(expectedUrl); expect(actualUrl).toBe(expectedUrl);
@@ -146,20 +146,20 @@ describe('fileHelpers', () => {
it('handles edge cases with empty strings', () => { it('handles edge cases with empty strings', () => {
expect(getFileUrl('', '')).toBe( expect(getFileUrl('', '')).toBe(
'http://localhost:8080/api/v1/workspaces//files/' 'http://localhost:8080/api/v1/workspaces//files/content?file_path='
); );
expect(getFileUrl('workspace', '')).toBe( expect(getFileUrl('workspace', '')).toBe(
'http://localhost:8080/api/v1/workspaces/workspace/files/' 'http://localhost:8080/api/v1/workspaces/workspace/files/content?file_path='
); );
expect(getFileUrl('', 'file.md')).toBe( expect(getFileUrl('', 'file.md')).toBe(
'http://localhost:8080/api/v1/workspaces//files/file.md' 'http://localhost:8080/api/v1/workspaces//files/content?file_path=file.md'
); );
}); });
it('uses the API base URL correctly', () => { it('uses the API base URL correctly', () => {
const url = getFileUrl('test', 'file.md'); const url = getFileUrl('test', 'file.md');
expect(url).toBe( expect(url).toBe(
'http://localhost:8080/api/v1/workspaces/test/files/file.md' 'http://localhost:8080/api/v1/workspaces/test/files/content?file_path=file.md'
); );
expect(url).toContain(window.API_BASE_URL); expect(url).toContain(window.API_BASE_URL);
}); });

2
app/src/utils/fileHelpers.ts
View File

@@ -13,5 +13,5 @@ export const isImageFile = (filePath: string): boolean => {
export const getFileUrl = (workspaceName: string, filePath: string) => { export const getFileUrl = (workspaceName: string, filePath: string) => {
return `${API_BASE_URL}/workspaces/${encodeURIComponent( return `${API_BASE_URL}/workspaces/${encodeURIComponent(
workspaceName workspaceName
)}/files/${encodeURIComponent(filePath)}`; )}/files/content?file_path=${encodeURIComponent(filePath)}`;
}; };

10
server/internal/handlers/file_handlers.go
View File

@@ -2,9 +2,11 @@ package handlers
import ( import (
"io" "io"
"mime"
"net/http" "net/http"
"net/url" "net/url"
"os" "os"
"path/filepath"
"time" "time"
"lemma/internal/context" "lemma/internal/context"
@@ -205,7 +207,13 @@ func (h *Handler) GetFileContent() http.HandlerFunc {
return return
} }
w.Header().Set("Content-Type", "text/plain") // Detect MIME type based on file extension
contentType := mime.TypeByExtension(filepath.Ext(decodedPath))
if contentType == "" {
// Fallback to text/plain if MIME type cannot be determined
contentType = "text/plain"
}
w.Header().Set("Content-Type", contentType)
_, err = w.Write(content) _, err = w.Write(content)
if err != nil { if err != nil {
log.Error("failed to write response", log.Error("failed to write response",