DuckWillow / pe

0 stars 0 forks source link

Invalid zoom link not detected in Schedule function #7

Open DuckWillow opened 2 years ago

DuckWillow commented 2 years ago

image.png

Nice try to detect 'The must begin with a “http://” or “https://” in order for the link to be valid.' But more effort should be put in to detect valid website.

nus-pe-bot commented 2 years ago

Team's Response

Thank you for your report.

The syntax of the URL you have given is valid under RFC 2396.

image.png

While we understand that https://https:// is not a valid webpage but according to the RFC 2396, it is a valid URI.

For example, https://localhost:1234// and http://a/ are both considered a valid URI (http://a can be valid if a is mapped to an IP address if your DNS server or /etc/hosts specifies it).

The syntax of URL is defined by RFC 2396: Uniform Resource Identifiers (URI): Generic Syntax

Please refer to http://www.ietf.org/rfc/rfc2396.txt

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your reason]