Skip to content
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

fix #2039 IPv6 regex accepts invalid formats #2453

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShreySinha02
Copy link

@ShreySinha02 ShreySinha02 commented Sep 14, 2024

Fixed the incorrect IPv6 address formats:

BC:e4d5:c:e7b9::%40i0nccymtl9cwfKo.5vaeXLSGRMe:EDh2qs5wkhnPws5xQKqafjfAMm6wGFCJ.bVFsZfb
1dC:0DF8:62D:3AC::%KTatXocjaFVioS0RTNQl4mA.V151o0RSy.JIu-D-D8.d3171ZWsSJ7PK4YjkJCRN0F
These addresses are now correctly identified as invalid.

Updated the IPv6 address validation regex to correctly handle and reject invalid IPv6 address formats. The previous regex allowed formats that were not compliant with IPv6 standards. This fix ensures that only valid IPv6 addresses are accepted.

The validation structure is based on the corrected regex from branch #2083, which had merge conflicts that were resolved for accurate validation. This reference ensures that the updated regex conforms to the expected IPv6 address format and resolves the issue with invalid formats.

Checklist

  • PR contains only changes related; no stray files, etc.
  • README updated (where applicable)
  • Tests written (where applicable)
  • References provided in PR (where applicable)

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

Successfully merging this pull request may close these issues.

1 participant