Skip to content

Fix version lookup and content-length handling #7687

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

niklub
Copy link
Collaborator

@niklub niklub commented Jun 4, 2025

Summary

  • avoid PackageNotFoundError when importing label_studio from source
  • guard against missing Content-Length headers when importing tasks from URL
  • add a small regression test for version import

Testing

  • pytest tests/test_version.py -q
  • pytest label_studio/tests/data_import/test_uploader.py::TestUploader::test_user_specified_block_without_default -q (fails: ModuleNotFoundError: No module named 'boto3')

https://chatgpt.com/codex/tasks/task_e_683fa4aa69a083219b471edf1c236cc8

@niklub niklub added the codex label Jun 4, 2025 — with ChatGPT Connector
Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 8c7a2a1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/683fe31c069ba90008aa6f6b

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for label-studio-playground canceled.

Name Link
🔨 Latest commit 8c7a2a1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/683fe31ce0da080008c30c32

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for label-studio-storybook canceled.

Name Link
🔨 Latest commit 8c7a2a1
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/683fe31cb248830008f8ae5d

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 8c7a2a1
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/683fe31cd5f58d00089b47a4

Copy link

sentry-io bot commented Jun 4, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: label_studio/data_import/uploader.py

Function Unhandled Issue
tasks_from_url [**ValidationError: "Failed to parse: { "id": 2, "data": { "image1": "https://m.media-amazon.com/images/I/810x8VKZbyL.AC_SL1500.jpg", "image2": "https://m.media-amazon.com/images/I/614ucfhtevL.jpg", "text1": "Let's rate your de...** ...
Event Count: 1
tasks_from_url ValidationError: ["HTTPSConnectionPool(host='101.43.178.136', port=8080): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPSConnection object at 0x7fe8126dc470>, 'Connection to 101.43.178.136 timed out. (connect timeout=None)'))"] ...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant