Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@SabrinaTardio
Copy link
Collaborator

@SabrinaTardio SabrinaTardio commented Aug 16, 2023

Task/Issue URL: https://app.asana.com/0/1177771139624306/1203127427026896/f

Description: Makes IPv6 valid url when used in navigation.

Steps to test this PR:

  1. Type on the search bar http://[::]:8080/ it should navigate to it rather than searching
  2. Try the same for other IPv6 or IPv4 addresses ex:
    http://192.168.0.1/
    http://203.0.113.0/
    http://[2001:0db8:85a3:0000:0000:8a2e:0370:7334]
    http://[2001:0db8::1]

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@SabrinaTardio SabrinaTardio self-assigned this Aug 16, 2023
@SabrinaTardio SabrinaTardio marked this pull request as ready for review August 16, 2023 15:40
Copy link
Contributor

@tomasstrba tomasstrba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! ✅

@SabrinaTardio SabrinaTardio merged commit 08f6e75 into develop Aug 21, 2023
@SabrinaTardio SabrinaTardio deleted the sabrina/extend-isValidURL-to-include-IPv6 branch August 21, 2023 15:16
samsymons added a commit that referenced this pull request Aug 22, 2023
# By Diego Rey Mendez (3) and others
# Via Diego Rey Mendez (1) and GitHub (1)
* develop:
  Require Duck Player scheme URL to be passed from YouTube Overlay User Script (#1519)
  extend valid urls to include IPv6 (#1492)
  Fix bug where Back navigation doesn't work properly with bangs (#1510)
  remove logic to keep previous search mode (#1500)
  Bump version to 1.53.0 (52)
  Update embedded files
  Adds a file that's generated when the unit tests are run (#1511)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants