Skip to content

fix : added off-by-one fix in url_validator.py parse_resource - #1330

Open
tmdeveloper007 wants to merge 1 commit into
komalharshita:mainfrom
tmdeveloper007:#1325
Open

fix : added off-by-one fix in url_validator.py parse_resource#1330
tmdeveloper007 wants to merge 1 commit into
komalharshita:mainfrom
tmdeveloper007:#1325

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

Summary of What Has Been Done

Fixed the off-by-one slice index bug in src/utils/url_validator.py in the parse_resource function. Changed raw[idx + 2:] to raw[idx + len(split_marker):] where split_marker = ": http" (7 characters).

Changes Made

  • src/utils/url_validator.py: Changed URL extraction from raw[idx + 2:].strip() to raw[idx + len(split_marker):].strip()

Impact it Made

  • Ensures URL parsing is correct for all label formats
  • Prevents edge-case failures with long label prefixes

Note

Please assign this PR to the tmdeveloper007 account.

Closes #1325

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

@tmdeveloper007 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

fix : add off-by-one fix in url_validator.py parse_resource

1 participant