search

fedora-infra / mirrormanager2

Rewrite of the MirrorManager application in Flask and SQLAlchemy
https://mirrormanager.fedoraproject.org
GNU General Public License v2.0
63 stars 48 forks source link

Remove all whitespaces around the mirror URLs #119

Closed adrianreber closed 8 years ago

adrianreber commented 8 years ago

Some mirrors where failing to be scanned as a leading whitespace broke our protocol detection (.startswith('rsync')). This ignores whitespace in the URLs.

pypingou commented 8 years ago

:+1: easy and safe

adrianreber commented 8 years ago

I have tested the code in the crawler. I was not able to test the code changes in app.py. I hope they make sense.

adrianreber commented 8 years ago

Without testing I got it wrong... Of course. This version now actually works.

pypingou commented 8 years ago

Looks fine to me :+1:

adrianreber commented 8 years ago

Thanks.