Add missing auth() calls to file upload/download methods#42
Merged
Conversation
Co-authored-by: OhYee <13498329+OhYee@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix missing auth call in post_file and get_file methods
Add missing auth() calls to file upload/download methods
Jan 22, 2026
Updated documentation strings to correctly reflect synchronous operations instead of asynchronous ones across multiple modules. Also updated import statements to use the correct runtime options module. fix: 修复同步方法文档和导入语句 更新了多个模块中的文档字符串,正确反映同步操作而不是异步操作。 同时更新了导入语句以使用正确的运行时选项模块。 Change-Id: I38a1ea35939d81b9359634ad80a833bb65278a8e Signed-off-by: OhYee <oyohyee@oyohyee.com>
Add CustomSandbox class to the sandbox module's exports to make it available for import and use in other parts of the application. This enables users to utilize the custom sandbox functionality that was previously not exposed through the main module interface. feat(sandbox): 添加 CustomSandbox 到模块导出 将 CustomSandbox 类添加到 sandbox 模块的导出中,使其可在应用程序的其他部分导入和使用。 这使用户能够利用以前未通过主模块接口公开的自定义沙箱功能。 Change-Id: Ida6b9787ae39f33e2e6902950c0d114d9508b84a
OhYee
approved these changes
Jan 22, 2026
OhYee
approved these changes
Jan 22, 2026
Sodawyx
approved these changes
Jan 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
post_file()andget_file()methods (and async variants) in DataAPI were not callingauth(), causing 401 Unauthorized errors onfile_system.upload()andfile_system.download()operations.Changes
auth()calls topost_file_async()andget_file_async()Implementation
Applied the same authentication pattern used in
get_video():This ensures authentication tokens (
X-API-KeyorAgentrun-Access-Token) are properly included in file operation requests.Original prompt
post_file()andget_file()methods missingauth()call in DataAPI #41💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.