Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, deshani, dill0wn, and 10 others

#7160 Trac-Tickets Importer Rejects URL Containing IP Address [ss6716]

Milestone: v1.2.0

Status: closed

Owner: nobody

Labels: p3 (267) support (446) 42cc (432) trac (14) import (116)

Component: General

Reviewer: Dave Brondsema

Updated: 2015-08-20

Created: 2014-02-10

Creator: Chris Tsai

Private: No

[forge:site-support:#6716]

I receive the following message..

'That is not a valid URL'

...when attempting to import tickets from a Trac project URL whose domain is an IP address.

Discussion

Chris Tsai - 2014-02-21

Same with SVN import: [forge:site-support:#6740]

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-02-21

Perhaps we could extend formencode.validators.URL to support IP addresses.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-02-21

Labels: support, p3, trac, import --> p3, support, 42cc, trac, import

Status: open --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-02-28

Closed #546. je/42cc_7160

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-02-28

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-03-04

QA: Dave Brondsema

Milestone: limbo --> forge-backlog
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-03-04

status: code-review --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-03-04

Looks good and works well, but we should have a unit test. Especially since regular expressions are involved - easy for those to go wrong :)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-03-14

Closed #557. Force-pushed je/42cc_7160 (rebase)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-03-14

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-03-27

status: code-review --> closed

Milestone: forge-backlog --> forge-apr-4
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2015-01-05

Milestone: unreleased --> asf_release_1.2.0
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.