incubator-allura-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Cory Johns" <masterbunn...@users.sf.net>
Subject [allura:tickets] Re: #7124 Validate Trac URLs before importing
Date Tue, 04 Feb 2014 20:10:06 GMT
The URL checks for all the other importer types are all just doing a 200 check.  I agree something
more robust would be nice but this will catch all the errors we've seen so far.

The URL normalization in `tracwikiexporter.scripts.wiki_from_trac.extractors.WikiExporter`
should be redundant now, since it should happen in the controller, so we could just remove
it, but I wasn't sure why it was put so deep in the first place so I left it.

We could also switch the `WikiExtractor` code to use the validator to do the normalization,
which would at least reduce the duplication.


---

** [tickets:#7124] Validate Trac URLs before importing**

**Status:** in-progress
**Labels:** import 
**Created:** Fri Jan 31, 2014 06:19 PM UTC by Cory Johns
**Last Updated:** Tue Feb 04, 2014 06:50 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.
Mime
  • Unnamed multipart/related (inline, None, 0 bytes)
View raw message