Fix: add client-side URL length validation to satisfy test_too_long_uri #1482
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.
Metadata
test_too_long_uricovers this)_send_request()to handle overly long URLs and fixtest_too_long_uri."Details
This PR adds a client-side URL length validation step to
_send_request()inopenml/_api_calls.py.The goal is to prevent extremely long URLs from being sent to the server and to ensure that
test_too_long_uribehaves as expected.🧩 What problem does this fix?
Currently, when a very long list of
data_ids is provided (e.g., 10,000 IDs):🛠️ What does this PR change?
This PR adds a fail-fast check before any network request is made:
This ensures:
✔ Test Status