Return-Path: X-Original-To: apmail-incubator-allura-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-allura-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5E0FD10A40 for ; Fri, 7 Feb 2014 15:44:51 +0000 (UTC) Received: (qmail 36364 invoked by uid 500); 7 Feb 2014 15:44:51 -0000 Delivered-To: apmail-incubator-allura-dev-archive@incubator.apache.org Received: (qmail 36300 invoked by uid 500); 7 Feb 2014 15:44:45 -0000 Mailing-List: contact allura-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: allura-dev@incubator.apache.org Delivered-To: mailing list allura-dev@incubator.apache.org Received: (qmail 36273 invoked by uid 99); 7 Feb 2014 15:44:41 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 07 Feb 2014 15:44:41 +0000 X-ASF-Spam-Status: No, hits=-0.1 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_MED,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of noreply@sourceforge.net designates 216.34.181.60 as permitted sender) Received: from [216.34.181.60] (HELO smtp.ch3.sourceforge.com) (216.34.181.60) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 07 Feb 2014 15:44:37 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.com; s=x; h=Date:References:In-Reply-To:Message-ID:Subject:Reply-To:From:To:MIME-Version:Content-Type; bh=PnS57JhD1xfz1B9V0Evn2o6tXdLBfODCVkKGiTxB46g=; b=dJ2YW2w6gsgK8DGrxrb/vw6U99YlwdvRMuy9AXp1y2q4ky0/mHOg0Z5Mtl2N1+3KIlmei/uaZBXhhK6QUMX+q+2DHDz5ovk2O5AKf/6H4rx15nn4u1uC/QcKnfiwUUJeoBHQmoN23vfP/qHf+RiFG02wrR9gGd2vLbV7U2D1xPQ=; Received: from localhost ([127.0.0.1] helo=sfs-alluradaemon-1.v29.ch3.sourceforge.com) by sfs-alluradaemon-1.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1WBnbE-0006vC-V2 for allura-dev@incubator.apache.org; Fri, 07 Feb 2014 15:44:16 +0000 Content-Type: multipart/related; boundary="===============8588834736867393605==" MIME-Version: 1.0 To: allura-dev@incubator.apache.org From: "Dave Brondsema" Reply-To: "[allura:tickets] " <7124@tickets.allura.p.re.sf.net> Subject: [allura:tickets] #7124 Validate Trac URLs before importing Message-ID:

In-Reply-To: <52ebe9460594ca512ea0916f.tickets@allura.p.sourceforge.net> References: <52ebe9460594ca512ea0916f.tickets@allura.p.sourceforge.net> Date: Fri, 07 Feb 2014 15:44:16 +0000 X-Virus-Checked: Checked by ClamAV on apache.org --===============8588834736867393605== Content-Type: multipart/alternative; boundary="===============0976728464706708461==" MIME-Version: 1.0 --===============0976728464706708461== MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Found one other issue, unhandled errors from requests if an invalid domain is entered, for example. This should handle it: https://sourceforge.net/p/allura/pastebin/52f4fee43e5e8346ccf15780 If that's good by you, I can commit & merge the branches (there's some rebasing and conflicts that I've got resolved locally already) --- ** [tickets:#7124] Validate Trac URLs before importing** **Status:** code-review **Labels:** import **Created:** Fri Jan 31, 2014 06:19 PM UTC by Cory Johns **Last Updated:** Thu Feb 06, 2014 08:35 PM UTC **Owner:** Cory Johns https://sourceforge.net/nf/admin/task_manager/view/52ebe3f21be1ce2af7b78b31 Validate Trac URLs so we don't get 404s or spurious NoneType errors during import. --- Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. --===============0976728464706708461== MIME-Version: 1.0 Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: 7bit

Found one other issue, unhandled errors from requests if an invalid domain is entered, for example. This should handle it: https://sourceforge.net/p/allura/pastebin/52f4fee43e5e8346ccf15780 If that's good by you, I can commit & merge the branches (there's some rebasing and conflicts that I've got resolved locally already)

[tickets:#7124] Validate Trac URLs before importing

Status: code-review
Labels: import
Created: Fri Jan 31, 2014 06:19 PM UTC by Cory Johns
Last Updated: Thu Feb 06, 2014 08:35 PM UTC
Owner: Cory Johns

https://sourceforge.net/nf/admin/task_manager/view/52ebe3f21be1ce2af7b78b31

Validate Trac URLs so we don't get 404s or spurious NoneType errors during import.

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

--===============0976728464706708461==-- --===============8588834736867393605==--