Opened 13 years ago

Closed 12 years ago

#4122 closed Bugs (fixed)

Inspect tool does not understand urls surrounded by spaces.

Reported by: Jeremiah Willcock Owned by: Daniel James
Milestone: Boost 1.43.0 Component: inspection script
Version: Boost Development Trunk Severity: Problem
Keywords: Cc:

Description

The inspection script does not recognize "https" as a valid URL protocol, leading to a warning in tools/build/v2/index.html (both in the trunk and release branch).

Change History (3)

comment:1 by Daniel James, 13 years ago

Owner: changed from John Maddock to Daniel James
Status: newassigned
Summary: Inspect tool does not recognize "https" as a valid protocolInspect tool does not understand urls surrounded by spaces.

It actually recognizes https, but it's confused here because the href attribute starts with a space, so it thinks the protocol is ' https'. I think it should strip the spaces (according to HTML5, href has to be a 'valid URL potentially surrounded by spaces', I haven't checked any other HTML versions). Also, the error message should be clearer to avoid such confusion.

comment:2 by Daniel James, 13 years ago

(In [61443]) Ignore spaces surrounding urls. Refs #4122.

comment:3 by Daniel James, 12 years ago

Resolution: fixed
Status: assignedclosed

(In [61828]) Ignore spaces surrounding urls. Fixes #4122.

Note: See TracTickets for help on using tickets.