New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optionally exclude certain protocols from MM #144

Merged
merged 1 commit into from Nov 11, 2015

Conversation

Projects
None yet
2 participants
@adrianreber
Member

adrianreber commented Nov 10, 2015

This change adds the possibility to exclude certain
protocols to be entered in MirrorManager. The default
value in the configuration file of

MM_PROTOCOL_REGEX = '^(?!ftp)(.*)$'

will prohibit that FTP based URLs are entered at all.

The goal (as discussed in #99) is to remove FTP URLs from Fedora's
MirrorManager setup.

Signed-off-by: Adrian Reber adrian@lisas.de

Optionally exclude certain protocols from MM
This change adds the possibility to exclude certain
protocols to be entered in MirrorManager. The default
value in the configuration file of

 MM_PROTOCOL_REGEX = '^(?!ftp)(.*)$'

will prohibit that FTP based URLs are entered at all.

The goal (as discussed in #99) is to remove FTP URLs from Fedora's
MirrorManager setup.

Signed-off-by: Adrian Reber <adrian@lisas.de>
@pypingou

This comment has been minimized.

Show comment
Hide comment
@pypingou

pypingou Nov 11, 2015

Member

Nice, good idea to use the form validator for this, 👍

Member

pypingou commented Nov 11, 2015

Nice, good idea to use the form validator for this, 👍

@adrianreber

This comment has been minimized.

Show comment
Hide comment
@adrianreber

adrianreber Nov 11, 2015

Member

I was just copying what I saw you did for the country code validation. Thanks for the validator introduction 😄

Member

adrianreber commented Nov 11, 2015

I was just copying what I saw you did for the country code validation. Thanks for the validator introduction 😄

adrianreber added a commit that referenced this pull request Nov 11, 2015

Merge pull request #144 from adrianreber/files
Optionally exclude certain protocols from MM

@adrianreber adrianreber merged commit f41feb2 into fedora-infra:master Nov 11, 2015

@adrianreber adrianreber deleted the adrianreber:files branch Dec 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment