github/genai-toolbox
1
0
Fork 0
mirror of https://github.com/googleapis/genai-toolbox.git synced 2026-01-11 08:28:11 -05:00
Code Issues Packages Projects Releases 14 Wiki Activity

fix(tools/looker): looker file content calls should not use url.QueryEscape (#1758)

Browse Source 
## Description

The file_path parameter for the get_project_file and delete_project_file
calls should not be url encoded.
This commit is contained in:
Dr. Strangelove
2025-10-21 14:29:38 -04:00
committed by GitHub
parent 5010a69a0e
commit 336de1bd04
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
internal/tools/looker/lookercommon/lookercommon.go
View File

@@ -316,7 +316,7 @@ func GetProjectFileContent(l *v4.LookerSDK, projectId string, filePath string, o
var result string
path := fmt.Sprintf("/projects/%s/file/content", url.PathEscape(projectId))
query := map[string]any{
"file_path": url.QueryEscape(filePath),
"file_path": filePath,
}
err := l.AuthSession.Do(&result, "GET", "/4.0", path, query, nil, options)
return result, err
@@ -325,7 +325,7 @@ func GetProjectFileContent(l *v4.LookerSDK, projectId string, filePath string, o
func DeleteProjectFile(l *v4.LookerSDK, projectId string, filePath string, options *rtl.ApiSettings) error {
path := fmt.Sprintf("/projects/%s/files", url.PathEscape(projectId))
query := map[string]any{
"file_path": url.QueryEscape(filePath),
"file_path": filePath,
}
err := l.AuthSession.Do(nil, "DELETE", "/4.0", path, query, nil, options)
return err